Re: [PATCH v2 0/3] sysroot followup patches

2010-08-22 Thread Paolo Bonzini
Paolo, are you going to squash this: http://lists.gnu.org/archive/html/libtool-patches/2010-08/msg00145.html perhaps without this change: -      if $opt_dry_run; then +      if ${opt_dry_run-false}; then into [PATCH] improve code for sysroot --mode=finish.eml Yes of course. Paolo

Re: [PATCH v2 0/3] sysroot followup patches

2010-08-22 Thread Ralf Wildenhues
Hello Paolo, Charles, I think these are the leftover items, right? * Paolo Bonzini wrote on Fri, Aug 13, 2010 at 03:23:02AM CEST: 4) patch for old libtool, to not barf on '=' 6) support in libltdl for '=' in .la files 7) doc updates. I think the sysroot branch is in pretty good shape and all

Re: [PATCH v2 0/3] sysroot followup patches

2010-08-22 Thread Charles Wilson
On 8/22/2010 11:26 AM, Ralf Wildenhues wrote: Hello Paolo, Charles, I think these are the leftover items, right? * Paolo Bonzini wrote on Fri, Aug 13, 2010 at 03:23:02AM CEST: 4) patch for old libtool, to not barf on '=' 6) support in libltdl for '=' in .la files 7) doc updates. Yes,

Re: [PATCH v2 0/3] sysroot followup patches

2010-08-22 Thread Ralf Wildenhues
Hi Charles, I agree with all you wrote. * Charles Wilson wrote on Sun, Aug 22, 2010 at 08:05:33PM CEST: Anyway, this test series will take DAYS to complete...so I'd appreciate it if the review (or, the reviewER's refresh/reread of the OLD reviews, whose links I will post) could commence

Re: [PATCH v2 0/3] sysroot followup patches

2010-08-16 Thread Charles Wilson
On 8/12/2010 9:23 PM, Paolo Bonzini wrote: These cover bullets 3/5 of Charles' list: 3) issue with '# Collect and forward deplibs of preopened libtool libs' In progress? Anything? 4,5) * patch for old libtool, to not barf on '=' * TBD mechanism to strip '=' from .la after

Re: [PATCH v2 0/3] sysroot followup patches

2010-08-16 Thread Ralf Wildenhues
Hi Charles, Paolo, * Charles Wilson wrote on Mon, Aug 16, 2010 at 05:12:03PM CEST: On 8/12/2010 9:23 PM, Paolo Bonzini wrote: These cover bullets 3/5 of Charles' list: 3) issue with '# Collect and forward deplibs of preopened libtool libs' In progress? Anything? 4,5) * patch

Re: [PATCH v2 0/3] sysroot followup patches

2010-08-16 Thread Charles Wilson
On 8/16/2010 3:27 PM, Ralf Wildenhues wrote: And yes, I'm all fine with the answer being distros, in case you need = support, please git cherry-pick $commit if we find out that works reasonably well to solve this issue. So yes, I agree with you on this. (We might want to make a branch-2.2.X

Re: [PATCH v2 0/3] sysroot followup patches

2010-08-16 Thread Charles Wilson
On 8/16/2010 11:12 AM, Charles Wilson wrote: mingw native (e.g. on MSYS): == old: 2 of 122 tests failed (2 tests were not run) (**) new: in progress. will update later, but I don't expect any regressions. (**) This is longstanding, and fixed by one of my unmerged

Re: [PATCH v2 0/3] sysroot followup patches

2010-08-16 Thread Ralf Wildenhues
* Charles Wilson wrote on Mon, Aug 16, 2010 at 10:27:46PM CEST: Taking all of this together, I wonder if a good approach might be the following: libltdl replaces the '=' with the value of the environment variable LIBLTDL_SYSROOT. So usually -- with this variable empty -- '=' is just stripped

Re: [PATCH v2 0/3] sysroot followup patches

2010-08-16 Thread Charles Wilson
On 8/16/2010 4:42 PM, Ralf Wildenhues wrote: Can you run VERBOSE=yes $srcdir/tests/fcdemo-conf.test; echo $? and post the output of that? VERBOSE=debug is even more informative, if that is necessary. Thanks. Attached. -- Chuck fcdemo-conf.test: === Running fcdemo-conf.test

Re: [PATCH v2 0/3] sysroot followup patches

2010-08-16 Thread Peter Rosin
Den 2010-08-16 22:39 skrev Charles Wilson: 111 appears to be a race condition. The reported error was: c:/mingw/bin/../lib/gcc/mingw32/4.4.0/../../../../mingw32/bin/ld.exe: cannot find

Re: [PATCH v2 0/3] sysroot followup patches

2010-08-16 Thread Ralf Wildenhues
* Charles Wilson wrote on Mon, Aug 16, 2010 at 10:50:21PM CEST: On 8/16/2010 4:42 PM, Ralf Wildenhues wrote: Can you run VERBOSE=yes $srcdir/tests/fcdemo-conf.test; echo $? and post the output of that? VERBOSE=debug is even more informative, if that is necessary. Thanks.

Re: [PATCH v2 0/3] sysroot followup patches

2010-08-16 Thread Charles Wilson
On 8/16/2010 5:07 PM, Ralf Wildenhues wrote: * Charles Wilson wrote on Mon, Aug 16, 2010 at 10:50:21PM CEST: On 8/16/2010 4:42 PM, Ralf Wildenhues wrote: Can you run VERBOSE=yes $srcdir/tests/fcdemo-conf.test; echo $? and post the output of that? VERBOSE=debug is even more informative,

Re: [PATCH v2 0/3] sysroot followup patches

2010-08-16 Thread Charles Wilson
On 8/16/2010 4:51 PM, Peter Rosin wrote: I don't think it's a race, I think it's from the code below: func_verbose creating GNU ld script: $output (approx line 7070 in ltmain.m4sh) where an absolute file name is written to the linker script and therefore escapes the MSYS