[bitbake-devel] [PATCH 1/6] event: add new property to BuildBase class

[Richard Purdie]

On Mon, 2015-10-05 at 10:43 +0100, Elliot Smith wrote:
> From: Ed Bartosh <ed.bartosh at linux.intel.com>
> 
> Added 'task' property to the base class of BuildStarted and
> BuildCompleted classes to store bitbake task if it's specified.
> 
> This is done as without task Build* events can't fully represent
> bitbake build. Task information is needed by UI classes to properly
> show or process Build* events.

This doesn't make sense I'm afraid. There is no one "task" which a build
start/complete event can correspond to. Consider something like:

"bitbake A:do_fetch B:do_rootfs"

There would be one BuildStarted event but task doesn't have a singular
value. We therefore don't want to add task information to these events
as it will ultimately mislead anything building on top of this
information.

I appreciate there is the -c option, but that just means "if there is no
specific task specified, use this", i.e. its a default.

Cheers,

Richard