Re: [patch] skip one more glade test

[Bruno Haible]

Hi Matthew,

> Would you accept the attached patch then, that makes
> all glade-related tests consistent by having them all fail in the absence
> of glade support in xgettext?  My intention with this and the previous
> patch is simply to make all the glade tests produce consistent results

You are entirely right. I completely missed the fact that I had already
decided in the opposite sense than what I explained in my last mail.

Thanks. I've applied your patch.

> Whilst on the subject, is there anything else that upstream expect
> rather than simply 'recommend' (as per the DEPENDENCIES') file?  Is it
> worth perhaps marking such dependencies as 'Mandatory' rather than simply
> 'Recommended'?

Well, the categories in the DEPENDENCIES file are "Mandatory",
"Highly recommended", "Recommended", and "Optional". There are also
explanations for which use-case they are "recommended". Ultimately,
you have to decide by yourself and by consensus with your users which
use-cases are important for them.

> My understanding is that cvs/git are runtime dependencies?

Good point. For the next release, I'll clarify which dependencies are
- runtime dependencies that are also build dependencies.
- runtime dependencies, but not build dependencies.
- build dependencies, but not runtime dependencies.

> As such, 
> I would think that adding '--with-git' to LFS would be the right thing
> to do so that our users can avoid installing the legacy CVS package, if
> they need to run 'autopoint'?

The dependency towards 'cvs' and 'git' is a bit complicated [1]:

* runtime dependencies that are also build dependencies:
  * git 1.6 or newer
    + Recommended.
      Needed by the 'autopoint' program, if configured with --with-git.

* runtime dependencies, but not build dependencies:
  * CVS 1.11 or newer
    + Recommended.
      Needed by the 'autopoint' program, if not configured with --without-cvs
      or --with-git.

* build dependencies, but not runtime dependencies:
  * CVS 1.11 or newer
    + Recommended.
      Needed if configuring with --without-cvs or --with-git.
  * RCS 5.7 or newer
    + Recommended.
      Needed if configuring with --without-cvs or --with-git.

The objective of all this is, indeed, that users can avoid installing the
legacy CVS package (in particular because of the undocumented privacy issues
that it has [2]).

Bruno

[1] http://git.savannah.gnu.org/gitweb/?p=gettext.git;a=blob;f=DEPENDENCIES
[2] http://lists.gnu.org/archive/html/bug-cvs/2007-01/msg00019.html