Re: [LyX/master] add inkscape wrapper script for Mac OS

2017-10-03 Thread Stephan Witt
Am 01.10.2017 um 09:38 schrieb Jürgen Spitzmüller : > > Am Sonntag, den 24.09.2017, 11:55 +0200 schrieb Stephan Witt: >> commit 350ef993e59b3ede7dd621a81f096ea78296c5c8 >> Author: Stephan Witt >> Date: Sun Sep 24 11:54:55 2017 +0200 >> >>add inkscape wrapper script for Mac OS >> >>The

Re: [LyX/master] add inkscape wrapper script for Mac OS

2017-09-24 Thread Jürgen Spitzmüller
Am Sonntag, den 24.09.2017, 18:57 +0200 schrieb Stephan Witt: > Isn’t this ok? Works for me as well. Jürgen signature.asc Description: This is a digitally signed message part

Re: [LyX/master] add inkscape wrapper script for Mac OS

2017-09-24 Thread Jürgen Spitzmüller
Am Sonntag, den 24.09.2017, 19:03 +0200 schrieb Jürgen Spitzmüller: > You have to pass the inkscape command as first argument. No, actually, you don't. That's optional (I forgot). Jürgen > > Jürgen signature.asc Description: This is a digitally signed message part

Re: [LyX/master] add inkscape wrapper script for Mac OS

2017-09-24 Thread Jürgen Spitzmüller
Am Sonntag, den 24.09.2017, 18:57 +0200 schrieb Stephan Witt: > Sorry, I neither read the script nor the python docs until now. > > I think it’s fine already. The subprocess.check_call gets the > args passed as array of strings and executes the sub process > with individual arguments. The spaces d

Re: [LyX/master] add inkscape wrapper script for Mac OS

