Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 403123

Summary: app-shells/gentoo-bashcomp - Add ebuild(1) --skip-manifest completion support
Product: Gentoo Linux Reporter: Sergei Shilovsky <sh13ld>
Component: [OLD] UnspecifiedAssignee: Gentoo Shell Tools project <shell-tools>
Status: UNCONFIRMED ---    
Severity: enhancement CC: sh13ld
Priority: Normal    
Version: unspecified   
Hardware: All   
OS: Linux   
Whiteboard:
Package list:
Runtime testing required: ---
Attachments: Fixed implementation of bash completion for ebuild command
Refixed implementation

Description Sergei Shilovsky 2012-02-11 05:32:58 UTC
after 'ebuild --skip-manifest' completion does not work properly

Reproducible: Always

Steps to Reproduce:
1. # emerge gentoo-bashcomp
2. # ebuild --skip-m<Tab>anifest <Tab><Tab>

Actual Results:  
clean      config     digest     help       manifest   package    postrm     prepare    qmerge     setup      unmerge    
compile    configure  fetch      install    merge      postinst   preinst    prerm      rpm        test       unpack

Expected Results:  
list of ebuild files in current directory

--skip-manifest is completed only right after 'ebuild'
Comment 1 Sergei Shilovsky 2012-02-13 17:23:49 UTC
To be more precise, it DOES complete --skip-manifest and other options as well as ebuild file names.
But as soon as one option or ebuild file name is typed it allows only command completion (e.g. digest, clean etc.) which prohibits completion of statements like 'ebuild --skip-manifest my.ebuild digest' or 'ebuild my.ebuild --skip-manifest digest' or 'ebuild my.ebuild digest --skip-manifest'
Comment 2 Alexey Sarychev 2013-11-08 04:43:39 UTC
Created attachment 362770 [details, diff]
Fixed implementation of bash completion for ebuild command

That bug bugged me for quite some time - finally I got around to looking into it.
Current implementation for ebuild bashcomp is broken, patch with a working one is attached.
Comment 3 Alexey Sarychev 2013-11-08 04:46:42 UTC
Created attachment 362772 [details, diff]
Refixed implementation

Small clean-up of the previous patch - one line was redundant, sorry.