Opened 4 years ago

Closed 4 years ago

Last modified 4 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 4 years ago.
macros.diff (694 bytes) - added by dmik 4 years ago.

Download all attachments as: .zip

Change History (9)

comment:1 Changed 4 years ago by dmik

  • Component changed from python to rpm

comment:2 Changed 4 years ago by dmik

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 Changed 4 years ago by dmik

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.

Changed 4 years ago by dmik

Changed 4 years ago by dmik

comment:4 Changed 4 years ago by dmik

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 Changed 4 years ago by ydario

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

comment:6 Changed 4 years ago by ydario

  • Resolution set to fixed
  • Status changed from new to closed

comment:7 Changed 4 years ago by dmik

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.