Re: [PATCH] maint.mk: get current gnulib revision correctly.

[Jim Meyering]

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