[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] maint.mk: get current gnulib revision correctly.
From: |
Jim Meyering |
Subject: |
Re: [PATCH] maint.mk: get current gnulib revision correctly. |
Date: |
Mon, 28 Oct 2013 16:45:53 -0700 |
On Mon, Oct 28, 2013 at 4:30 PM, Eric Blake <address@hidden> wrote:
> On 10/28/2013 05:10 PM, Jim Meyering wrote:
>>
>> * gnulib-tool: Some "cd" built-in functions print a directory name
>> to stdout when CDPATH is set, e.g.,
>
> More precisely, _all_ posix-conforming cd implementations are required
> to write to stdout if CDPATH had an effect. It's more than just "some".
>
>> +++ b/gnulib-tool
>> @@ -898,6 +898,9 @@ if test "X$1" = "X--no-reexec"; then
>> shift
>> fi
>>
>> +# Unset CDPATH. Otherwise, output from the 'cd dir' can surprise callers.
>> +(unset CDPATH) >/dev/null 2>&1 && unset CDPATH
>
> This comment wording is also a nicer alternative (by bypassing any
> implication on how many shells give output and without a judgment on
> whether such behavior is wrong, when it is clearly required by posix).
Hi Eric,
Since zsh's cd does not print anything by default, using "some"
seemed appropriate; it is probably the second most popular login shell.
Actually, I think the first incarnation of my log message used "Many", but
I scaled it back when I tried it.
Given that most shells ever linked to /bin/sh probably do have this feature,
I'll revert to using the stronger word.
Thanks,
Jim
- Re: [PATCH] maint.mk: get current gnulib revision correctly., (continued)