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 M

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

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. T

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 > /c/cygwin-1.7/usr/src/packages/libtool/git/build-libtool-sysroot-mingw-native/tests/testsuite.dir/112/test

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 fc

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 stripp

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 u

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.

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) > >

Re: [PATCH v2 4/4] add libtool --mode=finish mode for sysroot

2010-08-16 Thread Ralf Wildenhues
* Paolo Bonzini wrote on Sun, Aug 15, 2010 at 03:46:06PM CEST: > On 08/13/2010 12:59 AM, Ralf Wildenhues wrote: > >Erm, you should be mentioning the files you change in your log entries, > >and Libtool also doesn't auto-generate its ChangeLog (yet), so you need > >that, too, also for all the other

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

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, s

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 de