Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 93853 - genlop bash-completion doesn't use PORTDIR
Summary: genlop bash-completion doesn't use PORTDIR
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Other
: High normal (vote)
Assignee: Portage Tools Team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-05-24 14:33 UTC by Julien Allanos (RETIRED)
Modified: 2007-04-02 12:57 UTC (History)
2 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
Grabs PORTDIR from /etc/make.conf or defaults to /usr/portage (genlop-portdir.patch,688 bytes, patch)
2005-07-21 14:02 UTC, Jeroen Roovers (RETIRED)
Details | Diff
A patch to include in Portage directly. (genlop-0.30.3-portdir.patch,720 bytes, patch)
2005-07-21 14:48 UTC, Jeroen Roovers (RETIRED)
Details | Diff
Fix for genlop's bash-completion if PORTDIR isn't set in make.conf (patch,431 bytes, patch)
2005-10-07 03:51 UTC, Mike
Details | Diff
Or something like this... (genlop-0.30.5-bashcomp-portdir.patch,868 bytes, patch)
2005-10-07 04:59 UTC, Jeroen Roovers (RETIRED)
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Julien Allanos (RETIRED) gentoo-dev 2005-05-24 14:33:12 UTC
genlop bash-completion script should use PORTDIR instead using /usr/portage for portagedir (it is hard-coded atm). sorry, no time to provide a fix now...
Comment 1 Jeroen Roovers (RETIRED) gentoo-dev 2005-07-21 14:02:12 UTC
Created attachment 64011 [details, diff]
Grabs PORTDIR from /etc/make.conf or defaults to /usr/portage
Comment 2 Jeroen Roovers (RETIRED) gentoo-dev 2005-07-21 14:48:42 UTC
Created attachment 64016 [details, diff]
A patch to include in Portage directly.
Comment 3 Antonio Dolcetta 2005-07-28 08:12:38 UTC
fixed in next version
thank you
Comment 4 Mike 2005-10-07 03:51:41 UTC
Created attachment 70061 [details, diff]
Fix for genlop's bash-completion if PORTDIR isn't set in make.conf

The attached patch quotes ${portagedir} so it also works when $PORTDIR is not
set in /etc/make.conf.

Besides, genlop's bash-completion does not complete packages specified without
their category on the commandline, unlike emerge's completion does, for
example.

Therefore i think it's probably good to copy the relevant parts from emerge's
completion and implement them here?

Maybe i'll try if noone else does, but i'm no bash coder... if bash coding just
wouldn't be so ugly :(
Comment 5 Jeroen Roovers (RETIRED) gentoo-dev 2005-10-07 04:59:12 UTC
Created attachment 70066 [details, diff]
Or something like this...

Get PORTDIR from make.globals, then get PORTDIR from make.conf, then see if
${portagedir} is a (symlink to a) directory and only if that fails, set
${portagedir} to "/usr/portage".
Comment 6 Jeroen Roovers (RETIRED) gentoo-dev 2005-10-07 05:00:07 UTC
(In reply to comment #5)
> Created an attachment (id=70066) [edit]
> Or something like this...
> 
> Get PORTDIR from make.globals, then get PORTDIR from make.conf, then see if
> ${portagedir} is a (symlink to a) directory and only if that fails, set
> ${portagedir} to "/usr/portage".

That's a patch against genlop-0.30.5, btw.
Comment 7 Michael Cummings (RETIRED) gentoo-dev 2007-04-02 12:57:39 UTC
genlop has been moved into gentoo's svn per permission from the original authors to take over maintainership. I've adjusted the sourcing of make.conf to include make.global per the patch offered in this bug and those changes are available in  svn, thanks!