From: "Gavini, Sumanth" <sumanth.gavini@windriver.com>
To: Ross Burton <Ross.Burton@arm.com>
Cc: "openembedded-core@lists.openembedded.org"
<openembedded-core@lists.openembedded.org>,
"alex.kanavin@gmail.com" <alex.kanavin@gmail.com>,
"mathieu.dubois-briand@bootlin.com"
<mathieu.dubois-briand@bootlin.com>,
"mattware@cisco.com" <mattware@cisco.com>,
"esparlin@cisco.com" <esparlin@cisco.com>,
"peter.marko@siemens.com" <peter.marko@siemens.com>,
"MacLeod, Randy" <Randy.MacLeod@windriver.com>
Subject: Re: [OE-core] [PATCH v13 08/10] rpm: filter /usr/bin/pkg-config from RPM auto-generated Requires
Date: Wed, 12 Aug 2026 00:54:52 +0000 [thread overview]
Message-ID: <d9011376-e716-4d8f-be21-cb8cbf3a214c@windriver.com> (raw)
In-Reply-To: <1A55FDD7-E4F7-4423-BA3B-7A91F3586AB2@arm.com>
On 8/10/2026 4:43 AM, Ross Burton wrote:
> CAUTION: This email comes from a non Wind River email account!
> Do not click links or open attachments unless you recognize the sender and know the content is safe.
>
> On 10 Aug 2026, at 05:52, Sumanth Gavini via lists.openembedded.org <sumanth.gavini=windriver.com@lists.openembedded.org> wrote:
>>
>> * Added 0001-pkgconfigdeps.sh-Do-not-emit-usr-bin-pkg-config-as-a.patch
>> - pkgconfigdeps.sh unconditionally emits /usr/bin/pkg-config as a hard
>> Requires for packages shipping .pc files. In OE/Yocto cross-compilation,
>> pkg-config is a host-only build tool not needed on the target rootfs,
>> causing do_rootfs and do_populate_sdk failures with -dev packages.
>
> But that’s not entirely true - a package which ships a .pc file arguably needs pkgconfig of some sort for that to be useful, and it’s absolutely possible to do development on the target.
>
> You can argue whether this dependency is actually useful - and I don’t believe it is, the user of the .pc file should be the origin of the pkgconf dependency, but we can fix this and keep the existing behaviour by adding another RPROVIDES to pkgconf.
>
> (note that my WIP branch to remove rpmdeps entirely doesn’t generate these dependencies at all, as I really do believe they’re pointless)
>
> Ross
Hi Ross,
Thanks for the review.
You're right, the user of the .pc file should be the origin of the
pkgconf dependency, and removing the emission entirely is heavy-handed.
I've dropped patch 08 and instead added /usr/bin/pkg-config alongside
the existing pkgconfig in RPROVIDES:
RPROVIDES:${PN} += "pkgconfig /usr/bin/pkg-config"
This satisfies the auto-generated Requires from pkgconfigdeps.sh
without modifying the upstream rpm script.
I also had to remove /usr/bin/pkg-config from nativesdk-sdk-provides-dummy
because RPM 6's internal dependency generator auto-detects file-level
Provides from installed files nativesdk-pkgconf already provides
/usr/bin/pkg-config via the wrapper script it installs, so the dummy
entry caused a conflict.
Will send v14 with this change
--
# Sumanth Gavini
# Wind River Linux
next prev parent reply other threads:[~2026-08-12 0:55 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-08-10 4:52 [PATCH v13 00/10] rpm: 4.20.1 -> 6.0.2 Sumanth Gavini
2026-08-10 4:52 ` [PATCH v13 01/10] package_rpm.bbclass: Drop external dependency generator to support rpm 6 Sumanth Gavini
2026-08-10 4:52 ` [PATCH v13 02/10] package_rpm.bbclass: Define _lib and _libdir for rpmbuild Sumanth Gavini
2026-08-10 4:52 ` [PATCH v13 03/10] lib/oe/package.py: Don't add ldconfig_postinst_fragment for glibc or musl Sumanth Gavini
2026-08-10 4:52 ` [PATCH v13 04/10] lib/oe/package.py: Don't redirect stderr Sumanth Gavini
2026-08-10 4:52 ` [PATCH v13 05/10] target-sdk-provides-dummy: Add pkg-config to DUMMYPROVIDES Sumanth Gavini
2026-08-10 4:52 ` [PATCH v13 06/10] rpm: 4.20.1 -> 6.0.2 Sumanth Gavini
2026-08-10 4:52 ` [PATCH v13 07/10] libarchive: Make it work with rpm 6.0.2 Sumanth Gavini
2026-08-10 4:52 ` [PATCH v13 08/10] rpm: filter /usr/bin/pkg-config from RPM auto-generated Requires Sumanth Gavini
2026-08-10 9:43 ` [OE-core] " Ross Burton
2026-08-12 0:54 ` Gavini, Sumanth [this message]
2026-08-10 4:52 ` [PATCH v13 09/10] rpm: Fix pkgconfig() Provides missing in cross-built RPMs Sumanth Gavini
2026-08-11 14:19 ` Mathieu Dubois-Briand
2026-08-10 4:52 ` [PATCH v13 10/10] libarchive: Add RPM format reader to support rpm 6 Sumanth Gavini
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=d9011376-e716-4d8f-be21-cb8cbf3a214c@windriver.com \
--to=sumanth.gavini@windriver.com \
--cc=Randy.MacLeod@windriver.com \
--cc=Ross.Burton@arm.com \
--cc=alex.kanavin@gmail.com \
--cc=esparlin@cisco.com \
--cc=mathieu.dubois-briand@bootlin.com \
--cc=mattware@cisco.com \
--cc=openembedded-core@lists.openembedded.org \
--cc=peter.marko@siemens.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox