Am Montag, 4. August 2008 18:02:15 schrieb Bill Hoffman: > Hendrik Sattler wrote: > > Am Montag, 4. August 2008 17:48:28 schrieb Mathieu Malaterre: > >> That was not really my main concern anyway, instead I am more > >> concerned with the patch done for debian people (admittedly I don"t > >> understand what they are trying to do): > >> > >> http://ftp.de.debian.org/debian/pool/main/c/cmake/cmake_2.6.0-5.diff.gz > > > > The have > > /lib > > /lib32 -> /emul/ia32-linux/lib > > /lib64 -> /lib > > > > So adding the /lib64 to the library search will gain nothing as > > everything is at /lib already. The /lib64 link is only a compatibility > > thing for other distros that interpret the FHS in strange ways... > > But, does it hurt? I don't see us moving that into the main tree with > the if 0 as the lib64 actually does do something on other machines. I > guess we could have an if _DEBIAN_ or something like that. Or maybe we > could check to see if /lib == /lib64 as other distros might do the same > thing.
On a second look, the patch does not select /lib64 even when compiling 64bit binaries on 32bit system as cmSystemTools:FileIsDirectory() will return false on a symlink. Well that is broken in itself as having /lib64 as symlink is valid. Debian could also solve this by seeing the compilation of 64bit binaries on 32bit systems as kind of cross-compiling. Not sure how the layout is on i386. Why aren't the same checks done when compiling 32bit binaries on 64bit system, meaning using /lib32 instead of /lib in this case? If those stuff is not meant for 32<->64 bit "cross-compiling" but for some other directory layouts, everything is fine and the patch does essentially nothing (except saving a few cpu cycles). You might want to speak to the Debian Maintainer in this case, see http://packages.debian.org/sid/cmake (right side). Cmake-2.6 is not going to be in Debian Lenny, so no real time pressure on this. HS _______________________________________________ CMake mailing list CMake@cmake.org http://www.cmake.org/mailman/listinfo/cmake