Re: [PATCH] Conversion of the last two shell scripts fig2pstex and fig2pdftex.

2006-06-30 Thread Enrico Forestieri
On Fri, Jun 30, 2006 at 02:56:14PM -0500, Bo Peng wrote: > > I don't know if this is because the cygwin python needs sh for spawning > > external programs. Someone with a native python should perform the test. > > > > Under linux, when sh is removed, lyx2lyx does not work, although there > is no

Re: [PATCH] Conversion of the last two shell scripts fig2pstex and fig2pdftex.

2006-06-30 Thread Bo Peng
I don't know if this is because the cygwin python needs sh for spawning external programs. Someone with a native python should perform the test. Under linux, when sh is removed, lyx2lyx does not work, although there is no sign of sh being called in lyx2lyx. You are right, we need to test nativ

Re: [PATCH] Conversion of the last two shell scripts fig2pstex and fig2pdftex.

2006-06-30 Thread Enrico Forestieri
On Fri, Jun 30, 2006 at 09:18:53AM -0500, Bo Peng wrote: > > > > > > I like this *a lot* better than executing two commands. If you have > > > tested it, I will apply it unless I heard some other opinions. > > > > Yep, it works. > > OK. I will commit it. Please remove your windows/mingw/sh and s

Re: [PATCH] Conversion of the last two shell scripts fig2pstex and fig2pdftex.

2006-06-30 Thread Bo Peng
> > I like this *a lot* better than executing two commands. If you have > tested it, I will apply it unless I heard some other opinions. Yep, it works. OK. I will commit it. Please remove your windows/mingw/sh and see if sh is needed anywhere else. Bo

Re: [PATCH] Conversion of the last two shell scripts fig2pstex and fig2pdftex.

2006-06-30 Thread Enrico Forestieri
On Fri, Jun 30, 2006 at 11:15:23AM +0200, Enrico Forestieri wrote: > works in the windows shell, too ;-) ^^^ Oops... -- Enrico

Re: [PATCH] Conversion of the last two shell scripts fig2pstex and fig2pdftex.

2006-06-30 Thread Enrico Forestieri
On Fri, Jun 30, 2006 at 08:28:52AM +0200, Georg Baum wrote: > Abdelrazak Younes wrote: > > > Indeed, for me neither. Works for other programs though... Surely > > because it is a graphical program. > > No. The reason is that debug output is sent to stderr, not stdout. "lyx -dbg > info 2> foo.log

Re: [PATCH] Conversion of the last two shell scripts fig2pstex and fig2pdftex.

2006-06-30 Thread Enrico Forestieri
On Thu, Jun 29, 2006 at 04:11:10PM -0500, Bo Peng wrote: > > > Do you want a patch along these lines? I think it is more elegant. > > > > Here is my proposed patch. > > I like this *a lot* better than executing two commands. If you have > tested it, I will apply it unless I heard some other opin

Re: [PATCH] Conversion of the last two shell scripts fig2pstex and fig2pdftex.

2006-06-29 Thread Georg Baum
Abdelrazak Younes wrote: > Indeed, for me neither. Works for other programs though... Surely > because it is a graphical program. No. The reason is that debug output is sent to stderr, not stdout. "lyx -dbg info 2> foo.log" works on linux, but I don't know if the windows shell is clever enough to

Re: [PATCH] Conversion of the last two shell scripts fig2pstex and fig2pdftex.

2006-06-29 Thread Bo Peng
> Do you want a patch along these lines? I think it is more elegant. Here is my proposed patch. I like this *a lot* better than executing two commands. If you have tested it, I will apply it unless I heard some other opinions. Bo

Re: [PATCH] Conversion of the last two shell scripts fig2pstex and fig2pdftex.

2006-06-29 Thread Enrico Forestieri
On Thu, Jun 29, 2006 at 10:07:53PM +0200, Enrico Forestieri wrote: > On Thu, Jun 29, 2006 at 01:32:27PM -0500, Bo Peng wrote: > > OK. Here is the suggested patch that removes the last (?) call to > > shell. Any objection? > > Actually, I was trying another solution... > My idea is to query the reg

Re: [PATCH] Conversion of the last two shell scripts fig2pstex and fig2pdftex.

2006-06-29 Thread Enrico Forestieri
On Thu, Jun 29, 2006 at 01:32:27PM -0500, Bo Peng wrote: > OK. Here is the suggested patch that removes the last (?) call to > shell. Any objection? Actually, I was trying another solution... My idea is to query the registry about cygwin. I just tested the attached source and it works... Do you w

Re: [PATCH] Conversion of the last two shell scripts fig2pstex and fig2pdftex.

2006-06-29 Thread Bo Peng
OK. Here is the suggested patch that removes the last (?) call to shell. Any objection? Index: src/support/os_win32.C === --- src/support/os_win32.C (revision 14265) +++ src/support/os_win32.C (working copy) @@ -163,8 +163,

Re: [PATCH] Conversion of the last two shell scripts fig2pstex and fig2pdftex.

2006-06-29 Thread Bo Peng
> >That's from src/support/os_win32.C in the init() function. > >Here "sh -c uname" is tried in order to detect a cygwin environment. > >If the command is not found, it is assumed that there is no such > >environment. Hence, the warning is not harmful. > > If you only want to check if an environme

Re: [PATCH] Conversion of the last two shell scripts fig2pstex and fig2pdftex.

