KDE/kdelibs/kdecore
Alexander Neundorf
neundorf at kde.org
Wed Apr 29 19:17:51 CEST 2009
On Wednesday 29 April 2009, Andreas Pakulat wrote:
> On 29.04.09 11:33:05, Andrew Coles wrote:
> > SVN commit 960968 by coles:
> >
> > Likewise, need to add the library path too.
> >
> >
> >
> > M +1 -0 CMakeLists.txt
> >
> >
> > --- trunk/KDE/kdelibs/kdecore/CMakeLists.txt #960967:960968
> > @@ -44,6 +44,7 @@
> > # compile lzma support if available
> > if(LIBLZMA_FOUND)
> > include_directories(${LIBLZMA_INCLUDEDIR})
> > + link_directories(${LIBLZMA_LIBRARY_DIRS})
> > set(kdecore_OPTIONAL_SRCS ${kdecore_OPTIONAL_SRCS}
> > compression/kxzfilter.cpp ) set(kdecore_OPTIONAL_LIBS
> > ${kdecore_OPTIONAL_LIBS} ${LIBLZMA_LIBRARIES})
>
> This seems wrong. LIBLZMA_LIBRARIES should already contain absolute paths
> to the LZMA libs, hence the library dirs variable is uneeded.
Andreas is right, FindLibLZMA.cmake is broken, don't hack around this in the
CMakeLists.txt
Alex
More information about the Kde-buildsystem
mailing list