First Last Prev Next    No search results available      Search page      Enter new bug
Bug#: 207937
Alias:
Product:
Component:
Status: RESOLVED
Resolution: DUPLICATE of bug 158066
Assigned To: Gentoo KDE team <kde@gentoo.org>
Hardware:
OS:
Version:
Priority:
Severity:
Reporter: Jeroen Roovers <jer@gentoo.org>
Add CC:
CC:
Remove selected CCs
URL:
Summary:
Status Whiteboard:
Keywords:

Filename Description Type Creator Created Size Actions
kdesdk-207937-valgrind-removal.patch =kdesdk-207937-valgrind-removal.patch patch Jeroen Roovers 2008-02-11 16:58 0000 3.16 KB Details | Diff
Create a New Attachment (proposed patch, testcase, etc.) View All

Bug 207937 depends on: Show dependency tree
Bug 207937 blocks: 209443
Votes: 0    Show votes for this bug    Vote for this bug

Additional Comments: (this is where you put emerge --info)


Not eligible to see or edit group visibility for this bug.






View Bug Activity   |   Format For Printing   |   XML   |   Clone This Bug


Description:   Opened: 2008-01-28 17:29 0000
gentoo/cvs/gentoo-x86/kde-base/kdesdk $ grep valgrind kdesdk-*
kdesdk-3.5.5.ebuild:DEPEND="x86? ( >=dev-util/valgrind-3.2.0 )
kdesdk-3.5.6.ebuild:DEPEND="x86? ( >=dev-util/valgrind-3.2.0 )
kdesdk-3.5.6-r1.ebuild:DEPEND="!alpha? ( !sparc? ( !ia64? (
>=dev-util/valgrind-3.2.0 ) ) )
kdesdk-3.5.7.ebuild:DEPEND="!alpha? ( !sparc? ( !ia64? (
>=dev-util/valgrind-3.2.0 ) ) )
kdesdk-3.5.8.ebuild:DEPEND="!alpha? ( !sparc? ( !ia64? (
>=dev-util/valgrind-3.2.0 ) ) )

1) Rather than adding arch after arch that valgrind has no use for, it would be
better to simply add a USE=valgrind, use.mask that generally and reenable it
specifically for x86.

2) If that's too much trouble, then maybe it's better to not pull in valgrind
at all, in exactly the same way that kdesdk-4 doesn't, and maybe add an einfo
to the effect that installing valgrind might help using kdesdk. I suggest this
because the ebuilds and eclasses in no way use valgrind, which in turn suggests
that the build system automagically depends on valgrind or that valgrind is
merely a runtime dependency and should be in RDEPEND, not DEPEND.

------- Comment #1 From Jakub Moc (RETIRED) 2008-01-28 18:16:54 0000 -------
This is basically a dupe of Bug 158066 for kde-base/kcachegrind; I reopened it
now. I don't get this either.

------- Comment #2 From Jeroen Roovers 2008-02-11 16:58:39 0000 -------
Created an attachment (id=143219) [details]
=kdesdk-207937-valgrind-removal.patch

------- Comment #3 From Bo Ørsted Andresen (RETIRED) 2008-02-18 22:39:59 0000 -------
Thanks. The dependency is going away. It isn't needed at all at run or build
time.

*** This bug has been marked as a duplicate of bug 158066 ***

First Last Prev Next    No search results available      Search page      Enter new bug