[meta-virtualization] [PATCH 1/4] vgabios : extract biossums tool and build it separately, native

[Bruce Ashfield]

On Wed, Aug 23, 2017 at 5:13 PM, Christopher Clark <
christopher.w.clark at gmail.com> wrote:

> On Wed, Aug 23, 2017 at 1:18 PM, Christopher Clark
> <christopher.w.clark at gmail.com> wrote:
> > On Wed, Aug 23, 2017 at 5:54 AM, Bruce Ashfield
> > <bruce.ashfield at gmail.com> wrote:
> >>
> >>
> >> On Wed, Aug 23, 2017 at 2:43 AM, Christopher Clark
> >> <christopher.w.clark at gmail.com> wrote:
> >>>
> >>> On Tue, Aug 22, 2017 at 7:42 PM, Bruce Ashfield
> >>> <bruce.ashfield at gmail.com> wrote:
> >>> > On Tue, Aug 22, 2017 at 1:08 AM, <christopher.w.clark at gmail.com>
> wrote:
> >>> >> From: Christopher Clark <christopher.clark6 at baesystems.com>
> >>> >>
> >>> >> The biossums tool is used to generate the vgabios build product.
> >>> >>
> >>> >> This patch:
> >>> >> * adds a separate recipe "biossums" for the tool
> >>> >> * makes the vgabios recipe depend upon biossums-native
> >>> >> * makes the vgabios recipe use the native binary rather than a
> locally
> >>> >> built one
> >
> >>> >> +
> >>> >> +do_compile() {
> >>> >> +    oe_runmake clean
> >>> >
> >>> >
> >>> > What actually happens if the clean isn't run before each compile ?
>
> I figured it out: a 32-bit biossums binary is distributed inside the
> upstream source tarball, and make decides that it doesn't need to
> rebuild it since the timestamp is newer than the source code file.
>

Aha. Thanks for the information .. gives more confidence in the approach.
We could
also just rm the files, but using the clean mechanism to take care of it
properly makes
sense.

I've now merged the 4 patch series.

Bruce


>
> I think that makes the "clean" seem like a reasonable step to put
> before compile.
>
> Christopher
>



-- 
"Thou shalt not follow the NULL pointer, for chaos and madness await thee
at its end"
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.yoctoproject.org/pipermail/meta-virtualization/attachments/20170824/1a8e898d/attachment.html>