Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 784761 - app-metrics/alertmanager and app-metrics/blackbox_exporter: user/groups wrongly depcleaned (DEPEND defined twice)
Summary: app-metrics/alertmanager and app-metrics/blackbox_exporter: user/groups wrong...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: William Hubbs
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-04-21 07:38 UTC by Holger Hoffstätte
Modified: 2021-04-21 08:51 UTC (History)
3 users (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 Holger Hoffstätte 2021-04-21 07:38:53 UTC
Commits c9eafaaba5772ce80c13483819db017c9cb4b718 and 	332e3526ec6cd0b27314a9616c6a603de82cf81a changed the respective user/group dependencies from BDEPEND to DEPEND. On an existing installations which pulled these user/group packages in as dependency of the main package, the user/groups will now be depcleaned. New installations will no longer pull them in to begin with.

Reproducible: Always

Steps to Reproduce:
1. newly install alertmanager or blackbox_exporter
2. missing user/group

Actual Results:  
Missing user/group package on new install or depcleaned from existing installation.
Comment 1 Ionen Wolkens gentoo-dev 2021-04-21 07:57:17 UTC
These shouldn't be in BDEPEND (RDEPEND+DEPEND or RDEPEND-only to, in theory, not use the build machine's user/groups), but:

    DEPEND="
        acct-group/blackbox_exporter
        acct-user/blackbox_exporter
    "
    DEPEND="dev-util/promu"

It's defining DEPEND twice and the second one overrides it losing the dependency entirely. Also, I'm pretty sure this should be RDEPEND+DEPEND here, I see user/group both in the ebuild and init scripts.

CC'ing commit author

(normally prefer 1 bug per package but the issue is identical).
Comment 2 Mikle Kolyada (RETIRED) archtester Gentoo Infrastructure gentoo-dev Security 2021-04-21 08:09:41 UTC
thanks, fixed.
Comment 3 Sam James archtester Gentoo Infrastructure gentoo-dev Security 2021-04-21 08:51:49 UTC
Don’t we need RDEPEND too?