Opened 9 years ago

Closed 9 years ago

Last modified 9 years ago

#149 closed defect (fixed)

Standardize debug package creation, part 2

Reported by: dmik Owned by:
Priority: major Milestone:
Component: rpm Version:
Severity: medium Keywords:
Cc:

Description

There was #134 which dealt with this issue already but it also touched some other topics which were resolved and the ticket twas closed. However, the main task is not complete, hence the new ticket. It's target is to implement the %debug_package macro so that it automatically creates debug packages when invoked, according to the debug package creation rules that we establish.

Attachments (2)

brp-strip.os2.diff (5.9 KB ) - added by dmik 9 years ago.
macros.diff (694 bytes ) - added by dmik 9 years ago.

Download all attachments as: .zip

Change History (9)

comment:1 by dmik, 9 years ago

Component: pythonrpm

comment:2 by dmik, 9 years ago

I think that we should go the "separate debug package for each sub-package" way by allowing %debug_package take the -n argument to specify the sub-package name (as many other commands already do).

comment:3 by dmik, 9 years ago

I implemented %debug_package in its current form (one -debug per the whole .spec file, just like on other platforms). The diffs are attached. Note that this means the following:

  • If %debug_package is present in the spec file, debug files are automatically generated and placed to a -debug sub package.
  • If %debug_package is absent, no debug files are generated at all.
  • Compression (by lxlite) is controlled separately, via the %_strip_no_compress macro (if it's set to 1, no compression is done, otherwise lxlite is called)
  • It's now possible to tune which files are to be compressed and which are to have debug info extracted, via the %_strip_opts macro (options are to be described in the Wiki).

Yuri, please review and apply it to RPM ASAP. Two packages already use %debug_package: fontconfig and fontconfig-legacy. Note that once the fixes are applied, the manual debug package specification must be replaced with %debug_package in each .spec that uses it, the manual specification will no longer work (because debug file generation is now implicitly controlled by %debug_package).

Please don't close this ticket after applying the patches, as it's still to be decided if we should support multiple debug sub-packages or not.

by dmik, 9 years ago

Attachment: brp-strip.os2.diff added

by dmik, 9 years ago

Attachment: macros.diff added

comment:4 by dmik, 9 years ago

Please note that macros.diff must be applied to each macros file in /usr/lib/platform/*-os2-emx/ (but I guess you know that).

comment:5 by Yuri Dario, 9 years ago

rpm: standardize debug package creation, ticket#149
Committed revision r615.

comment:6 by Yuri Dario, 9 years ago

Resolution: fixed
Status: newclosed

comment:7 by dmik, 9 years ago

Note that I added the following sections to the HowTo describing the current state of affairs:

I also created #159 for the remaining -dbgsym question.

Note: See TracTickets for help on using tickets.