Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 57068 - issue with ebuild.allmasked
Summary: issue with ebuild.allmasked
Status: RESOLVED FIXED
Alias: None
Product: Portage Development
Classification: Unclassified
Component: Repoman (show other bugs)
Hardware: All All
: High normal (vote)
Assignee: Portage team
URL:
Whiteboard:
Keywords: InVCS
Depends on:
Blocks:
 
Reported: 2004-07-14 13:04 UTC by Thomas Matthijs (RETIRED)
Modified: 2004-08-16 11:04 UTC (History)
0 users

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


Attachments
repoman-ebuild.allmasked.patch (repoman-fix.patch,716 bytes, patch)
2004-07-14 13:50 UTC, Brian Harring (RETIRED)
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Thomas Matthijs (RETIRED) gentoo-dev 2004-07-14 13:04:25 UTC
in dev-java/sun-jdk, repoman tells me
  ebuild.allmasked     1
   dev-java/sun-jdk

which is apearly wrong
>=1.49 is package.masked, it should possible tell me that
(i'm not totaly sure whats wrong, ferringb told me on irc its a bug 
"There is something funky going on w/ gvisible")
Comment 1 Brian Harring (RETIRED) gentoo-dev 2004-07-14 13:40:51 UTC
repoman is directly setting portage.groups around line 710; what is happening, is that portage.groups isn't being reset the system default, and this is getting flagged.

So, either the check should be moved above the arch check, or portage.groups should be reset.

Easy way to verify this is the issue is run
cd ${PORTDIR}/dev-java/sun-jdk
repoman scan --ignore-other-arches

And watch ebuild.allmasked no longer appear :)
This is kind of an oddball check imo.  It shouldn't be limited to the users groups; in other words, portage.groups ought to be set to all arches (since it's entirely possible a dev may need to commit a syntax correction on an ebuild that isn't available on their arch).
Comment 2 Brian Harring (RETIRED) gentoo-dev 2004-07-14 13:50:54 UTC
Created attachment 35418 [details, diff]
repoman-ebuild.allmasked.patch

Resets portage.groups to keywords "*", eg ebuild.allmasked only is flagged when
the ebuild isn't available for any arch, period (package.mask or no keywords).
Also corrected the indentation.
Comment 3 Jon Hood (RETIRED) gentoo-dev 2004-08-07 20:26:12 UTC
I ran into this just committing net-p2p/dc-qt and net-p2p/limewire, though KEYWORDS is correctly set... using _pre17
Comment 4 Marius Mauch (RETIRED) gentoo-dev 2004-08-07 20:44:44 UTC
I don't see a problem with those packages, all ebuilds there are masked as ~arch so the warning is valid.
Comment 5 Jon Hood (RETIRED) gentoo-dev 2004-08-07 21:09:05 UTC
you're right; I missunderstood the warning. I thought it was referring to masking a whole package in package.mask. Sorry for the confusion :-\.
Comment 6 Brian Harring (RETIRED) gentoo-dev 2004-08-16 11:04:11 UTC
commited this in pre14 (might have been pre15). Either way, it's fixed.