2006-06-29 Thread Abdelrazak Younes
Edwin Leuven wrote: Abdelrazak Younes wrote: You can adjust the proprerties by clicking on the top-left corner of the window. i know, but i cannot increase the buffer beyond 999 lines... Weird, mine is 3000. You can also do "lyx -dbg info > foo.log" doesn't do anything for me Indeed,

Re: [PATCH] Conversion of the last two shell scripts fig2pstex and fig2pdftex.

2006-06-29 Thread Enrico Forestieri
On Thu, Jun 29, 2006 at 11:33:29AM -0500, Bo Peng wrote: > >That's from src/support/os_win32.C in the init() function. > >Here "sh -c uname" is tried in order to detect a cygwin environment. > >If the command is not found, it is assumed that there is no such > >environment. Hence, the warning is no

Re: [PATCH] Conversion of the last two shell scripts fig2pstex and fig2pdftex.

2006-06-29 Thread Edwin Leuven
Abdelrazak Younes wrote: You can adjust the proprerties by clicking on the top-left corner of the window. i know, but i cannot increase the buffer beyond 999 lines... You can also do "lyx -dbg info > foo.log" doesn't do anything for me

Re: [PATCH] Conversion of the last two shell scripts fig2pstex and fig2pdftex.

2006-06-29 Thread Abdelrazak Younes
Edwin Leuven wrote: Bo Peng wrote: when i launch lyx is see this in the command window: Could you provide more debug information, like lyx -dbg info? have no idea how to make the command buffer store more than 999 lines in windows... You can adjust the proprerties by clicking on the top-

Re: [PATCH] Conversion of the last two shell scripts fig2pstex and fig2pdftex.

2006-06-29 Thread Bo Peng
That's from src/support/os_win32.C in the init() function. Here "sh -c uname" is tried in order to detect a cygwin environment. If the command is not found, it is assumed that there is no such environment. Hence, the warning is not harmful. If you only want to check if an environment is cygwin,

Re: [PATCH] Conversion of the last two shell scripts fig2pstex and fig2pdftex.

2006-06-29 Thread Enrico Forestieri
On Thu, Jun 29, 2006 at 05:43:15PM +0200, Edwin Leuven wrote: > Bo Peng wrote: > >>does this mean we don't need shell anymore? > > > >Yes. No shell is needed for lyx/trunk now. The windows installer of > >lyx-1.4.3 (?) no longer has to bundle sh, sed, find etc. > > when i launch lyx is see this in

Re: [PATCH] Conversion of the last two shell scripts fig2pstex and fig2pdftex.

2006-06-29 Thread Edwin Leuven
Bo Peng wrote: when i launch lyx is see this in the command window: Could you provide more debug information, like lyx -dbg info? have no idea how to make the command buffer store more than 999 lines in windows...

Re: [PATCH] Conversion of the last two shell scripts fig2pstex and fig2pdftex.

2006-06-29 Thread Bo Peng
when i launch lyx is see this in the command window: Could you provide more debug information, like lyx -dbg info? Bo

Re: [PATCH] Conversion of the last two shell scripts fig2pstex and fig2pdftex.

2006-06-29 Thread Bo Peng
when i launch lyx is see this in the command window: 'sh' is not recognized as an internal or external command, operable program or batch file. Under linux, if I remove /bin/sh, I do not see this error, but lyx2lyx does not work. Bo

Re: [PATCH] Conversion of the last two shell scripts fig2pstex and fig2pdftex.

2006-06-29 Thread Edwin Leuven
Bo Peng wrote: does this mean we don't need shell anymore? Yes. No shell is needed for lyx/trunk now. The windows installer of lyx-1.4.3 (?) no longer has to bundle sh, sed, find etc. when i launch lyx is see this in the command window: 'sh' is not recognized as an internal or external comm

Re: [PATCH] Conversion of the last two shell scripts fig2pstex and fig2pdftex.

2006-06-29 Thread Bo Peng
does this mean we don't need shell anymore? Yes. No shell is needed for lyx/trunk now. The windows installer of lyx-1.4.3 (?) no longer has to bundle sh, sed, find etc. Bo

Re: [PATCH] Conversion of the last two shell scripts fig2pstex and fig2pdftex.

2006-06-29 Thread Edwin Leuven
Bo Peng wrote: Here is the result of my tests. Thank you very much. The problems have been corrected, and the patch submitted. does this mean we don't need shell anymore?

Re: [PATCH] Conversion of the last two shell scripts fig2pstex and fig2pdftex.

2006-06-29 Thread Bo Peng
Here is the result of my tests. Thank you very much. The problems have been corrected, and the patch submitted. Bo

Re: [PATCH] Conversion of the last two shell scripts fig2pstex and fig2pdftex.

2006-06-29 Thread Enrico Forestieri
On Wed, Jun 28, 2006 at 10:44:55PM -0500, Bo Peng wrote: > Attached. Please test. > > Method: > python fig2pstex.py file.fig file.pstex > python fig2pdftex.py file1.fig file1.pdftex > > You should get valid file.eps, file.pstex, file1.pdf, file1.pdftex. Here is the result of my tests. 1) When t

[PATCH] Conversion of the last two shell scripts fig2pstex and fig2pdftex.

2006-06-28 Thread Bo Peng
Attached. Please test. Method: python fig2pstex.py file.fig file.pstex python fig2pdftex.py file1.fig file1.pdftex You should get valid file.eps, file.pstex, file1.pdf, file1.pdftex. Bo Index: lib/scripts/fig2pdftex.py === --- lib/