Subject: Re: CVS commit: src/share/mk
To: John Hawkinson <jhawk@mit.edu>
From: Luke Mewburn <lukem@NetBSD.org>
List: source-changes
Date: 10/28/2003 11:33:20
On Sat, Oct 18, 2003 at 08:58:42PM +0000, John Hawkinson wrote:
| > Implement MAKEVERBOSE to control how "verbose" the standard make(1) rules are.
| > Supported values:
| > 0 Minimal output ("quiet")
| > 1 Describe what is occurring
| > 2 Describe what is occurring and echo the actual command ("verbose")
| > This is the default, and is equivalent to the previous behaviour.
|
| I'm pretty disappointed you didn't see fit to address the concerns
| I raised about the sense of this being backwards. Any particular
| you reason you did not reply before your implementation? I would
| have appreciated knowing how you were going to implement this before
| you did it.
FWIW: the general feedback I have received about this work has been
positive.
You were asking for what I consider to be a minor semantic difference
in how this is controlled (MAKEVERBOSE versus your MAKEBRIEF), and
one that I didn't feel had enough justification to change from I'd
already done, especially since no-one else expressed any similar
sentiments to yours.
In any case, the method the infrastructure is controlled is based on
one set of checks of MAKEVERBOSE in <bsd.own.mk>; if it turns out in
our experience using the mechanism shows us that your MAKEBRIEF
semantics are better/make more sense, it's trivial to change.
Having the MAKEVERBOSE stuff available to more than just the "build.sh"
infrastructure is useful as well; this came up in my discussions with
a few people about this stuff before it was committed.