On Sat, Aug 08, 2020 at 07:04:03AM +0200, Jan Kratochvil wrote: > On Fri, 07 Aug 2020 13:55:35 +0200, Mark Wielaard wrote: > > But I pulled > > that branch and reviewed the actual commits (1d080e02 and c804a960). > > git clone -b types g...@github.com:jankratochvil/rpm.git rpm-types > > commit 8b5bbcc6d586be50b6a251256c39c3b0332b1f2b > debugedit: Fix missing relocation of .debug_types section. > commit 1d080e02409d181169d3aec2a19192418f253fd3 > [NFC] debugedit: Move code from edit_dwarf2() to edit_info(). > > > > It would be better to have a separate RPM_DEBUGEDIT_TYPES_SETUP that > > does that and leave RPM_DEBUGEDIT_SETUP as is. Otherwise we change the > > existing tests too. > > done as: RPM_DEBUGEDIT_SETUP([-fdebug-types-section])
Nice. Using $1 is better than what I suggested. Commits look good. Should go into the main branch IMHO. Thanks, Mark _______________________________________________ Rpm-maint mailing list Rpm-maint@lists.rpm.org http://lists.rpm.org/mailman/listinfo/rpm-maint