Am 01.06.2013 um 04:37 schrieb Uwe Stöhr <uwesto...@web.de>:
> Am 31.05.2013 19:53, schrieb Stephan Witt:
>
> >> LyX does currently not compile.
> >
> > Here I can compile it.
>
> Jürgen fixed it. I could compile LyX and your change works.
>
>> I read it so one should use quote_shell for shell command arguments.
>> Why then the first quote_python is used?
>
> I think because this path is transferred to the configure.py and so it
> becomes a python script. So until we get any problems, I would leave the code
> as we have it right now.
Me too.
But the documentation in filetools.h clearly states for quote_python:
/*
Quote for python. Use this if you want to store a filename in a python
script.
Example: os << "infile = " << quoteName(filename) << '\\n';
*/
Here we have a command line argument for a python call to be parsed by a shell
or a shell similar QProcess call.
We do not store the argument in a python script.
So it should be quote_shell. Or do I miss something?
Stephan
