Re: LyXRC - %%s/scripts

2016-10-24 Thread Jean-Marc Lasgouttes
Le 24/10/2016 à 16:29, Tommaso Cucinotta a écrit : On 24/10/2016 15:31, Scott Kostyshak wrote: Do not hesitate to commit to master once you have tested it. This is our playground, after all. Only stable is subject to authorization from Richard. +1. If it worked for you and the change makes

Re: LyXRC - %%s/scripts

2016-10-24 Thread Tommaso Cucinotta
On 24/10/2016 15:31, Scott Kostyshak wrote: Do not hesitate to commit to master once you have tested it. This is our playground, after all. Only stable is subject to authorization from Richard. +1. If it worked for you and the change makes logical sense, I would say go ahead for master.

Re: LyXRC - %%s/scripts

2016-10-24 Thread Scott Kostyshak
On Mon, Oct 24, 2016 at 09:42:10AM +0200, Jean-Marc Lasgouttes wrote: > Le 24/10/2016 à 08:27, Tommaso Cucinotta a écrit : > > shall I commit to regular master, or would you like to give it a try first? > > Do not hesitate to commit to master once you have tested it. This is our > playground,

Re: LyXRC - %%s/scripts

2016-10-24 Thread Jean-Marc Lasgouttes
Le 24/10/2016 à 08:27, Tommaso Cucinotta a écrit : On 24/10/2016 03:32, Scott Kostyshak wrote: On Sun, Oct 23, 2016 at 09:47:10PM +0200, Tommaso Cucinotta wrote: For me, this [1] fixes the issue: commit d5495caa in tommaso/master Thanks for coming up with a fix shall I commit to regular

Re: LyXRC - %%s/scripts

2016-10-24 Thread Tommaso Cucinotta
On 24/10/2016 03:32, Scott Kostyshak wrote: On Sun, Oct 23, 2016 at 09:47:10PM +0200, Tommaso Cucinotta wrote: For me, this [1] fixes the issue: commit d5495caa in tommaso/master Thanks for coming up with a fix shall I commit to regular master, or would you like to give it a try first?

Re: LyXRC - %%s/scripts

2016-10-23 Thread Scott Kostyshak
On Sun, Oct 23, 2016 at 09:47:10PM +0200, Tommaso Cucinotta wrote: > For me, this [1] fixes the issue: commit d5495caa in tommaso/master Thanks for coming up with a fix and sorry for the trouble my commit caused. > FYI, with this patch, I can finally have a proper patch for gnuplot scripts: >

Re: LyXRC - %%s/scripts

2016-10-23 Thread Tommaso Cucinotta
On 20/10/2016 18:11, Scott Kostyshak wrote: I have no idea. If you don't figure out the problem/fix, I can probably take a look at this next week. For me, this [1] fixes the issue: commit d5495caa in tommaso/master FYI, with this patch, I can finally have a proper patch for gnuplot scripts:

Re: LyXRC - %%s/scripts

2016-10-20 Thread Stephan Witt
Am 19.10.2016 um 23:54 schrieb Tommaso Cucinotta : > > On 19/10/2016 23:10, Scott Kostyshak wrote: >> See 8b66f9ce. Perhaps the commit introduced a bug? > > might be. AFAICS, the attempt to replace $$s centrally in > filetools.cpp:commandPrep() fails because the string getting

Re: LyXRC - %%s/scripts

2016-10-20 Thread Scott Kostyshak
On Wed, Oct 19, 2016 at 11:54:06PM +0200, Tommaso Cucinotta wrote: > Do you know when / under what condition, such a Python intermediate script is > created and how to prevent that? I have no idea. If you don't figure out the problem/fix, I can probably take a look at this next week. Scott

Re: LyXRC - %%s/scripts

2016-10-19 Thread Tommaso Cucinotta
On 19/10/2016 23:10, Scott Kostyshak wrote: See 8b66f9ce. Perhaps the commit introduced a bug? might be. AFAICS, the attempt to replace $$s centrally in filetools.cpp:commandPrep() fails because the string getting there, is NOT the converter command as from the \converter line, but rather a

Re: LyXRC - %%s/scripts

2016-10-19 Thread Scott Kostyshak
On Wed, Oct 19, 2016 at 10:57:05PM +0200, Tommaso Cucinotta wrote: > Hi, > > LyXRC specifies "%%s/scripts/" as converter command, but how/where is the > %%s/ expanded/processed, when running those commands? See 8b66f9ce. Perhaps the commit introduced a bug? Scott signature.asc