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

Bug #1572242 reported by Shlomi Fish on 2016-04-19
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Inkscape
Wishlist
Shlomi Fish
Inkscape-docs
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.

jazzynico (jazzynico) on 2016-06-04
tags: added: cli exporting svg
Changed in inkscape:
importance: Undecided → Wishlist
status: New → Triaged
milestone: none → 0.93
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) on 2017-04-18
Changed in inkscape:
assignee: nobody → Shlomi Fish (shlomif-gmail)
status: Triaged → In Progress
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.

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
jazzynico (jazzynico) wrote :

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

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.

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.

jazzynico (jazzynico) wrote :

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

Changed in inkscape:
status: In Progress → Fix Committed
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?

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.

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.

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.

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
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers