Re: [PATCH] Enable runtime cwrapper debugging; add tests

[Bob Friesenhahn]

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/