[Bash-completion-devel] [SCM] bash-completion branch, master, updated. 65d9348f3e5a31adf38ace19a3c1d4e6a302fe9e

[Ville Skyttä]

On Saturday 23 May 2009, David Paleino wrote:
> Hello,
>
> On Sat, 23 May 2009 08:45:49 +0000, Ville Skyttä wrote:
> > The following commit has been merged in the master branch:
> > commit 65d9348f3e5a31adf38ace19a3c1d4e6a302fe9e
> > Author: Ville Skyttä <ville.skytta at iki.fi>
> > Date:   Sat May 23 11:45:39 2009 +0300
> >
> >     Fix man completion (was broken by recent $UNAME/$USERLAND changes).
>
> How was that broken?
> man(1) completion is working here, and I haven't yet pulled this commit.

$USERLAND is not set inside _man() when it runs (and it shouldn't be):

$ man + local cur prev sect manpath manext mansect UNAME
+ manext='@([0-9lnp]|[0-9][px]|man)?(.@(gz|bz2|lzma))'
+ mansect='@([0-9lnp]|[0-9][px])'
+ COMPREPLY=()
++ _get_cword
++ [[ 0 -eq 0 ]]
++ printf %s ''
+ cur=
+ prev=man
+ [[ man == -l ]]
+ _expand
+ [[ '' == \~*/* ]]
+ [[ '' == \~* ]]
+ [[ '' == */* ]]
++ uname -s
+ UNAME=Linux
+ UNAME=Linux
+ '[' = GNU -o Linux = FreeBSD -o Linux = Cygwin ']'
bash: [: too many arguments

It could be that it worked before USERLAND was unset at the end of 
bash_completion by another commit which I requested, perhaps you weren't 
running with a version that had that commit applied?