Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 187479 - qt[34] eclasses inject bogus debug USE flag into lots of ebuilds
Summary: qt[34] eclasses inject bogus debug USE flag into lots of ebuilds
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Eclasses (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: Qt Bug Alias
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 120116
  Show dependency tree
 
Reported: 2007-08-02 09:19 UTC by Jakub Moc (RETIRED)
Modified: 2007-08-02 21:24 UTC (History)
1 user (show)

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

Attachments
qt3.eclass patch (qt3.eclass.diff,563 bytes, patch)
2007-08-02 09:30 UTC, Jakub Moc (RETIRED) 		Details | Diff
qt4.eclass patch (qt4.eclass.diff,620 bytes, text/plain)
2007-08-02 09:32 UTC, Jakub Moc (RETIRED) 		Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Jakub Moc (RETIRED) gentoo-dev 2007-08-02 09:19:46 UTC 
See net-im/skype for example. Please, make this optional (see gnome2.eclass for example) how to do this) at least.
Comment 1 Jakub Moc (RETIRED) gentoo-dev 2007-08-02 09:30:53 UTC 
Created attachment 126687 [details, diff]
qt3.eclass patch
Comment 2 Jakub Moc (RETIRED) gentoo-dev 2007-08-02 09:32:36 UTC 
Created attachment 126688 [details]
qt4.eclass patch

Forget the gnome2.eclass suggestion; seeing the loads of ebuilds that have grown themselves a debug use flag overnight for no reason, I made it disabled by default. If an ebuild has debug configure option, it should state it in its own IUSE.
Comment 3 Davide Pesavento (RETIRED) gentoo-dev 2007-08-02 10:24:01 UTC 
You're right jakub. Injection of 'debug' in IUSE was an idea of troll. I didn't change that because I thought he had a good reason to do that, but now I see that this effectively turns out to be completely wrong rather than being an improvement.

Caleb: please apply jakub's patch ASAP.
Comment 4 Caleb Tennis (RETIRED) gentoo-dev 2007-08-02 11:22:53 UTC 
should be fixed.
Comment 5 Arfrever Frehtes Taifersar Arahesis (RETIRED) gentoo-dev 2007-08-02 14:57:42 UTC 
IUSE="${IUSE}" is needless.
Comment 6 Davide Pesavento (RETIRED) gentoo-dev 2007-08-02 21:24:18 UTC 
Yeah, that line should be completely removed.