Santiago Vila <[EMAIL PROTECTED]> writes:

> On Thu, 13 Dec 2007, Simon Josefsson wrote:
>
>> Santiago Vila <[EMAIL PROTECTED]> writes:
>>
>> > I think you should have read /usr/share/doc/gettext/README.Debian before
>> > reporting this as a bug.
>> 
>> Ah, sorry.  I did look through old bug reports first, but there is
>> always some documentation you can fail to read...
>> 
>> Still, the situation seems sub-optimal.  Could there not be a package
>> 'gettext-autopoint' that could contain autopoint, and depend on Cvs?
>
> If I do that, there will be someone who will ask: I installed the gettext
> package, where the $%&#@ is autopoint?
>
> I'm trying not to deviate too much from the PACKAGING file in the
> source code, which recommends two packages, and we already have four
> of them.
>
> So, there is no optimal solution, I expect that the current one is
> good enough and people read the READMEs.

The patch below would have avoided the bug report from me.

Which gives the following user experience:

[EMAIL PROTECTED]:~/src/gnutls$ autopoint
autopoint: *** cvs program not found, see /usr/share/doc/gettext/README.Debian
autopoint: *** Stop.
[EMAIL PROTECTED]:~/src/gnutls$

It even fits in 80 columns. :)

I understand if you think the extra maintenance work associated with
maintaining patches is too much work for too little gain though.  I just
wanted to offer the obvious patch for consideration; I recall seeing
debian-specific hints in error messages before.

Thanks,
/Simon

--- /usr/bin/autopoint.orig     2007-12-14 11:46:38.000000000 +0100
+++ /usr/bin/autopoint  2007-12-14 11:52:32.000000000 +0100
@@ -340,7 +340,7 @@
 # users of CVS.
 #
 # Check availability of the CVS program.
-(cvs -v) >/dev/null 2>/dev/null || func_fatal_error "cvs program not found"
+(cvs -v) >/dev/null 2>/dev/null || func_fatal_error "cvs program not found, 
see /usr/share/doc/gettext/README.Debian"
 
 # Check in which directory config.rpath, mkinstalldirs etc. belong.
 auxdir=`cat "$configure_in" | grep '^AC_CONFIG_AUX_DIR' | sed -n -e 
's/AC_CONFIG_AUX_DIR(\([^()]*\))/\1/p' | sed -e 's/^\[\(.*\)\]$/\1/' | sed -e 
1q`



-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Reply via email to