svn commit: r185499 - head

[Alfred Perlstein]

* 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?

thanks,
-Alfred