Subject: Re: CVS commit: pkgsrc/mk/wrapper
To: Joerg Sonnenberger <joerg@britannica.bec.de>
From: Roland Illig <rillig@NetBSD.org>
List: pkgsrc-changes
Date: 12/17/2006 19:17:57
Joerg Sonnenberger wrote:
> On Sat, Dec 09, 2006 at 12:11:28PM +0000, Roland Illig wrote:
> 
>>Module Name:	pkgsrc
>>Committed By:	rillig
>>Date:		Sat Dec  9 12:11:28 UTC 2006
>>
>>Modified Files:
>>	pkgsrc/mk/wrapper: wrapper.sh
>>
>>Log Message:
>>Since the wrapper transformations are usually quite hard to follow and
>>may lead to failure for some packages, a note to the user is printed
>>when a wrapped command fails, including the "real" command line of the
>>wrapped command.
>>
>>This change arose from the implicit addition of the -std=c99, -std=gnu99
>>or -c99 options, which made some packages fail with obscure error
>>messages. This change helps users get to the real cause more quickly.
> 
> 
> Are you sure that this is not breaking scripts which parse the output of
> e.g. cc?

This change only applies in cases where the compiler is failing anyway, 
so I don't think it will break much things. The error message is written 
to stderr, so it does not interfere with output from "cc -E", for example.

> I also don't like the way this pollutes e.g. the already
> obscure output of configure. Can we direct this to a file please?

No. :) The sole intention of this change was to make this message visible.

Roland