Summary: | has_version and built_with_use ignore package.provided | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Paul Bredbury <brebs> |
Component: | [OLD] Core system | Assignee: | Gentoo's Team for Core System packages <base-system> |
Status: | RESOLVED WONTFIX | ||
Severity: | normal | CC: | mikachu |
Priority: | High | ||
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Attachments: |
portageq.patch
eutils.eclass.patch irc.log |
Description
Paul Bredbury
2006-07-09 20:54:25 UTC
Created attachment 93098 [details, diff]
portageq.patch
Fixes has_version to look in package.provided.
Created attachment 93099 [details, diff]
eutils.eclass.patch
Fixes built_with_use to use the fixed has_version.
if you're using built_with_use on a package that isnt installed, then your ebuild logic is flawed in a larger sense perhaps the correct fix would be to just call 'die' (In reply to comment #3) > if you're using built_with_use on a package that isnt installed, then your > ebuild logic is flawed in a larger sense Nope, it is Portage's logic that is flawed (and fixed by these patches). Having to work around Portage's flaws in ebuilds, rather than fixing Portage's commands so that they do not produce the *opposite* result of what anyone would reasonably expect, is no way to program. There are two bugs which these patches fix: * has_version ignores package.provided. * built_with_use is wrong, as mentioned above. Should I split this into 2 bugs rather than 1? two different issues file a new bug about the has_version as that is a portage issue as for built_with_use, my previous comment still stands Created attachment 93613 [details]
irc.log
Here's the discussion on IRC, for reference. Note that the sentence "i can come up with a circumstance going the opposite direction where yours breaks but mine works" on line 94 turns out to be false.
*** Bug 163435 has been marked as a duplicate of this bug. *** |