2017-09-24 Thread Stephan Witt
> Am 24.09.2017 um 18:57 schrieb Stephan Witt : > > Am 24.09.2017 um 18:32 schrieb Jürgen Spitzmüller : >> >> Am Sonntag, den 24.09.2017, 17:30 +0200 schrieb Stephan Witt: OK, but we might need to re-add quotes in the script (like we had to do for the first argument (inkscape comm

Re: [LyX/master] add inkscape wrapper script for Mac OS

2017-09-24 Thread Stephan Witt
Am 24.09.2017 um 18:32 schrieb Jürgen Spitzmüller : > > Am Sonntag, den 24.09.2017, 17:30 +0200 schrieb Stephan Witt: >>> OK, but we might need to re-add quotes in the script (like we had >>> to do >>> for the first argument (inkscape command), for windows. >> >> Yes, that should be done to be sa

Re: [LyX/master] add inkscape wrapper script for Mac OS

2017-09-24 Thread Kornel Benko
Am Sonntag, 24. September 2017 um 18:41:47, schrieb Stephan Witt > Am 24.09.2017 um 16:55 schrieb Kornel Benko : > > > > Am Sonntag, 24. September 2017 um 16:18:08, schrieb Kornel Benko > > > >> Am Sonntag, 24. September 2017 um 14:50:19, schrieb Stephan Witt > >> > >>> Am 24.09.2017 um 12:3

Re: [LyX/master] add inkscape wrapper script for Mac OS

2017-09-24 Thread Stephan Witt
Am 24.09.2017 um 16:55 schrieb Kornel Benko : > > Am Sonntag, 24. September 2017 um 16:18:08, schrieb Kornel Benko > >> Am Sonntag, 24. September 2017 um 14:50:19, schrieb Stephan Witt >> >>> Am 24.09.2017 um 12:30 schrieb Kornel Benko : Am Sonntag, 24. September 2017 um 11:55:38, s

Re: [LyX/master] add inkscape wrapper script for Mac OS

2017-09-24 Thread Jürgen Spitzmüller
Am Sonntag, den 24.09.2017, 17:30 +0200 schrieb Stephan Witt: > > OK, but we might need to re-add quotes in the script (like we had > > to do > > for the first argument (inkscape command), for windows. > > Yes, that should be done to be safe anyway. > Every shell script needs to be written likewis

Re: [LyX/master] add inkscape wrapper script for Mac OS

2017-09-24 Thread Stephan Witt
Am 24.09.2017 um 16:25 schrieb Jürgen Spitzmüller : > > Am Sonntag, den 24.09.2017, 13:57 +0200 schrieb Stephan Witt: >> I’d say you see the path names in svg2pdftex unquoted because of the >> shell is removing them. > > Makes sense. > >> Nevertheless you get the path names with spaces passed as

Re: [LyX/master] add inkscape wrapper script for Mac OS

2017-09-24 Thread Kornel Benko
Am Sonntag, 24. September 2017 um 16:18:08, schrieb Kornel Benko > Am Sonntag, 24. September 2017 um 14:50:19, schrieb Stephan Witt > > > Am 24.09.2017 um 12:30 schrieb Kornel Benko : > > > > > > Am Sonntag, 24. September 2017 um 11:55:38, schrieb Stephan Witt > > > > > >> commit 350ef993e59

Re: [LyX/master] add inkscape wrapper script for Mac OS

2017-09-24 Thread Jürgen Spitzmüller
Am Sonntag, den 24.09.2017, 13:57 +0200 schrieb Stephan Witt: > I’d say you see the path names in svg2pdftex unquoted because of the > shell is removing them. Makes sense. > Nevertheless you get the path names with spaces passed as one > argument. OK, but we might need to re-add quotes in the sc

Re: [LyX/master] add inkscape wrapper script for Mac OS

2017-09-24 Thread Kornel Benko
Am Sonntag, 24. September 2017 um 14:50:19, schrieb Stephan Witt > Am 24.09.2017 um 12:30 schrieb Kornel Benko : > > > > Am Sonntag, 24. September 2017 um 11:55:38, schrieb Stephan Witt > > > >> commit 350ef993e59b3ede7dd621a81f096ea78296c5c8 > >> Author: Stephan Witt > >> Date: Sun Sep 24

Re: [LyX/master] add inkscape wrapper script for Mac OS

2017-09-24 Thread Stephan Witt
Am 24.09.2017 um 12:30 schrieb Kornel Benko : > > Am Sonntag, 24. September 2017 um 11:55:38, schrieb Stephan Witt > >> commit 350ef993e59b3ede7dd621a81f096ea78296c5c8 >> Author: Stephan Witt >> Date: Sun Sep 24 11:54:55 2017 +0200 >> >>add inkscape wrapper script for Mac OS >> > > Pro

Re: [LyX/master] add inkscape wrapper script for Mac OS

2017-09-24 Thread Stephan Witt
Am 24.09.2017 um 13:08 schrieb Jürgen Spitzmüller : > > Am Sonntag, den 24.09.2017, 12:18 +0200 schrieb Stephan Witt: >> I think they’ll be quoted automatically. > > At least in the svg2pdftex script, they end unquoted. I expect problems > if the path to the tmp dir contains spaces. See method C

Re: [LyX/master] add inkscape wrapper script for Mac OS

2017-09-24 Thread Jürgen Spitzmüller
Am Sonntag, den 24.09.2017, 12:18 +0200 schrieb Stephan Witt: > I think they’ll be quoted automatically. At least in the svg2pdftex script, they end unquoted. I expect problems if the path to the tmp dir contains spaces. Jürgen signature.asc Description: This is a digitally signed message part

Re: [LyX/master] add inkscape wrapper script for Mac OS

2017-09-24 Thread Kornel Benko
Am Sonntag, 24. September 2017 um 11:55:38, schrieb Stephan Witt > commit 350ef993e59b3ede7dd621a81f096ea78296c5c8 > Author: Stephan Witt > Date: Sun Sep 24 11:54:55 2017 +0200 > > add inkscape wrapper script for Mac OS > Probably it doesn't matter, (because nobody cares for cmake on MacOS

Re: [LyX/master] add inkscape wrapper script for Mac OS

2017-09-24 Thread Stephan Witt
Am 24.09.2017 um 12:03 schrieb Jürgen Spitzmüller : > > Am Sonntag, den 24.09.2017, 11:55 +0200 schrieb Stephan Witt: >> -checkProg('an EMF -> EPS converter', ['metafile2eps $$i $$o', >> 'wmf2eps -o $$o $$i', inkscape_name + ' --file=$$i --export-area- >> drawing --without-gui --export-eps=$$o

Re: [LyX/master] add inkscape wrapper script for Mac OS

2017-09-24 Thread Jürgen Spitzmüller
Am Sonntag, den 24.09.2017, 11:55 +0200 schrieb Stephan Witt: > -checkProg('an EMF -> EPS converter', ['metafile2eps $$i $$o', > 'wmf2eps -o $$o $$i', inkscape_name + ' --file=$$i --export-area- > drawing --without-gui --export-eps=$$o'], > +checkProg('an EMF -> EPS converter', ['metafile2e