[Buildroot] [PATCH RFC] core: enable per-package log files

[Arnout Vandecappelle]

On 18-10-17 11:10, Thomas Petazzoni wrote:
> Hello,
> 
> On Wed, 18 Oct 2017 11:05:10 +0200, Anisse Astier wrote:
> 
>> But as I said, there should be a top logfile by default, removing the
>> need for tee.
> 
> I'm not sure how this solves the problem. Indeed, even if we have a
> logfile by default, it will be done using "| tee" so that stdout has
> all the messages, and the logfile as well.
> 
> But I am not sure we want to log everything by default. I find it
> somewhat neat that Buildroot doesn't enforce a logging strategy. Some
> users prefer to redirect everything, some users prefer a "| tee" thing.
> Some users prefer the log to be appended after the existing log, some
> users prefer to truncate and replace the existing log.
> 
> So, if we're going the -Orecurse route, I'd prefer to not change our
> logging strategy, i.e not log anything by default, and leave it up to
> the user.

 I agree here. Leave as much as possible to the user. It is for this very reason
that I'd prefer the -Orecurse to be specified by the user and not enforced in
our Makefile. Users who always want this behaviour can make an alias, or export
MAKEFLAGS="-Orecurse -j5", or use brmake.

 It is nice if the more interesting use cases can be supported in utils/brmake,
so users don't have to reinvent their logging strategy all the time.

 Regardless, I consider the logging one of the least priority things in the
whole top-level parallel build thing.

 Regards,
 Arnout

-- 
Arnout Vandecappelle                          arnout at mind be
Senior Embedded Software Architect            +32-16-286500
Essensium/Mind                                http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium           BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint:  7493 020B C7E3 8618 8DEC 222C 82EB F404 F9AC 0DDF