Re: path of pstoedit (and fig2dev) for fltk printing on octave for windows

[Ben Abbott]

On Aug 12, 2010, at 5:27 PM, Tatsuro MATSUOKA wrote:

> Hello
> 
> I have tried to print emf file in the recent development source.
> (The latest changeset id is changeset 10885 36a7163a5484. Wed Aug 11 14:55:10)
> 
> I have met following error.
> 
> octave:19> print('fltk_test.emf','-demf')
> pstoedit command: c:\Program Files\pstoedit/pstoedit.exe -f fig
> C:/DOCUME~1/Tatsu/LOCALS~1/Temp\oct-15.eps 
> C:/DOCUME~1/Tatsu/LOCALS~1/Temp\oct-14.fig 2>&1'c:\Program'
> is not recognized as an internal or external command, operable program or 
> batch file.
> 
> warning: print.m: error running pstoedit.
> fig2dev command: c:\Program Files\fig2dev/fig2dev.exe -L emf
> C:/DOCUME~1/Tatsu/LOCALS~1/Temp\oct-14.fig fltk_test.emf 2>&1'c:\Program' is 
> not recognized as an
> internal or external command, operable program or batch file.
> 
> warning: print.m: error running fig2dev.
> No such file or directory
> warning: print.m: failed to delete temporay file, 
> 'C:/DOCUME~1/Tatsu/LOCALS~1/Temp\oct-14.fig'.
> 
> Obviously this is white space problem.
> 
> I have reinstall pstoedit (and fig2dev) to the directory path of which does 
> not have any white space
> and modify path environmental variable.  In this case, of course, printing 
> itself went well.
> 
> For the fig2dev, there are not default directory.  However, for pstoedit, the 
> default install path is
> "C:\Program Files\pstoedit".
> 
> If the command pstoedit command
> c:\Program Files\pstoedit/pstoedit.exe
> is quoted by double quotes "" like
> "c:\Program Files\pstoedit/pstoedit.exe"
> Perhaps error may not be appeared.
> 
> The above treatment is perhaps only required on octave for windows (cygwin is 
> not the case) and the
> function ispc can be used.
> 
> 
> I have quick look at the script but did not figure out where I have to modify.
> 
> BTW, for ghostscript on windows, the default install directory is "c:\gs"
> 
> Regards
> 
> Tatsuro 

Please try the attached. If it works as expected, I'll push the change.

Ben

changeset.patch
Description: Binary data