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

[Bob Friesenhahn]

On Mon, 14 Dec 2009, Charles Wilson wrote:
> Two related lines of inquiry:
> 1) Under *normal* development rules -- e.g not a pre-release
> bug-fix-only phase, nor a not-quite-pre-release code slush like (I
> think) we're in right now, for 2.2.8 -- surely you aren't suggesting
> that EVERY contribution must be validated on EVERY platform, prior to
> push?  These were tested on cyg/ming and linux, so in general, during
> /normal/ development, that should be sufficient contra reveiwer
> comments, right?

The bar seems to be raised quite high now in libtool development so 
that only those who practice the art regularly are able to satisfy all 
the requirements.

It seems to me that if the patch works on several platforms, and there 
is no reason to believe that any issues with other platforms can't be 
reasonably fixed, that the patch should be accepted.  100% coverage on 
all platforms is not a reasonable requirement for a patch.

Bob
--
Bob Friesenhahn
address@hidden, http://www.simplesystems.org/users/bfriesen/
GraphicsMagick Maintainer,    http://www.GraphicsMagick.org/