Zsh Mailing List Archive
Messages sorted by: Reverse Date, Date, Thread, Author

Re: [PATCH] Add missing commands to Debian apt completion



Laurent Arnoud wrote on Fri, Nov 25, 2016 at 17:19:05 +0100:
> On Fri, Nov 25, 2016 at 09:35:51AM +0000, Daniel Shahaf wrote:
> > Laurent Arnoud wrote on Tue, Nov 22, 2016 at 19:31:37 +0100:
> > > +++ b/Completion/Debian/Command/_apt
> > > @@ -402,17 +402,29 @@ _apt-cmd () {
> > >      /$'show\0'/ /$'[^\0]#\0'/ ':packages::_deb_packages "$expl_packages[@]" avail' \# \
> > > +    /$'showsrc\0'/ /$'[^\0]#\0'/ ':packages::_deb_packages "$expl_packages[@]" avail' \# \| \
> > 
> > I had to move the 'showsrc' line to above the 'show' line for «apt
> > showsrc <TAB>» to work.  Is this expected, or a bug in _apt_arguments
> > or _regex_arguments?
> 
> Yes you are right, I'm not sure about the bug but it is done like that in
> `_apt-cache` function. I will just do as you did since I'm not an expert with
> this.

Actually, I looked again and a «\|» is missing from the 'list' and
'show' lines.  I'll push your new patch with this modification.  Thanks
for the patch!



Messages sorted by: Reverse Date, Date, Thread, Author