Re: Multiple test failures with --disable-shared

2010-06-14 Thread Ralf Wildenhues
Hi Peter, * Peter O'Gorman wrote on Mon, Jun 14, 2010 at 05:18:55AM CEST: On 06/13/2010 12:07 AM, Ralf Wildenhues wrote: * Peter O'Gorman wrote on Sun, Jun 13, 2010 at 06:07:44AM CEST: [ the resident test ] I'm not sure what testing that modules cannot be unloaded gets you when you don't have

MSVC: Find potential libs regardless of case.

2010-06-14 Thread Peter Rosin
Hi! Here's the next patch on the MSVC branch. I decided to merge in a couple of things further ahead, no point in reviewing stuff when an improvement is around the corner, right? What I'm looking for is a go-ahead that this patch is ready for master, but then delay pushing it until enough of

Re: Multiple test failures with --disable-shared

2010-06-14 Thread Peter O'Gorman
On 06/14/2010 12:58 AM, Ralf Wildenhues wrote: IIRC this one could also be fixed by using LTDL_SET_PRELOADED_SYMBOLS, I don't remember for sure, but I recall being convinced that these were all testsuite issues. Well, if that fixes it, then that is fine with me of course. Ok thanks, pushed

Re: Enable colored Autotest testsuite output if available.

2010-06-14 Thread Eric Blake
On 06/13/2010 05:17 AM, Ralf Wildenhues wrote: OK to commit this patch if this one is accepted into Autoconf? http://lists.gnu.org/archive/html/autoconf-patches/2010-06/msg00031.html The move the testsuite.at code above AT_INIT should have no impact beside reordering things in the

Re: MSVC: Find potential libs regardless of case.

2010-06-14 Thread Ralf Wildenhues
Hi Peter, * Peter Rosin wrote on Mon, Jun 14, 2010 at 01:26:04PM CEST: Here's the next patch on the MSVC branch. I decided to merge in a couple of things further ahead, no point in reviewing stuff when an improvement is around the corner, right? this patch is ok for master, provided that it

Re: MSVC: Find potential libs regardless of case.

2010-06-14 Thread Eric Blake
On 06/14/2010 03:01 PM, Ralf Wildenhues wrote: +eval `$EGREP '^(old_library)=' $libdir/libbar.la` This is probably lacking double-quoting around the `...` text. Can you verify? For that matter, it's not safe in the presence of signals, since some shells end up doing eval '' if the `` is

Re: [libtool 2.2.11a] testsuite: 48 69 92 failed [cygwin]

2010-06-14 Thread Ralf Wildenhues
* Charles Wilson wrote on Sun, Jun 13, 2010 at 08:51:00PM CEST: On 6/12/2010 4:58 AM, Ralf Wildenhues wrote: * Charles Wilson wrote on Fri, Jun 11, 2010 at 02:28:41PM CEST: In 48, the problem occurs during libtool --clean: /usr/src/packages/libtool/git/build/libtool: line 1693:

Re: [libtool 2.2.11a] testsuite: 48 69 92 failed [cygwin]

2010-06-14 Thread Charles Wilson
On 6/14/2010 11:26 PM, Ralf Wildenhues wrote: Thanks. $objdir is a global variable set in the initial section of the libtool script, and temporarily overriding it in func_mode_uninstall is the wrong thing to do. It looked fishy to me, but I assumed it was put there for a reason...but I