[Rpm-maint] [rpm-software-management/rpm] Add proper program logic for debuginfo enablement (PR #3036)

[Panu Matilainen]

Oh it's certainly not the best way imaginable, it'd be nice to move this all to a script or something, but ... its complicated. The decision to create debuginfo packages must happen before %install in the spec is parsed, because *extracting* debuginfo is hooked into %__spec_install_post based on whether %__debug_package is defined or not, so it needs to be present when the main spec is parsed. If that gets defined in the .specpart, it happens too late and the extraction never happens. There are four different debuginfo-specific macros involved in the logic...

After banging my head against this all for a few hours yesterday, I concluded that it's best to have the logic in a single spot where it's "easy" to follow, rather than have it scattered around in macros and spec bits, a little C and whatnot. 

I do have this "not *quite* right" feeling about this so lets leave it to dry out here for a few days and see if the vision clears up :sweat_smile: 

It's a headache for sure. 

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

Message ID: <rpm-software-management/rpm/pull/3036/c2058316068 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rpm.org/pipermail/rpm-maint/attachments/20240415/813f5a05/attachment.html>