On Tuesday 04 November 2008 13:34:23 Guy K. Kloss wrote:
> Thanks to Kai-Uwe and Hal for the proposed fixes. I got further, but it
> still doesn't want to build :-/
>
> On Mon, 03 Nov 2008 21:21:35 Kai-Uwe Behrmann wrote:
> > attached you will find a patch for basically compiling the library and
> > tifficc. It consists of a trivial makefile and some fixes/hacks (take
> > your pick;) to let the code run on Linux.
>
> I couldn't find a make file, just the diff for lcms2.h containing the added
> pre-processor macros.

The patch has the makefile.  I only had to add the include for stddef.h and 
add -fPIC to the CFLAGS line of the makefile that the patch created and it 
built.

>
> On Tue, 04 Nov 2008 12:36:16 Hal V. Engel wrote:
> > It also needs:
> >
> > #include <stddef.h>
> >
> > in lcms2.h to build.  It also needs -fPIC added to the CFLAGS on amd64
> > systems.
>
> And I needed apparently a -DNON_WINDOWS switch for gcc.

I didn't need to do this.  The only place NON_WINDOWS is used is in cmscgats.c 
and it is used to #include <sys/io.h>, define the directory seperator 
character and in a function that checks to see if the path is absolute.  
Adding this to my CFLAGS didn't seem to affect compilation at all on my 
system.  Does cmscgats.c even need anything form sys/io.h?

>
> Next to a bag full of warnings is now barfs at at cmscnvrt.c here:
>
> src/cmscnvrt.c:429: error: static declaration of '_cmsDefaultICCintents'
> follows non-static declaration
> src/cmscnvrt.c:45: error: previous declaration of '_cmsDefaultICCintents'
> was here
> scons: *** [Debug/cmscnvrt.os] Error 1
> scons: building terminated because of errors.
>
>
> Just removing the keyword "static" from the implementation in line 428
> doesn't help, as something still forks up somewhere else in cmsintrp.c
> then:
>
> src/cmsintrp.c:189: error: conflicting types for '_cmsComputeInterpParams'
> src/lcms2_internal.h:312: error: previous declaration of
> '_cmsComputeInterpParams' was here
> scons: *** [Debug/cmsintrp.os] Error 1
> scons: building terminated because of errors.
>
>
> Any further input would be highly welcome. BTW, did it actually already
> compile for you (Kai-Uwe and Hal) on a non-Wintendo platform?

Yes but it appears that NON_WINDOWS does need to be defined to get the correct 
definition for the directory separator and to make the absolute path function 
work correctly.

Hal

-------------------------------------------------------------------------
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
Build the coolest Linux based applications with Moblin SDK & win great prizes
Grand prize is a trip for two to an Open Source event anywhere in the world
http://moblin-contest.org/redirect.php?banner_id=100&url=/
_______________________________________________
Lcms-user mailing list
Lcms-user@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/lcms-user

Reply via email to