inkscape should return non-0 on 'errors'

Bug #993500 reported by DimStar
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Inkscape
Fix Released
Medium
Kris
inkscape (openSUSE)
Fix Released
Wishlist

Bug Description

$ inkscape -z -f nonexistant_file

prints several error messagages, but its exit-value is 0, regardless if the
file after -f exists or not.

Tags: cli
Revision history for this message
In , J6 (j6) wrote :

$ inkscape -z -f nonexistant_file

prints several error messagages, but its exit-value is 0, regardless if the file after -f exists or not.

Revision history for this message
In , J6 (j6) wrote :

Created an attachment (id=487198)
inkscape_error_return_nonzero.patch

Please review the attached patch. It appears to work for me.

Revision history for this message
DimStar (dominique-launchpad-net) wrote :
Revision history for this message
In , Dominique Leuenberger aka DimStar (dimstar) wrote :

Filed the bug upstream; let's see what initial reactions are

Kris (kris-degussem)
Changed in inkscape:
status: New → Confirmed
importance: Undecided → Medium
milestone: none → 0.49
status: Confirmed → In Progress
Kris (kris-degussem)
Changed in inkscape:
status: In Progress → Triaged
Changed in inkscape (openSUSE):
importance: Unknown → Wishlist
status: Unknown → Confirmed
Revision history for this message
Kris (kris-degussem) wrote :

Can not inspect the influence on windows inkscape.exe always quits with non-zero value and inscape.com always exits with errorlevel 0.

Could not apply patch. Hence, slithly modified patch in attachment.

su_v (suv-lp)
tags: added: cli crash
removed: feature
su_v (suv-lp)
tags: removed: crash
Revision history for this message
Kris (kris-degussem) wrote :

Because there were no further reactions, committed patch in revision 11357.

Changed in inkscape:
assignee: nobody → Kris (kris-degussem)
status: Triaged → Fix Committed
Revision history for this message
In , Vincent Untz (vuntz) wrote :

*** Bug 743745 has been marked as a duplicate of this bug. ***

Revision history for this message
In , Vincent Untz (vuntz) wrote :

Patch is in G:A, will be pushed to Factory soon. Thanks to Dominique for doing all the work (including the sr :-))

Changed in inkscape (openSUSE):
status: Confirmed → Fix Released
Bryce Harrington (bryce)
Changed in inkscape:
status: Fix Committed → Fix Released
Revision history for this message
In , Swamp-a (swamp-a) wrote :

This is an autogenerated message for OBS integration:
This bug (758275) was mentioned in
https://build.opensuse.org/request/show/695939 15.1 / inkscape

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.