Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 922255 - Add pkgconf as a BDEPEND
Summary: Add pkgconf as a BDEPEND
Status: RESOLVED WORKSFORME
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal enhancement (vote)
Assignee: Gentoo Linux bug wranglers
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2024-01-16 23:48 UTC by Raul Rangel
Modified: 2024-01-17 19:32 UTC (History)
2 users (show)

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

Attachments
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 Raul Rangel 2024-01-16 23:48:41 UTC 
I'm curious how people feel about adding pkgconf as  BDEPEND. I'm trying to remove `pkgconf` from my [implicit-system](https://chromium.googlesource.com/chromiumos/overlays/chromiumos-overlay/+/refs/heads/main/virtual/target-sdk-implicit-system/target-sdk-implicit-system-9999.ebuild#71) because not all ebuilds use it. Would Gentoo be receptive of patches that add pkgconf as a BDEPEND?

Reproducible: Always
Comment 1 Eli Schwartz 2024-01-17 00:21:26 UTC 
Sorry, what? Adding pkgconf as a BDEPEND to which package?

In general, any package which makes use of pkg-config and doesn't have a BDEPEND on virtual/pkgconfig is probably broken? Does it build from a minimal system that has been depcleaned --with-bdeps=n and re-emerged?

Keep in mind it's also an RDEPEND for cmake and meson.
Comment 2 Sam James archtester Gentoo Infrastructure gentoo-dev Security 2024-01-17 00:23:10 UTC 
Packages needing pkgconf are likely calling pkg-config instead (or should), and should BDEPEND on virtual/pkgconfig.
Comment 3 Mike Gilbert gentoo-dev 2024-01-17 19:26:20 UTC 
Right, Gentoo ebuilds should already be putting virtual/pkgconfig in BDEPEND when necessary.

Please do file bugs and/or send patches if there are cases where it is missing.
Comment 4 Raul Rangel 2024-01-17 19:32:25 UTC 
Sorry, I should have checked if existing ebuilds had it. Thanks for the confirmation! I'll send some patches.