Re: mdemo ltdl failure

2007-04-19 Thread Ralf Wildenhues
Hi Charles, * Charles Wilson wrote on Thu, Apr 19, 2007 at 03:02:08AM CEST: This is because the test is just too ugly for words, not to mention brittle. Trying to tease out malloc issues outside of a dedicated malloc testsuite is just plain silly. I think the biggest problem with the

Re: [cygwin, libtool] use shell function to emit wrapper scripts and wrapper.exe source

2007-04-19 Thread Ralf Wildenhues
* Charles Wilson wrote on Thu, Apr 19, 2007 at 06:13:19AM CEST: Test results -- new tests. Unexpected failures: 14: Java convenience archives FAILED (convenience.at:273) 16: Link order of deplibs. FAILED (link-order2.at:129) 49: Run tests with low max_cmd_len FAILED

Re: [cygwin, libtool] use shell function to emit wrapper scripts and wrapper.exe source

2007-04-19 Thread Ralf Wildenhues
* Charles Wilson wrote on Thu, Apr 19, 2007 at 03:55:20AM CEST: [Added libtool-patches to CC list. Discussion of this patch should probably drop libtool and cygwin] Done. Okay, here's the first bit. It's pretty simple. Testing is in progress (and in conjuction with the new argz fix I just

Re: mdemo ltdl failure

2007-04-19 Thread Charles Wilson
Ralf Wildenhues wrote: * Charles Wilson wrote: This is because the test is just too ugly for words, not to mention brittle. Trying to tease out malloc issues outside of a dedicated malloc testsuite is just plain silly. I think the biggest problem with the previous patch was that it was

Re: [cygwin, libtool] use shell function to emit wrapper scripts and wrapper.exe source

2007-04-19 Thread Charles Wilson
Ralf Wildenhues wrote: * Charles Wilson wrote: Test results -- new tests. Unexpected failures: 14: Java convenience archives FAILED (convenience.at:273) 16: Link order of deplibs. FAILED (link-order2.at:129) 49: Run tests with low max_cmd_len FAILED (cmdline_wrap.at:42) are

Re: mdemo ltdl failure

2007-04-19 Thread Ralf Wildenhues
* Charles Wilson wrote on Thu, Apr 19, 2007 at 08:08:41PM CEST: Ralf Wildenhues wrote: It's a good choice of testing, and should be done again with the final patch. Plus one test on Solaris with its /bin/sh. (Just noting this, I can probably do these tests then.) will do for linux

Re: mdemo ltdl failure

2007-04-19 Thread Ralf Wildenhues
* Charles Wilson wrote on Thu, Apr 19, 2007 at 08:08:41PM CEST: --- libltdl/m4/argz.m4 25 Mar 2006 11:05:02 - 1.3 +++ libltdl/m4/argz.m4 17 Mar 2007 06:09:50 - [...] +os_ver=$(uname -r | $SED -e 's,^\([[0123456789\.]]*\).*,\1,') +

Re: [cygwin, libtool] use shell function to emit wrapper scripts and wrapper.exe source

2007-04-19 Thread Charles Wilson
Ralf Wildenhues wrote: Thanks. Please resubmit with the new functions moved to right before func_mode_link (or func_mode_install/func_mode_execute, in case you intend to use them from inside them in a followup patch); there's a real performance benefit to doing this for compile mode, as it

Re: mdemo ltdl failure

2007-04-19 Thread Charles Wilson
Hopefully the attached patch addresses all comments...Recapping: The argz functions (specifically, argz_insert) supplied by cygwin are buggy, in wierd use-dependent malloc-related ways. I've already submitted a patch to newlib to fix that error which has been accepted

Re: mdemo ltdl failure

2007-04-19 Thread Charles Wilson
Charles Wilson wrote: Under case (1), currently running the new-style testsuite. Will report that later in a follow-up message. I expect the following: 14: Java convenience archives FAILED (convenience.at:273) 16: Link order of deplibs. FAILED (link-order2.at:129) 49: Run