On 08/31/2011 03:25 PM, Julien Rioux wrote: > Attached is a patch to fix issues #6339 and #7579 in configure. The > bug is that we check whether perl or java are installed, but we do not > check that the actual scripts are also installed. > > In the patch, I look once for the java interpreter and once for the > perl interpreter. This information is passed onto the subsequent > checks for converters. At the moment, we search once for a perl script > and twice for a java script. Both the interpreter and the script needs > to be present before we add a line to lyxrc.defaults > > One question I have is this: > > addToRC(r'\splitindex_command "%s -w %s"' % (perl, os.path.join(path, > splitindex))) > > I feel that I need to add "" around the second %s to protect against > spaces in the path to splitindex, but how to add them? maybe like this: > > addToRC(r'\splitindex_command "%s -w \"%s\""' % (perl, > os.path.join(path, splitindex))) > > is that right? > I don't think that will work. There are already double quotes there, so the first of the ones you are adding will close the first one already there. Perhaps you could add (escaped) single quotes?
Richard
