Review Request: Install LibKexiv2Config.cmake

Alexander Neundorf neundorf at kde.org
Mon Jan 23 20:36:30 UTC 2012


On Sunday 15 January 2012, Yury G. Kudryashov wrote:
> Le 15 января 2012 15:51:32 Alexander Neundorf a écrit :
> > On Sunday 15 January 2012, Yury Georgievich Kudryashov wrote:
> > > > On Jan. 15, 2012, 1:45 p.m., Alexander Neundorf wrote:
> > > > > IF(WIN32 AND NOT CYGWIN)
> > > > > 
> > > > >     SET(DEF_CMAKEFILES_INSTALL_DIR CMake)
> > > > > 
> > > > > ELSE()
> > > > > 
> > > > >     SET(DEF_CMAKEFILES_INSTALL_DIR
> > > > >     ${LIB_INSTALL_DIR}/cmake/LibKexiv2)> > >
> > > > > 
> > > > > ENDIF()
> > > > > 
> > > > > Why is WIN32 special cased ?
> > > 
> > > `man cmake` says that cmake does not look into $prefix/lib/cmake/name
> > > on Windows.
> > 
> > Indeed.
> > Our KDE-on-Windows never complained about that the Config.cmake files
> > would not be found.
> > Also, from a quick look at cmFindPackageCommand.cxx, it looks like lib/
> > and
> 
> > share/ should also be searched under Windows, no ifdefs there:
> Should I prepare a patch to cmake documentation?

Improving docs is always good :-)

Alex


More information about the Kde-buildsystem mailing list