[Bug 1013275] New: rpmbuild does not stop on %error
http://bugzilla.suse.com/show_bug.cgi?id=1013275 Bug ID: 1013275 Summary: rpmbuild does not stop on %error Classification: openSUSE Product: openSUSE Distribution Version: Leap 42.3 Hardware: Other OS: Other Status: NEW Severity: Normal Priority: P5 - None Component: Development Assignee: bnc-team-screening@forge.provo.novell.com Reporter: jmatejek@suse.com QA Contact: qa-bugs@suse.de Found By: --- Blocker: --- When building a package that contains the %error macro, rpmbuild will print the error but continue the build. This makes it impossible to deliberately fail build, unless this is in %prep/%build/%install, where one can put "exit 1" after the %error declaration. The docs say: %{error:...} print ... to stderr and return BADSPEC i don't know exactly what "return BADSPEC" means, but it is definitely not doing what I expect to reproduce: take a working spec file and put %{error:hello world} somewhere in it expected result: rpmbuild fails, printing "Error: hello world" actual result: rpmbuild prints "Error: hello world" but the build finishes successfully -- You are receiving this mail because: You are on the CC list for the bug.
http://bugzilla.suse.com/show_bug.cgi?id=1013275 Chenzi Cao <chcao@suse.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|bnc-team-screening@forge.pr |mls@suse.com |ovo.novell.com | -- You are receiving this mail because: You are on the CC list for the bug.
http://bugzilla.suse.com/show_bug.cgi?id=1013275 http://bugzilla.suse.com/show_bug.cgi?id=1013275#c1 Karl Cheng <qantas94heavy@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |qantas94heavy@gmail.com --- Comment #1 from Karl Cheng <qantas94heavy@gmail.com> --- This was only fixed upstream in RPM 4.14: https://github.com/rpm-software-management/rpm/commit/94e8cd6058cba07b5b9778... It is only one line long, though this won't fix it for OBS builds based on openSUSE:Leap:42.3. Should this still be patched? -- You are receiving this mail because: You are on the CC list for the bug.
http://bugzilla.suse.com/show_bug.cgi?id=1013275 http://bugzilla.suse.com/show_bug.cgi?id=1013275#c2 Michael Schröder <mls@suse.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |RESOLVED Resolution|--- |WONTFIX --- Comment #2 from Michael Schröder <mls@suse.com> --- I don't think it's worth to backport this. Leap 15 has rpm-4.14 which contains the commit. -- You are receiving this mail because: You are on the CC list for the bug.
participants (1)
-
bugzilla_noreply@novell.com