|
From: | Bob Friesenhahn |
Subject: | Re: [PATCH] Enable runtime cwrapper debugging; add tests |
Date: | Sun, 14 Feb 2010 09:55:37 -0600 (CST) |
User-agent: | Alpine 2.01 (GSO 1266 2009-07-14) |
Charles,Within the bounds of technical recommendations/comments made already (by Ralf and perhaps others), I recommend that you commit your log-jammed Cygwin patches according to the 72 hour rule. Otherwise it is unlikely that there will be any forward progress. If anything becomes broken for some other platform, it is certain to be fixed.
Development libtool has not been released for quite a long time now (since September 2008) and will need to undergo regression testing on a wide variety of platforms prior to release.
Bob On Sun, 14 Feb 2010, Charles Wilson wrote:
Charles Wilson wrote:Charles Wilson wrote:Charles Wilson wrote:Ralf Wildenhues wrote:* Charles Wilson wrote on Mon, Jun 22, 2009 at 03:50:42AM CEST:* libltdl/config/ltmain.m4sh (func_emit_cwrapperexe_src) [ltwrapper_debugprintf]: Renamed to...I think functions should still be put in (parens) in the ChangeLog entry, not in [brackets], according to GCS.[... other review comments ...] Okay, here's a followon patch to "Enable runtime cwrapper debugging; add tests" http://lists.gnu.org/archive/html/libtool-patches/2009-12/msg00014.html The first patch, in addition to addressing various points raised in this thread, obsoletes the "extra" mingw fix: http://lists.gnu.org/archive/html/libtool-patches/2009-12/msg00017.html Assuming these two patches are approved, I'll squash them together into a single commit (and update the commit history) before pushing. I just figured it was easier to review, by presenting the response to the review comments separately.ping...ping, again. -- Chuck
-- Bob Friesenhahn address@hidden, http://www.simplesystems.org/users/bfriesen/ GraphicsMagick Maintainer, http://www.GraphicsMagick.org/
[Prev in Thread] | Current Thread | [Next in Thread] |