On Tuesday 09 February 2010, Richard Dale wrote: > On Sunday 07 February 2010 02:41:15 pm Alexander Neundorf wrote: > > On Friday 05 February 2010, Richard Dale wrote: > > > On Friday 05 February 2010 06:57:24 pm Alexander Neundorf wrote: > > > > On Friday 05 February 2010, Melton, Ryan wrote: > > > > > Won't work on Windows. Windows filenames are case-preserving but > > > > > case-insensitive, so FindRuby.cmake and FindRUBY.cmake can't exist > > > > > together in the same directory. Ryan > > > > > > > > Grmbl... > > > > > > > > Should we add a subdirectory cmake/modules/JustForFindRUBY/, add this > > > > also to CMAKE_MODULE_PATH and move FindRUBY.cmake there ? > > > > > > > > I mean, we cannot just replace the file, since then everywhere where > > > > find_package(RUBY) > > > > is used, FindRuby.cmake will not be loaded :-/ > > > > > > > > Does anybody have a non-hackish idea what we can do ? > > > > > > Why do we need two 'find ruby's in the first place? Surely there isn't > > > that muchy Ruby code in KDE to make it too difficult to convert to just > > > use one of them? > > > > Removing (or renaming) FindRUBY.cmake to FindRuby.cmake would be a source > > incompatible change and would break the build of everything that uses it > > (inside KDE svn or outside). > > Yes, but as far as the KDE project is concerned cmake and Ruby are only > used in kdebindings/ruby, and I think we should switch to the standard > script that is provided with cmake 2.8.
I agree, but there is also stuff which uses kdelibs which is not in KDE svn. Removing/renaming FindRUBY.cmake to FindRuby.cmake would break those apps. We may require a cmake 2.8.something for KDE 4.5 maybe... Alex _______________________________________________ Kde-buildsystem mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-buildsystem
