[Koha-patches] [PATCH] CORRECTION: fixes print statements with missing 'STDERR',

Joe Atzberger joe.atzberger at liblime.com
Mon Mar 9 15:02:24 CET 2009


On Sun, Mar 8, 2009 at 7:31 AM, Mason James <mason.loves.sushi at gmail.com>wrote:

> err, the 'warn sprintf()'s still actually output to STDOUT :/
> this patch fixes that.
>

Um, no they don't.  "$debug and printf ..." would go to STDOUT, but warns
would still go to STDERR.  To demonstrate:

perl -e 'warn sprintf "foo"' >/dev/null

The lines you changed to warns probably should also have the trailing "\n"
removed since warn takes care of that itself.  I think I agree with the
intention of the patch, but the description and execution have issues.

--joe
-------------- next part --------------
An HTML attachment was scrubbed...
URL: </pipermail/koha-patches/attachments/20090309/8283086f/attachment-0002.htm>


More information about the Koha-patches mailing list