On Tue, Jul 17, 2007 at 06:48:38AM -0600, Eric Blake wrote: > According to Peter Rosin on 7/17/2007 2:49 AM: > > I think the test will also fail on cygwin with managed mount, this > > should probably be detected and the test should be skipped if run > > on a managed mount just as it is skipped on non-(cygwin|mingw). > > I still think searching for libs case-insensitively on cygwin is wrong - > the philosophy of cygwin is to be a Linux emulation, where case matters; > and even though you can't have dual case files without a managed mount, it > just seems like perpetuating problems if cygwin allows a library with the > wrong spelling. For mingw, on the other hand, the goal is indeed to be as > accomodating to Windows as possible, so having libtool ignore case in > library searches on mingw seems like it might be acceptable (although I > have not reviewed your patch for its technical merits).
I do not really care one way or the other for this patch and cygwin, but I will say this. The only thing that will happen (if I got it right) is that libtool will refuse to build a shared lib due to missing deps (and -no-undefined). With this nocaseglob patch, libtool will build a shared lib. The .la file will continue to list -lfoo as a dep even without the patch so whoever uses the lib will still link with the Foo shared lib in the end. All this of course assumes non-managed mounts. It is easy enough to exclude cygwin from it, but should I? Cheers, Peter
