|
From: <br...@ph...> - 2006-07-18 18:30:05
|
Kim Leng Goh wrote: > I was following the instructions from http://www.gnuplot.info/links.html : > "This is a completely unorganised list of gnuplot related web and ftp > sites. Please write to gnuplot-beta mailing list if you know any other > useful links." For telling us about the program, you followed it correctly. But that doesn't mean we're interested in reading about possible bugs in other people's programs. Even if those programs use gnuplot. >> > 1) Modify GraphSoapBindingImpl.java and change >> > exe.run(exeGnuPlot + " \"" + cmdsFile + "\"", _tempDir, null); >> > to >> > exe.run(exeGnuPlot + " " + cmdsFile, _tempDir, null); >> >> I don't think that's a good change. After your change, any use of a >> plot filename containing blanks would fail. This change breaks more >> than it fixes > > Without the change, it doesn't work for me. Maybe it works on Windows. > > btw, the plot filename won't contain blanks (ah hah, thanks, now I > what those quotes and back slashes were for): How can you possibly be sure about that? For all you know, a user could well have set up his system such that > GraphSoapBindingImpl.java: cmdsFile = File.createTempFile("gnuplot_", > ".cmds", _tempDir); returns a pathname with a blank somewhere inside it. Note it doesn't have to be in the filename part of the pathname --- it could be in _tempDir, though. |