Interesting problem with FindCLucene.cmake

Alexander Neundorf neundorf at kde.org
Mon Jan 14 22:33:01 CET 2008


On Monday 14 January 2008, Ismail Dönmez wrote:
> Monday 14 January 2008 21:58:56 tarihinde Alexander Neundorf şunları 
yazmıştı:
> > On Monday 14 January 2008, Ismail Dönmez wrote:
> > > Monday 14 January 2008 21:18:07 tarihinde Andreas Pakulat şunları
>
> yazmıştı:
> > > > Well, check whats inside TRIAL_LIBRARY_PATHS, probably that throws
> > > > off find_path.
> > >
> > > I didn't get what you mean by "throws off", here is the definition for
> > > TRIAL_LIBRARY_PATHS :
> > >
> > >   SET(
> > >     $ENV{CLUCENE_HOME}/lib${LIB_SUFFIX}
> > >     ${CMAKE_INSTALL_PREFIX}/lib${LIB_SUFFIX}
> > >     /usr/local/lib${LIB_SUFFIX}
> > >     /usr/lib${LIB_SUFFIX}
> > >     /sw/lib${LIB_SUFFIX}
> > >     NO_DEFAULT_PATH
> > >   )
> > >
> > > Regards,
> > > ismail
> >
> > Without looking at the docs or sources:
> >
> > This means the call looks something like this:
> >
> > FIND_PATH(CLUCENE_LIBRARY_DIR
> >       NAMES CLucene/clucene-config.h PATHS
> > $ENV{CLUCENE_HOME}/lib${LIB_SUFFIX}
> > ${CMAKE_INSTALL_PREFIX}/lib${LIB_SUFFIX}
> >     /usr/local/lib${LIB_SUFFIX}
> >     /usr/lib${LIB_SUFFIX}
> >     /sw/lib${LIB_SUFFIX}
> >     NO_DEFAULT_PATH
> >     /usr/include
> >     )
> >
> > NO_DEFAULT_PATH should be at the end of the list of arguments, so the
> > directory (directories) coming after it are probably ignored.
> >
> > If you remove NAMES and PATHS you get probably the old "signature" of
> > FIND_PATH, which is more or less just
> > FIND_PATH(VAR <filename> <path1> ... <pathN>)
> > so that NO_DEFAULT_PATH is interpreted as a directory and so the default
> > paths are searched.
>
> Ok then FindCLucene.cmake files in KDE4 repo is broken :-) How shall we
> proceed, am I allowed to remove NO_DEFAULT_PATH from TRIAL_LIBRARY_PATHS ?

I'd say remove it from the variables and add it explicitely as the last 
argument for all commands where these variables are used.

Alex


More information about the Kde-buildsystem mailing list