Re: [PATCH v2 2/2] rpm: 4.20.1 -> 6.0.1

[Robert Yang <liezhi.yang@windriver.com>]

On 1/20/26 21:04, Robert Yang wrote:
> Hi Alexender,
> 
> On 1/19/26 20:54, Alexander Kanavin wrote:
>> On Mon, 19 Jan 2026 at 07:09, <liezhi.yang@windriver.com> wrote:
>>>    - 0001-macros.in-Set-_rpmformat-to-4.patch
>>>      Fixed:
>>>      error: External dependency generator is incompatible with v6 packages
>>
>> This one is creating technical debt. I can imagine support for v4
>> packages will be dropped at some not-too-distant point. We've had this
>> deprecation warning for a long time, but it looks like the matter is
>> becoming more pressing now. Can you look into it?
> 
> OK, I will look into it.

I checked the history, the external dependency is for rpm per file dependency,
but that function hasn't been used any more since 2012:

===========
commit be40f6d0bb80274366af00461112af65687a4de8
Author: Richard Purdie <richard.purdie@linuxfoundation.org>
Date:   Thu Feb 9 09:51:10 2012 +0000

     package_rpm.bbclass: Optimise per file dependency handling

===========

And back to 2010:
===========
commit 84f7f70308eed0ac96abeb5a762e9b7765e5db91
Author: Mark Hatle <mark.hatle@windriver.com>
Date:   Mon Aug 30 16:44:31 2010 -0500

     package_rpm: Enable per file dependencies
===========

The per file dependency was for creating the rpm, but Richard disabled that
in 2012, and there were no complains or updates in the past 13 years.

I think we can remove that code safely, then we can move to rpm 6.

I will send a V3 for it if no objections.

> 
>>
>>>    - 0001-macros.in-Set-_pkgverify_level-to-digest-as-RCPM-4.2.patch
>>>      Fixed:
>>>      GPG check FAILED
>>>
>>> * Add scdoc-native to DEPENDS to fix do_configure error for both native and
>>>    target:
>>>    Could not find SCDOC using the following names: scdoc
>>
>> Is this a hard dependency? Is there a cmake switch to turn it off?
> 
> I will check and fix that.

Yes, it's a hard dependency since the man pages in *man/* are scdoc sources.

// Robert

> 
>>
>>>    Note, both target and native need scdoc-native.
>>>
>>> * Add bash to RDEPENDS:${PN} to fix:
>>>    ERROR: rpm-1_6.0.1-r0 do_package_qa: QA Issue: /usr/lib/rpm/rpm-setup- 
>>> autosign
>>>    contained in package rpm requires /bin/bash, but no providers found in
>>>    RDEPENDS:rpm? [file-rdeps]
>>
>> All things that depend on bash are specifically placed into
>> ${PN}-build, so that the main package will not pull in gplv3 licensed
>> items. I can imagine this will fail in no-gpl3 tests too.
> 
> I will move it into rpm-build.
> 
>>
>>> -EXTRA_OECMAKE:append:class-native = " -DCMAKE_INSTALL_SYSCONFDIR:PATH=/etc - 
>>> DCMAKE_INSTALL_LOCALSTATEDIR:PATH=/var"
>>> +EXTRA_OECMAKE:append:class-native = " -DCMAKE_INSTALL_SYSCONFDIR:PATH=/etc - 
>>> DCMAKE_INSTALL_LOCALSTATEDIR:PATH=/var -DLIBELF_FOUND=False"
>>
>> Addition of LIBELF_FOUND=False is not explained. Also I think it can
>> be easily provided?
> 
> Good catch, we don't need this change, it has been fixed via 0001-tools- 
> elfdeps.cc-Remove-format-module.patch
> 
> // Robert
> 
>>
>> Otherwise, looks good, thanks for working on it!
>>
>> Alex
>