Bug 16950

Summary: rpm systemd-inhibit plugin not built due to missing BuildRequire (Patch attached)
Product: Mageia Reporter: Neal Gompa <ngompa13>
Component: RPM PackagesAssignee: Thierry Vignaud <thierry.vignaud>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: Normal    
Version: Cauldron   
Target Milestone: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Source RPM: rpm-4.13.0-0.rc1.11.mga6.src.rpm CVE:
Status comment:
Attachments: Patch adding dbus-devel BuildRequire for building systemd inhibit plugin

Description Neal Gompa 2015-10-13 06:11:20 CEST
Description of problem:
The systemd-inhibit plugin for RPM (which prevents system shutdown during RPM transactions) was not previously built, due to a missing built-time requirement for dbus-devel. The attached patch corrects this issue.

Without this plugin, it's possible for a service or a user to shutdown/reboot the system during an RPM transaction.

Version-Release number of selected component (if applicable):
4.13.0-0.rc1.11.mga6

Reproducible: 

Steps to Reproduce:
Comment 1 Neal Gompa 2015-10-13 06:12:17 CEST
Created attachment 7119 [details]
Patch adding dbus-devel BuildRequire for building systemd inhibit plugin

Patch attached.
Comment 2 Thierry Vignaud 2015-10-13 11:19:57 CEST
That's not critical: it has always been like this.
Of course, it's better to enable this new feature, so I'll do.
Thanks for the report

Status: NEW => ASSIGNED
Severity: critical => normal

Comment 3 Neal Gompa 2015-10-13 16:08:10 CEST
No problem, I thought it was supposed to be built and it was somehow forgotten, that's why I thought it was critical.
Comment 4 Neal Gompa 2015-10-13 17:58:00 CEST
I only found out because DNF listed it as a requirement, and I saw that the rpm package didn't include the plugin.
Comment 5 Thierry Vignaud 2015-10-13 21:43:27 CEST
Fixed in SVN.
I won't rebuild rpm just for that

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

Comment 6 Neal Gompa 2015-10-13 21:45:41 CEST
I don't expect you to, but having the fix in place will mean that when rpm 4.13.0 final comes out, the plugin will come with it.