Hello Peter, * Peter O'Gorman wrote on Wed, Jun 06, 2007 at 07:22:56AM CEST: > On Mac OS X it is getting fairly common to configure with -isysroot > in CFLAGS (hopefully will become less common again when all systems > are fully fat in 10.5), this patch solves the issue that there are > no .la files in the sysroots, but there are in /.
It seems weird that this should be fixed on Darwin only but remain broken everywhere else. IIRC there were reports for other systems, too. OTOH, your patch isn't a whole fix: it will cause libtool to put the wrong paths into new installed .la files. Things get even worse if in the sysroot there are already installed .la files (because then /sys/root/.../lib*.la will end up in dependency_libs). So the fact that there are no .la files there helps a wee bit but not fully. A full solution would keep in mind the sysroot prefix and carry the information through all the way. That's definitely post next release I would say though. So maybe after all there is a reason to accept it for Darwin but nowhere else. Hmm. I'm quite undecided here. What a mess. A small style nit: please do not use sed and awk in one pipe. Drop the case construct and either something like _lt_root=`echo $ac_link" " | sed -n 's/.* -isysroot \([^ ]*\) .*/\1/p` if test -n "$_lt_root"; then ... fi or just walk the arguments (for lt_arg in $ac_link ...) Cheers, Ralf