While the code looks ok, I wonder if there is a better way to do this? We 
should be able to add the debuginfo generation into `%__spec_install_pre`. This 
was not possible in the past as you could not expand a package definition into 
the `%install` section. But now we can just use the same route as here and just 
drop it into a dynamic `.specpart` file. We'd basically need the same code as 
`doDebugPackage` but as a macro - or probably better - luascript. But we could 
leave `%install` as a normal build script without any special treatment in the 
rpm C code.

As this is about getting things to work for 4.20 I am fine with just keeping 
this patch as is and have a second run at this later on during 6.0 development.

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

Message ID: <rpm-software-management/rpm/pull/3036/c2056899...@github.com>
_______________________________________________
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint

Reply via email to