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] Factor the sed command used to make a regex from a literal.

2010-08-16 Thread Ralf Wildenhues
* Paolo Bonzini wrote on Sun, Aug 15, 2010 at 07:06:25PM CEST: In func_generate_dlsyms, the resulting regex is used with grep rather than sed. I don't think grep has problems with \/ in practice, but of course I can change it to use sed, too. Oh, I managed to overlook that. In that case,

Re: [PATCH] improve code for sysroot --mode=finish

2010-08-16 Thread Ralf Wildenhues
* Paolo Bonzini wrote on Sun, Aug 15, 2010 at 06:57:18PM CEST: Here are the improvements you suggested. I'll send a patch for sed_make_literal_regex soon. Ok? Yes, with the nit addressed. Thanks! Ralf * libltdl/config/ltmain.m4sh (func_mode_finish): Change sysroot_regex and sysroot_cmd

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