[pulseaudio-discuss] [PATCH 4/6] shell-completion: zsh: Don't use multiple option sets for one command

Tanu Kaskinen tanu.kaskinen at linux.intel.com
Sat Aug 16 23:31:50 PDT 2014


On Sat, 2014-08-16 at 20:15 +0200, Damir Jelić wrote:
> On Sat, Aug 16, 2014 at 02:48:17PM +0300, Tanu Kaskinen wrote:
> > Dividing the options to multiple sets was misguided (I don't know what
> > the original motivation was). For example, with pacat the -s and
> > --server options were specified as one set. It meant that after the
> > user had written "pactl --server foo", further completions stopped to
> > work. The "server" option set didn't contain any other options, so
> > once Zsh detected that the "server" option set was in use, it thought
> > that no other options were valid.
> > 
> > The special casing for "-s", "-n", "--server" and "--client-*" at the
> > end of _pactl_completion() was probably an attempt to deal with this
> > problem. Those special cases are unnecessary now that the option
> > specification given to _arguments is more correct.
> 
> The motivation was to stop the completions of -s if --server was
> provided (or vice versa).

Ok, I updated the commit message with this information.

> I still don't know how to do this properly, so
> I guess it's better to just complete it instead to leave a broken
> hack there.
> 
> ACK on this and on the rest of the patches.

Thanks, I now pushed the set.

-- 
Tanu



More information about the pulseaudio-discuss mailing list