Ted,
I had to restore the code you deleted in file.cpp:
const gchar *ext = sp_repr_attr(repr, "inkscape:output_extension"); if (ext == NULL) { // plain svg files do not know no inkscape:output_extensions; // old Inkscape or Sodipodi documents also don't have it // Save as Inkscape SVG in that case ext = SP_MODULE_KEY_OUTPUT_SVG_INKSCAPE; } success = file_save (doc, fn, Inkscape::Extension::db.get(ext));
because without it, it saved old inkscape documents (without inkscape:output_extension) as plain svg, thus effectively breaking them. If you want to do this differently, you can, but please test thoroughly, and maybe it's best not to change this any more until the release. The current CVS is tested and works, let's not break it again :)
_________________________________________________________________ MSN Premium helps eliminate e-mail viruses. Get 2 months FREE* http://join.msn.com/?pgmarket=en-ca&page=byoa/prem&xAPID=1994&DI...
On Sat, 2004-04-03 at 00:59, bulia byak wrote:
I had to restore the code you deleted in file.cpp:
<snip>
because without it, it saved old inkscape documents (without inkscape:output_extension) as plain svg, thus effectively breaking them. If you want to do this differently, you can, but please test thoroughly, and maybe it's best not to change this any more until the release. The current CVS is tested and works, let's not break it again :)
Hmmm, I tried it again, and it still seems to work. What file are you trying? Does the 'inkscape:output_extension' field show up in the XML editor for that file?
The problem with that code is that it will break any chance of autodetect working on files. If we are going to leave that code in, I think we should remove autodetect from the menu. I'd rather get all of this working.
--Ted
participants (2)
-
bulia byak -
Ted Gould