Inkscape rev 12138 will not start from Windows Explorer gui (missing liblcms-1.dll)

[Alvin Penner]

- this rev starts normally from a command prompt
- if you double-click on Inkscape.exe directly from the Explorer gui, or if
- you right-click on a svg file and open it with Inkscape, then you get the attached message concerning liblcms-1.dll:
- the file liblcms-1.dll is not in the Inkscape installation directory, where it normally would be. Instead the file liblcms2-2.dll is present.

[jazzynico (jazzynico)]

Reproduced on Windows XP, Inkscape revision 12138.
I've recently modified the build.xml file to replace lcms1 with lcms2 (that was added with libcdr support). I guess there's something wrong here...

[jazzynico (jazzynico)]

Some precisions. Since there's no lcms1 call in the trunk code, it is not needed by Inkscape itself, but by one (or some) of its dependencies that was compiled with lcms1...
I suggest we just copy the missing lib in the install folder as before, but still use lcms2 in Inkscape.

<off-topic>
I have high hopes that the cross-compiled libs could replace the current devlibs completely. Then we would have something more consistent, cleaner, and far easier to manage.
</off-topic>

[jazzynico (jazzynico)]

Should be fixed revision 12139.
Thanks for the report, Alvin!

[Alvin Penner (apenner)]

thanks, that appears to have fixed it.

I see that the file cms-color-types.h has a reference to LCMS1 with no corresponding ref to LCMS2. Most of the other files appear to refer to either one, using whichever is present.

[jazzynico (jazzynico)]

If I understand correctly, cms-color-types.h is just a compatibility layer that provides unique class and type names that can be used with lcms1 or lcms2, and thus should not be the cause of the issue (HAVE_LIBLCMS1 is not set in build.xml).

[Alvin Penner (apenner)]

yes, sorry for the noise. After I looked inside that file I saw that this was just a kind of conditional header for other stuff. Packaging has always been a mystery to me, so I better stop commenting before I say something I'll regret...

[jazzynico (jazzynico)]

Alvin, don't be sorry, I could have done something wrong when switching to lcms2, or missed an include somewhere in the code.

[jazzynico (jazzynico)]

Tested on Windows XP, Inkscape revision 12147 with the OpenSuse cross-compiled libs, and everything works as expected without liblcms1.dll.