Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 540430 - app-cdr/cdrtools-* missing RDEPEND on libcap
Summary: app-cdr/cdrtools-* missing RDEPEND on libcap
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Eclasses (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Rick Farina (Zero_Chaos)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-02-18 05:15 UTC by Rick Farina (Zero_Chaos)
Modified: 2015-02-18 16:55 UTC (History)
1 user (show)

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


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Rick Farina (Zero_Chaos) gentoo-dev 2015-02-18 05:15:40 UTC
While testing kensington's depcheck and trying to work it into a QA check for portage I came across what I thought was an interesting false positive:

 * Messages for package app-cdr/cdrtools-3.01_alpha17:

 * /usr/lib64/librscg.so.1.0 links to /lib64/libcap.so.2
 *   Missing RDEPEND on sys-libs/libcap


Upon further investigation, it looks like fcaps.eclass only sets libcap as a DEPEND, but in this case it gets linked in and is needed at runtime.  Is this an unusual case for libcap to actually be linked in, or should the eclass be setting RDEPEND as well?  I don't know anything about filecaps or libcap so I'm assigning this to the person who has made all the recent commits, and cc:ing the retired developer who wrote the eclass originally.  Please correct my mistakes at will :-)
Comment 1 Jeroen Roovers (RETIRED) gentoo-dev 2015-02-18 06:39:11 UTC
fcaps.eclass uses the tools from sys-libs/libcap to set capabilities on executables as per the directions in the ebuilds that use the eclass.

The eclass doesn't purport to handle the use of sys-libs/libcap for other purposes, like linking against its libraries. Those ebuilds should set their own dependencies properly.
Comment 2 Rick Farina (Zero_Chaos) gentoo-dev 2015-02-18 15:33:36 UTC
After a slightly longer explaination of how filecaps work (thanks aballier) This is only a missing dep in the cdrtools ebuild.  libcap is needed for fcaps.eclass to set filecaps but is not nessesarily needed by everything which uses filecaps once this initial setup is done.

Adjusted cdrtools ebuild accordingly.  Thanks!
Comment 4 Rick Farina (Zero_Chaos) gentoo-dev 2015-02-18 16:55:28 UTC
Thanks vapier, that will help avoid confusion like mine.