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

Bug 621468

Summary: app-portage/gentoolkit-0.4.0 Update the manpage for revdep-rebuild SEARCH_DIRS_MASK
Product: Portage Development Reporter: Chris Smith <chris>
Component: ToolsAssignee: Portage Tools Team <tools-portage>
Status: CONFIRMED ---    
Severity: normal CC: jstein
Priority: Normal    
Version: unspecified   
Hardware: All   
OS: Linux   
Whiteboard:
Package list:
Runtime testing required: ---

Description Chris Smith 2017-06-11 02:59:59 UTC
Python coded revdep-rebuild ignores SEARCH_DIRS_MASK in make.conf.
The script revdep-rebuild.sh works properly.
Comment 1 Brian Dolbec (RETIRED) gentoo-dev 2017-06-12 00:33:12 UTC
The correct place for masks is in /etc/revdep-rebuild/99revdep-rebuild.

Application ebuilds also install mask files there on an as needed basis.

Please move you masks from make.conf to the /etc/revdep-rebuild directory.

The only reason the bash version picks it up there is because it sources make.conf for some settings.  The python version gets those settings from portage api calls.
Comment 2 Chris Smith 2017-06-12 01:19:47 UTC
(In reply to Brian Dolbec from comment #1)
> The correct place for masks is in /etc/revdep-rebuild/99revdep-rebuild.
> 
> Application ebuilds also install mask files there on an as needed basis.
> 
> Please move you masks from make.conf to the /etc/revdep-rebuild directory.
> 
> The only reason the bash version picks it up there is because it sources
> make.conf for some settings.  The python version gets those settings from
> portage api calls.

Then one would think that the man page should reflect this and that also the SEARCH_DIRS variable in /etc/revdep-rebuild/99revdep-rebuild would be adhered to, which it isn't as many more paths are searched besides those listed.
Comment 3 Brian Dolbec (RETIRED) gentoo-dev 2017-06-12 01:54:42 UTC
Good point.  We'll have to update the man page properly.

And yes the revdep-rebuild directory is additive for both bash and python versions.  So as a pkg installs a mask file, it then becomes part of the total masks.

Re-opened for that.
Comment 4 Paul Varner (RETIRED) gentoo-dev 2017-06-12 16:30:56 UTC
This is a change in the intended behavior that was designed into the bash script.

We definitely need to discuss and document how to do it correctly for the python version. The biggest issue that will bite people is /etc/revdep-rebuild is not under configuration protection. So you don't want to modify files placed there by ebuilds or changes will be lost.