Summary: | net-analyzer/wireshark: dumpcap linked against libcap but missing from RDEPEND ? | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | René 'Necoro' Neumann <gentoo> |
Component: | Eclasses | Assignee: | Gentoo Netmon project <netmon> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | vapier, zerochaos |
Priority: | Normal | ||
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- |
Description
René 'Necoro' Neumann
2013-04-22 12:31:06 UTC
mmm, no. if a package links against libcap, then it needs to express that dependency (both in configure & DEPEND) themselves. fcaps.eclass is not a paper over ebuild bugs. that said, wireshark does include libcap already in RDEPEND & DEPEND. so i don't see what you're talking about. you didn't provide `emerge --info` like you need to with every bug report. wireshark-1.6.13.ebuild: RDEPEND=... caps? ( qsys-libs/libcap ) ... (In reply to comment #1) > wireshark-1.6.13.ebuild: > RDEPEND=... caps? ( qsys-libs/libcap ) ... 10 Mar 2013; Jeroen Roovers <jer@gentoo.org> -wireshark-1.6.13.ebuild, -wireshark-1.8.5.ebuild: Old. mmm, the change is incorrect. ebuilds should not be messing with USE=filecaps. wireshark should only be using USE=caps. (In reply to comment #3) > mmm, the change is incorrect. ebuilds should not be messing with > USE=filecaps. wireshark should only be using USE=caps. Provide a patch or argue how it is now wrong, please. (In reply to comment #4) the patch is simple: revert what you committed. as for the why, read the Gentoo thread on the eclass. this USE flag is *only* for the eclass. ebuilds should only be using USE=caps. (In reply to comment #5) > (In reply to comment #4) > > the patch is simple: revert what you committed. as for the why, read the > Gentoo thread on the eclass. this USE flag is *only* for the eclass. > ebuilds should only be using USE=caps. Now you need to explain 1) why USE=filecaps cannot be used in ebuilds (especially since it is exposed by the eclass to its users anyway - it's not like an internal variable or something) 2) why users should be juggling both IUSE="caps filecaps" when a single USE flag covers the entire issue. 3) why reverting to the former incorrect state is better than what is there now. 4) how the result of the "Gentoo thread" (gentoo-dev@ mailing list?) is reflected in the eclass itself (which is what I read to figure out how to use it, not some flamewar on a mailing list). (In reply to comment #6) like i already told you, read the gentoo dev thread on the topic. i'm not going to copy & paste the thread for you. considering i authored the eclass, i don't know what else you need: USE=filecaps is *only* for the eclass, and USE=caps is only for the ebuilds. (In reply to comment #1) > wireshark-1.6.13.ebuild: > RDEPEND=... caps? ( qsys-libs/libcap ) ... Uh - that's why I explicitly stated 'wireshark-1.8.6'. And this did _not_ include this dep (it does now, but it's up to you discussing the right way to handle it). If emerge --info is still needed, I can supply it tomorrow (I just thought it being useless, as the error seemed obvious and easily verifiable -- I was wrong obviously). I have no idea whether it is fixed now, but let's say that it is and that people can file duplicate bug reports asking what the difference between caps and filecaps is supposed to accomplish. (In reply to comment #9) yes, the current ebuild looks correct, thanks |