From: Richard Purdie <richard.purdie@linuxfoundation.org>
To: sumanth.gavini@windriver.com, openembedded-core@lists.openembedded.org
Cc: alex.kanavin@gmail.com, mathieu.dubois-briand@bootlin.com,
mattware@cisco.com, esparlin@cisco.com, peter.marko@siemens.com,
randy.macleod@windriver.com
Subject: Re: [OE-core] [PATCH v15 01/12] package_rpm.bbclass: Drop external dependency generator to support rpm 6
Date: Sat, 22 Aug 2026 10:05:04 +0100 [thread overview]
Message-ID: <fe869a36b0dc0ce880f72759c31edcb55a536faa.camel@linuxfoundation.org> (raw)
In-Reply-To: <20260816231104.3871015-2-sumanth.gavini@windriver.com>
On Sun, 2026-08-16 at 16:10 -0700, Sumanth Gavini via lists.openembedded.org wrote:
> The rpm 6 doesn't support external dependency generator. Oe-core used it for
> per file dependency which was saved into pn.requires and pn.provides,but that
> hasn't been used any more since 2012 (rev:
> be40f6d0bb80274366af00461112af65687a4de8), and there were no complains or
> updates in the past 13 years, so just drop it to support rpm 6.
>
> Signed-off-by: Robert Yang <liezhi.yang@windriver.com>
> Signed-off-by: Sumanth Gavini <sumanth.gavini@windriver.com>
> ---
> meta/classes-global/package_rpm.bbclass | 84 +------------------------
> 1 file changed, 3 insertions(+), 81 deletions(-)
I think this patch is the right thing to do but it should be split into
two changes. One is the removal of support for "per file" dependencies
in the rpm backend, i.e. MERGEPERFILEDEPS=0. We can say that hasn't
been enabled/used in a long time and has performance issues. That
change is most of this commit and is only tangentially related to
disabling the external dependency generator.
The second patch is the single change to remove external dependency
generator support. There are probably other pieces which go with that
one.
Cheers,
Richard
next prev parent reply other threads:[~2026-08-22 9:05 UTC|newest]
Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-08-16 23:10 [PATCH v15 00/12] rpm: 4.20.1 -> 6.0.2 Sumanth Gavini
2026-08-16 23:10 ` [PATCH v15 01/12] package_rpm.bbclass: Drop external dependency generator to support rpm 6 Sumanth Gavini
2026-08-17 8:55 ` Mathieu Dubois-Briand
2026-08-17 13:34 ` Gavini, Sumanth
2026-08-17 14:48 ` Mathieu Dubois-Briand
2026-08-17 20:15 ` Alexander Kanavin
2026-08-19 7:10 ` Mathieu Dubois-Briand
2026-08-22 9:05 ` Richard Purdie [this message]
2026-08-16 23:10 ` [PATCH v15 02/12] package_rpm.bbclass: Define _lib and _libdir for rpmbuild Sumanth Gavini
2026-08-16 23:10 ` [PATCH v15 03/12] lib/oe/package.py: Don't add ldconfig_postinst_fragment for glibc or musl Sumanth Gavini
2026-08-16 23:10 ` [PATCH v15 04/12] lib/oe/package.py: Don't redirect stderr Sumanth Gavini
2026-08-16 23:10 ` [PATCH v15 05/12] target-sdk-provides-dummy: Add pkg-config to DUMMYPROVIDES Sumanth Gavini
2026-08-16 23:10 ` [PATCH v15 06/12] rpm: 4.20.1 -> 6.0.2 Sumanth Gavini
2026-08-16 23:10 ` [PATCH v15 07/12] libarchive: Make it work with rpm 6.0.2 Sumanth Gavini
2026-08-16 23:11 ` [PATCH v15 08/12] rpm: Fix pkgconfig() Provides missing in cross-built RPMs Sumanth Gavini
2026-08-16 23:11 ` [PATCH v15 09/12] libarchive: Add RPM format reader to support rpm 6 Sumanth Gavini
2026-08-16 23:11 ` [PATCH v15 10/12] libarchive: Disable RPM filter auto-bidding and update huge_rpm test Sumanth Gavini
2026-08-23 8:06 ` [OE-core] " Richard Purdie
2026-08-16 23:11 ` [PATCH v15 11/12] package_rpm.bbclass: Suppress pkgconfig() Requires for sstate compatibility Sumanth Gavini
2026-08-16 23:11 ` [PATCH v15 12/12] lib/oe/package_manager/rpm: Enable DNF filelists to resolve file deps Sumanth Gavini
2026-08-17 22:20 ` [OE-core] [PATCH v15 00/12] rpm: 4.20.1 -> 6.0.2 Richard Purdie
2026-08-18 1:13 ` Ryan Eatmon
2026-08-18 16:09 ` Gavini, Sumanth
2026-08-18 21:05 ` Richard Purdie
2026-08-21 23:49 ` Gavini, Sumanth
2026-08-24 18:01 ` Richard Purdie
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=fe869a36b0dc0ce880f72759c31edcb55a536faa.camel@linuxfoundation.org \
--to=richard.purdie@linuxfoundation.org \
--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 \
--cc=randy.macleod@windriver.com \
--cc=sumanth.gavini@windriver.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