rpm on ubuntu 16.04 fails to honor non-zero exit from preun scriptlet

Bug #1594670 reported by bugproxy
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
rpm (Debian)
Fix Released
Unknown
rpm (Ubuntu)
Fix Released
Undecided
Taco Screen team

Bug Description

---Problem Description---
rpm erase does not honour a non-zero exit code from preuninstall scriptlet.

---uname output---
Linux kanyu 4.4.0-22-generic #40-Ubuntu SMP Thu May 12 22:03:46 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux

Machine Type = VMware

---Debugger---
A debugger is not configured

---Steps to Reproduce---
 Install the IBM MQ rpms
create and start a queue manager
attempt to uninstall ( rpm -qa | grep MQSeries | xargs rpm -e)
preuninstall scriptlets correct exit with status of 1 but rpm continues with erase.

This is occurring on x86_64, ppcle and Z
rpm on Ubuntu 14.4 and earlier behaved as expected, erase aborted on non-zero exit from preun

ats@kanyu:~$ apt list rpm
Listing... Done
rpm/xenial,now 4.12.0.1+dfsg1-3build3 amd64 [installed]
ats@kanyu:~$

Userspace tool common name: rpm

The userspace tool has the following bit modes: 64-bit

Userspace rpm: rpm/xenial,now 4.12.0.1+dfsg1-3build3 amd64

bugproxy (bugproxy)
tags: added: architecture-all bugnameltc-142844 severity-high targetmilestone-inin1604
Changed in ubuntu:
assignee: nobody → Taco Screen team (taco-screen-team)
Revision history for this message
Ubuntu Foundations Team Bug Bot (crichton) wrote :

Thank you for taking the time to report this bug and helping to make Ubuntu better. It seems that your bug report is not filed about a specific source package though, rather it is just filed against Ubuntu in general. It is important that bug reports be filed about source packages so that people interested in the package can find the bugs about it. You can find some hints about determining what package your bug might be about at https://wiki.ubuntu.com/Bugs/FindRightPackage. You might also ask for help in the #ubuntu-bugs irc channel on Freenode.

To change the source package that this bug is filed about visit https://bugs.launchpad.net/ubuntu/+bug/1594670/+editstatus and add the package name in the text box next to the word Package.

[This is an automated message. I apologize if it reached you inappropriately; please just reply to this message indicating so.]

tags: added: bot-comment
affects: ubuntu → apt-rpm (Ubuntu)
Steve Langasek (vorlon)
affects: apt-rpm (Ubuntu) → rpm (Ubuntu)
Changed in rpm (Debian):
status: Unknown → Confirmed
Revision history for this message
Frédéric Bonnard (frediz) wrote :

I sent a patch here : https://github.com/rpm-software-management/rpm/pull/74 .
Review seems positive, but we still need to wait for the merged.
I also opend the bug in Debian and sent the debdiff : https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=830194 .
Would it be possible to have the patch pushed in the packaging for 16.04 ?

Revision history for this message
Frédéric Bonnard (frediz) wrote :

Here is the debdiff that applies on top of rpm-4.12.0.1+dfsg1 .

Revision history for this message
Steve Langasek (vorlon) wrote :

Installing rpm packages on an Ubuntu system is not supported, at all; it's expected that one use 'alien' to convert packages from rpm to deb format before attempting to install them.

This bug therefore does not rise to the level of an SRU for 16.04.

Revision history for this message
Ubuntu Foundations Team Bug Bot (crichton) wrote :

The attachment "rpm-fix-preun.debdiff" seems to be a patch. If it isn't, please remove the "patch" flag from the attachment, remove the "patch" tag, and if you are a member of the ~ubuntu-reviewers, unsubscribe the team.

[This is an automated message performed by a Launchpad user owned by ~brian-murray, for any issues please contact him.]

tags: added: patch
Revision history for this message
bugproxy (bugproxy) wrote : Comment bridged from LTC Bugzilla

------- Comment From <email address hidden> 2016-08-01 05:15 EDT-------
Patch has been merged upstream. I asked if Debian Maintainer can embed the patch in the packaging. Let's see?

Changed in rpm (Debian):
status: Confirmed → Fix Released
Revision history for this message
Frédéric Bonnard (frediz) wrote :

My patch has landed in Debian, as rpm 4.12.0.1+dfsg1-6 and it got synced in Ubuntu Yakkety.

Steve Langasek (vorlon)
Changed in rpm (Ubuntu):
status: New → Fix Released
bugproxy (bugproxy)
tags: added: targetmilestone-inin1610
removed: targetmilestone-inin1604
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.