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 con

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.

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 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-21 Thread Charles Wilson
On 8/16/2010 4:42 PM, Ralf Wildenhues wrote: > * 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 -

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

[PATCH v2 0/3] sysroot followup patches

2010-08-12 Thread Paolo Bonzini
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 deployment on $host 6) support in libltdl for '=' in .la fi