[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-devel
Subject:    Re: make -f Makefile.cvs failes for kdeextragear-1
From:       David Faure <faure () kde ! org>
Date:       2005-01-07 22:34:13
Message-ID: 200501072334.13757.faure () kde ! org
[Download RAW message or body]

On Friday 07 January 2005 20:42, Jonathan Brugge wrote:
> On Friday 07 January 2005 19:17, Miles Stevenson wrote:
> > gwenview/gvimageutils/Makefile.am:9: libgvimageutils_la_SOURCES was already
> > defined in condition TRUE, which implies condition GV_USE_MMX_TRUE
> >
> >   libgvimageutils_la_SOURCES (User, where =
> > gwenview/gvimageutils/Makefile.am:9) +=
> >   {
> >     TRUE => \
> >         gvimageutils.cpp \
> >         jpegcontent.cpp \
> >         scale.cpp \
> >         transupp.c
> >
> >   }
> > gwenview/gvimageutils/Makefile.am:9: warning: automake does not support
> > conditional definition of libgvimageutils_la_SOURCES in
> > libgvimageutils_la_SOURCES
> > Use of uninitialized value in concatenation (.) or string
> > at /usr/bin/automake-1.6 line 8449.
> >
> > : am_libgvimageutils_la_OBJECTS was already defined in condition TRUE,
> > : which
> >
> > implies condition GV_USE_MMX_TRUE
> >   am_libgvimageutils_la_OBJECTS (Automake, where = undefined) =
> >   {
> >     TRUE => gvimageutils.lo jpegcontent.lo scale.lo transupp.lo
> >   }
> > make[1]: *** [cvs] Error 1
> > make: *** [all] Error 2
> 
> I solved this by turning the following lines in 
> gwenview/gvimageutils/Makefile.am into comments:
> 
> if GV_USE_MMX
> libgvimageutils_la_SOURCES += asm_scale.S
> endif

Conditional sources are bad, from what I hear, due to dependency problems.
A solution is a subdir with a noinst_LTLIBRARIES. Coolo, any more lightweight solution?

-- 
David Faure, faure@kde.org, sponsored by Trolltech to work on KDE,
Konqueror (http://www.konqueror.org), and KOffice (http://www.koffice.org).
 
>> Visit http://mail.kde.org/mailman/listinfo/kde-devel#unsub to unsubscribe <<
[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic