[yocto] [meta-swupd][PATCH] swupd-image.bbclass: do_swupd_update() depends on time-native

André Draszik git at andred.net
Tue Jan 17 08:49:29 PST 2017


On Tue, 2017-01-17 at 17:42 +0100, Patrick Ohly wrote:
> On Tue, 2017-01-17 at 15:59 +0000, André Draszik wrote:
> > The shell script uses time, which is either a bash built-in, or
> > a GNU utility. Not all build machines will have either bash or
> > GNU time available out of the box. Make sure it is available.
> 
> Good catch. I'm just wondering whether it wouldn't be simpler to remove
> the usage of "time" instead, or make it optional?
> 
> The "time" utility was useful while working on performance of the code,

I thought the same, and was contemplating removing its use...

> but now that this is done, it's merely nice to have when one can get it

...but I was curious as well, so I left it in and added the dependency.

> for free, which isn't the case when time-native has to be compiled
> first.

It's a one-time host-depend, which should also be satisfied from sstate
after that, so I thought it'd be OK to keep it.

I have no preference either way.


Cheers,
Andre'




More information about the yocto mailing list