svn commit: r185499 - head

[M. Warner Losh]

In message: <20081201132554.GD27096 at elvis.mu.org>
            Alfred Perlstein <alfred at FreeBSD.org> writes:
: * Dag-Erling Sm??rgrav <des at des.no> [081201 03:25] wrote:
: > Tim Kientzle <kientzle at freebsd.org> writes:
: > > Alfred Perlstein <alfred at freebsd.org> writes:
: > > > Log:
: > > >   Provide a 'tinderbox' target that compiles enough of FreeBSD that
: > > >   a developer can rest reasonably assured that the tinderbox will not
: > > >   be broken.  This target leverages most of 'universe' but will exit
: > > >   non-zero and output a summary at the end.
: > > >     "make tinderbox"
: > > Excellent!
: > 
: > I really don't see the point, especially now that 'make universe' will
: > issue a warning on stdout for every build that fails.  As I've tried to
: > explain to Alfred, the tinderbox does nothing magical, and nothing that
: > isn't already covered by 'make universe'.  I also asked him to discuss
: > any changes with Poul-Henning before committing them.
: 
: I'll summarize the changes here:
: 
: Problems with make universe:
:   1) Make universe returns success on failure.
:   2) Make universe outputs the following on failure:
:     --------------------------------------------------------------
:     make universe completed on `LC_ALL=C date`"
:                        (started ${STARTTIME})"
:     --------------------------------------------------------------
: 
: This basically means you can't chain 'make universe' into something
: like "make tinderbox || (echo oops | mail -s tinderbox failed...)".
: 
: The summary at the end "completed" is very misleading.
: 
: The tinderbox target differs as follows:
: 
:   It returns an error if it fails.
:   It outputs a summary of the failures at the end.
:   The name reflects what its for. ("what the heck is a universe?")
: 
: 
: If you strongly believe that a target should return "true" on failure
: and/or output "i worked" on failure then I'm not sure how to approach
: that mindset, maybe you can help me understand where you're coming from
: on that?  Where is the utility in it?  Is there something I'm missing?

I think you ignored the discussions that said that 'universe' should
be modified to exit with an error when there was an error, but
shouldn't bail out EARLY.  There were even patches to do this.

Based on that, why do something completely new and different?

Warner