Bug 22261 - Bugs in gendistrib when hdlists, etc. are not stored in the master media_info directory
Summary: Bugs in gendistrib when hdlists, etc. are not stored in the master media_info...
Status: RESOLVED FIXED
Alias: None
Product: Mageia
Classification: Unclassified
Component: RPM Packages (show other bugs)
Version: Cauldron
Hardware: All Linux
Priority: Normal normal
Target Milestone: ---
Assignee: Mageia tools maintainers
QA Contact:
URL:
Whiteboard:
Keywords: PATCH
Depends on:
Blocks:
 
Reported: 2017-12-24 00:20 CET by Martin Whitaker
Modified: 2018-03-11 15:50 CET (History)
1 user (show)

See Also:
Source RPM: rpmtools-7.01-1.mga7.src.rpm
CVE:
Status comment:

Attachments
Fix for first bug (936 bytes, text/plain)
2017-12-24 00:20 CET, Martin Whitaker 		Details
Fix for second bug (1.35 KB, text/plain)
2017-12-24 00:20 CET, Martin Whitaker 		Details
View All Add an attachment (proposed patch, testcase, etc.)

Description Martin Whitaker 2017-12-24 00:20:18 CET 
Created attachment 9849 [details]
Fix for first bug

In the media.cfg file you can specify a relative path in hdlist= and thus link directly to the master hdlist file in an individual repository. But if you do this, gendistrib deletes the master file and replaces it with a soft link which links back to itself. The same holds for synthesis files.

gendistrib also assumes that the hdlist and synthesis files are stored in (or linked via) the master media_info directory and tries to calculate md5sums for them, leading to unwanted error messages when they aren't there.

The attached patches fix these bugs.
Comment 1 Martin Whitaker 2017-12-24 00:20:50 CET 
Created attachment 9850 [details]
Fix for second bug
Comment 2 Martin Whitaker 2018-03-11 14:51:24 CET 
As there's no registered maintainer for the rpmtools package, I've pushed these fixes myself.

Resolution: (none) => FIXED
Status: NEW => RESOLVED

Comment 3 Thomas Backlund 2018-03-11 14:55:22 CET 
since we want them on infra, maybe we should push it as an official update for mga6 too...

CC: (none) => tmb

Comment 4 Martin Whitaker 2018-03-11 15:50:44 CET 
(In reply to Thomas Backlund from comment #3)
> since we want them on infra, maybe we should push it as an official update
> for mga6 too...

My only problem with that would be coming up with a test procedure for QA...
Note You need to log in before you can comment on or make changes to this bug.