[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [patch] skip one more glade test
From: |
Bruno Haible |
Subject: |
Re: [patch] skip one more glade test |
Date: |
Sun, 23 May 2010 01:27:39 +0200 |
User-agent: |
KMail/1.9.9 |
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