[Rpm-maint] [rpm-software-management/rpm] Upstream debuginfo enablement (PR #3040)

Florian Festi notifications at github.com
Tue Apr 16 09:23:14 UTC 2024


! This removes the %ifnarch noarch check. We need to find a solution for this before merging (or decise it is just an optimization we don't really need)

All these years, enabling debuginfo has required distros to hijack the spec %install section with a macro like this:

    %install %{?_enable_debug_packages:%{?buildsubdir:%{debug_package}}}\
    %%install\
    %{nil}

This for a widely used, longtime upstream supported feature is just gross, and also very non-obvious, feeble and whatnot. And totally prevents the new append/prepend options from being used with %install.

Turn this isto a proper macro that drops the package definition into a .SPECPART file. This way debuginfo can be part of the %install script without messing up the parsing.

Fixes: #2204
Fixes: #1878
You can view, comment on, or merge this pull request online at:

  https://github.com/rpm-software-management/rpm/pull/3040

-- Commit Summary --

  * DONT Upstream debuginfo enablement DONT

-- File Changes --

    M macros.in (6)
    M tests/data/macros.debug (4)

-- Patch Links --

https://github.com/rpm-software-management/rpm/pull/3040.patch
https://github.com/rpm-software-management/rpm/pull/3040.diff

-- 
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/3040
You are receiving this because you are subscribed to this thread.

Message ID: <rpm-software-management/rpm/pull/3040 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rpm.org/pipermail/rpm-maint/attachments/20240416/cbf8c16d/attachment.html>


More information about the Rpm-maint mailing list