[PATCH] Add --export-inkscape-svg=FILENAME command line option to export as inkscape-style SVG

Bug #1572242 reported by Shlomi Fish
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Inkscape
Fix Released
Wishlist
Shlomi Fish
Inkscape-docs
Invalid
Wishlist
jazzynico

Bug Description

This patch adds an --export-inkscape-svg=... cmdline option so one can do:

    inkscape -z --export-inkscape-svg=temp.svg sample.dxf

and get an inkscape-style SVG. It was inspired/motivated by a conversation today on the #inkscape chatroom on Freenode where someone needed it as part of their workflow. Trying --verb FileSave --verb FileQuit did not appear to work properly.

Please consider applying it and let me know if you need anything else.

Tags: cli exporting svg
Revision history for this message
Shlomi Fish (shlomif-gmail) wrote :
jazzynico (jazzynico)
tags: added: cli exporting svg
Changed in inkscape:
importance: Undecided → Wishlist
status: New → Triaged
milestone: none → 0.93
Revision history for this message
Shlomi Fish (shlomif-gmail) wrote :

Hi all!

As inkscape 0.92.x was already released, please apply the patch and close this bug report.

jazzynico (jazzynico)
Changed in inkscape:
assignee: nobody → Shlomi Fish (shlomif-gmail)
status: Triaged → In Progress
Revision history for this message
Shlomi Fish (shlomif-gmail) wrote :

@jazzynico : why did you assign this issue to me? I wrote the patch but I cannot apply it and thus close this bug report. Please assign it to someone else.

Revision history for this message
jazzynico (jazzynico) wrote :

@Shlomi - Don't worry, assigning the report to you is just a way to be sure you will be granted for the fix when applied to the branch. I doesn't mean you need to do something else.

Changed in inkscape-docs:
status: New → Triaged
importance: Undecided → Wishlist
assignee: nobody → jazzynico (jazzynico)
milestone: none → 0.93
Revision history for this message
jazzynico (jazzynico) wrote :

Just a reminder to myself to update the man page for 0.93 when the patch is accepted.

Revision history for this message
jazzynico (jazzynico) wrote :

Patch tested successfully on Xubuntu 16.04, lp:inkscape rev. 15627.

@Shlomi, your patch doesn't allow exporting to inkscape-svg with the export-id and --export-text-to-path options. Is it intentional?
If not, here's a modified patch that should work with the missing options.

Revision history for this message
Shlomi Fish (shlomif-gmail) wrote :

@jazzynico: thanks for the clarification regarding the assigning stuff. Regarding the patch - no that omission was not intentional and I'm perfectly fine with applying the modified patch that remedies it.

Revision history for this message
jazzynico (jazzynico) wrote :

Patch (v2) committed in lp:inkscape rev. 15628.
Thanks Shlomi!

Changed in inkscape:
status: In Progress → Fix Committed
Revision history for this message
Patrick Storz (ede123) wrote :

I was wondering: Shouldn't we call this option simply "--export-svg"?

As "Inkscape SVG" is Inkscape's native file format, it should be fine to assume that it will be used when "exporting" SVG?

Revision history for this message
Hachmann (marenhachmann) wrote :

@Eduard: That would be shorter, yes.
But this way it's clearer, esp. as Inkscape SVGs can also contain incompatible things (mesh gradients, not sure about flowed text status in 0.93)
I'd vote for keeping it like this.

Revision history for this message
Hachmann (marenhachmann) wrote :

But would it make sense to also have a short command? What I mean by this is one of the kind:

-l

-s is still free. -i and -I are taken already.

Revision history for this message
Patrick Storz (ede123) wrote :

> But this way it's clearer

I'd say command line arguments do not need to be overly clear (that's what 'inkscape --help' and 'man inkscape' are for) but short and memorable so they are easy to use but unless someone feels this would be an improvement I don't really care.

Short arguments: Would make sense consistency wise. I'd say it should be chosen wisely, but looking at the mess we already have we can probably pick whatever letter is still available without making it any worse. ;-)
'-s' certainly is not the worst choice , though.

Revision history for this message
Patrick Storz (ede123) wrote :

Just stumbled across this while scouring through open documentation issues.

Just for completeness:
"--export-inkscape-svg" seems to be gone in master again (before ever getting released; RIP)

However, Inkscape exports as Inkscape SVG by default now when using
  inkscape in.svg -o out.svg
or using the "--export-type" parameter.

Changed in inkscape-docs:
status: Triaged → Invalid
Max Gaukler (mgmax)
Changed in inkscape:
status: Fix Committed → Fix Released
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.