Interesting problem with FindCLucene.cmake

[Ismail Dönmez]

Monday 14 January 2008 23:33:01 tarihinde Alexander Neundorf şunları yazmıştı:
> 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.

Thanks, will do.



-- 
Never learn by your mistakes, if you do you may never dare to try again.