Am 02.01.2019 um 16:36 schrieb Stephan Witt <st.w...@gmx.net>: > > Am 01.01.2019 um 19:57 schrieb Enrico Forestieri <for...@lyx.org>: >> >> On Tue, Jan 01, 2019 at 02:10:37PM +0100, Stephan Witt wrote: >>> Hi all, >>> >>> first I wish all of you a Happy New Year. >>> >>> I investigated the cause of the issue with displaying SVG (including our own >>> icons in Users Guide) on-screen. >>> >>> IMO the root cause is the existence of two ways to use the configured >>> converters: >>> 1. the conversion process to load external material to present it on-screen >>> (build_script in GraphicsConverter.cpp) and >>> 2. the conversion process to prepare external material for use by external >>> programs like LaTeX (Converters::convert) >>> >>> Both are using the utilities detected by lib/configure.py. But the meaning >>> of >>> the „$$i“ and „$$o“ placeholders is different. (1) is passing the complete >>> absolute path of input and output file. (2) is operating in the temporary >>> directory and uses relative path names only. On Mac the inkscape binary is a >>> shell script itself and it’s not working with relative path names for input >>> and output files. Therefore I added the prefix „$$p“ in front of the names >>> to >>> get absolute path names. Unfortunately this breaks the conversion (1). >>> >>> One possible solution for the current issue is it to change (1) to replace >>> „$$p“ with an empty string. Is this a good idea or is some more fundamental >>> change to unify the converters to prefer? Any other ideas? >> >> What about changing (1) to use relative path names for "$$i" and "$$o" >> and then adding "$$p" to obtain absolute ones? In this way "$$p" maintains >> its meaning in both cases. > > Ok, that’s a possibility - and I like it. But it would be a „more fundamental > change“, IMO. > I’ll see how much work it is and if I can test it.
I’m trying to verify how this can work without regressions. ATM, I don’t understand how the Rscript converters can work - there are $$p$$i and $$p$$o parameters for converter configuration in configure.py already. Does it work on Linux and how? I don’t have it here and cannot test it. Stephan
