default converter for EPS -> PNG is "convert" on Windows

[Yu Jin]

Am Sa., 28. Nov. 2020 um 12:43 Uhr schrieb Yu Jin <yu_jin at lyx.org>:

> Am Sa., 28. Nov. 2020 um 11:19 Uhr schrieb Yu Jin <yu_jin at lyx.org>:
>
>> When I install LyX on Windows, the converter for eps -> png is "convert",
>> while for enhanced metafile -> eps and Windows metafile -> eps is "magick".
>> I'm only packaging magick in the installers, so "convert" does not exist
>> and therefore eps/pdf preview does not work. When I run reconfigure from
>> the GUI, then the eps -> png converter is changed to magick and the preview
>> works. Is there a reason for "convert" in this converter or can we change
>> the default to "magick" too? Or should I be packaging both convert and
>> magick (which would be additional 15 MB though because of the portable
>> imagemagick), I'd rather not, because magick and convert are the exact same
>> files with different names (portable imagemagick release).
>>
>
> There might be a bug somewhere, the configure log says:
>
> INFO: checking for an EPS -> PNG converter...
> INFO: +checking for "magick"...  no
> INFO: +checking for "convert"...  yes
> DEBUG: Add to RC:
> \converter eps        png        "convert $$i $$o" ""
>
> I will try to find it.
>
Oh, I see what is going on here. First the convert found in configure
script is not the imagemagick convert, but the Windows convert:
https://savage.net.au/ImageMagick/html/install-convert.html

When configure.py is executed from the Windows installer, the environment
it inherits does not contain the paths defined in the LyX's path prefix, so
it does not find magick. It finds Windows' convert instead
(C:\Windows\System32\convert.exe)...

I will fix it in the Installer by updating/manipulating the environment
just before running the script. But as for the convert.exe, I propose to
get rid of it, if it's just for this eps -> png converter.
-- 
    Eugene
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.lyx.org/pipermail/lyx-devel/attachments/20201128/a6142bc1/attachment.html>