[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] Add --lt-* options to shell wrapper
From: |
Charles Wilson |
Subject: |
Re: [PATCH] Add --lt-* options to shell wrapper |
Date: |
Fri, 15 Jan 2010 20:51:09 -0500 |
User-agent: |
Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.8.1.23) Gecko/20090812 Thunderbird/2.0.0.23 Mnenhy/0.7.6.666 |
Charles Wilson wrote:
> This one, I think is OK for pre-2.2.8 -- what do you guys think?
> OK to push?
In response to review comments over here:
"Re: [PATCH] Enable runtime cwrapper debugging; add tests"
http://lists.gnu.org/archive/html/libtool-patches/2009-12/msg00014.html
I've created a followon patch to this one which allows the cwrapper
tests to pass on platforms which don't use the cwrapper but instaed use
the shell wrapper (e.g. linux).
If this follow-on, and the original patch for this thread, are approved,
then I'll squash these two patches into a single commit and update the
log before pushing.
Full test suite in progress on cygwin (but cwrapper test passes, as do a
number of spot-checked tests in the old test suite). Linux results:
Old: All 94 tests passed
New: Only two unexpected results (in particular, the cwrapper test passed)
21: passing CXX flags through libtool FAILED (flags.at:24)
100: Run tests with low max_cmd_len FAILED
(cmdline_wrap.at:43)
Err.. 21 should have been skipped, because I haven't installed g++ on
the linux box yet. And 100 is just a repeat of 21.
--
Chuck
update-shwrapper.chg
Description: Text document
update-shwrapper.patch
Description: Text Data
- Re: [PATCH] Add --lt-* options to shell wrapper,
Charles Wilson <=