[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