Re: [PATCH 1/2] imx-base.inc: remove bootloader from image_wks depends

[Max Krummenacher <max.oss.09@gmail.com>]

Hi Fabio

Am Donnerstag, den 19.07.2018, 17:46 -0300 schrieb Fabio Berton:
> Hi Max!
> 
> The virtual/bootloader it's in WKS_FILE_DEPENDS because the default
> WKS_FILE is imx-uboot-bootpart.wks, this wks uses u-boot.imx file to
> create final image.

My bad. I thought that in my use case U-Boot is also installed and
that it got build and deployed through some other magical dependency
and I failed to check its existence after removing virtual/bootloader.

> 
> I saw this "File exists ibfdt.h" error and I'll try to figure out
> other solution, because e.g. imx6ullevk need to use dtc-145 and need
> to has virtual/bootloader.
> 
> For machines the use sdimage-bootpart.wks and dtc-145, like
> apalis-imx6, we can add WKS_FILE_DEPENDS = "" in machine config file.

Eventually for one image type or another these machines also like
to have U-Boot built and deployed, so that would only be a shortterm
solution.

I looked a bit into this and found two possible ways out:

- change the dtc-145 recipe so that only a renamed dtc binary gets
installed, none of the libraries, none of the headers. Thus in the
image_wic tasks there is no clash between files.
Then the U-Boot Makefile must be patched to use the renamed dtc
binary.

- Drop the dtc-145 recipe and build the dtc as part of the U-Boot
build. That way the dtc 1.4.5. is contained in the U-Boot work area
and never gets deployed. Seems a bit hackish to me though.

I could prepare a proper patch for one or the other.

What you think?

Regards
Max

> 
> Thanks!
> 
> Fabio Berton
> 
> On Mon, Jul 16, 2018 at 9:31 AM, Max Krummenacher <max.oss.09@gmail.com> wrote:
> > With virtual/bootloader in WKS_FILE_DEPENDS the recipe-sysroot(-native) gets
> > populated by whatever virtual/bootloader and its DEPENDS put in the
> > recipe-sysroot.
> > 
> > For U-Boot recipes which work around the fdt.h et. al. include mess by using
> > the older dtc-145-native this results in a error during do_image_wic on first
> > build. A second run of bitbake succeeds, but has some releated warning in
> > the log.do_image_wks logfile.
> > 
> > > Exception: FileExistsError: [Errno 17] File exists: '...tmp/sysroots-components/x86_64/dtc-
> > > 145-native/usr/include/libfdt.h' -> '.../tmp/work/apalis_imx6-fslc-linux-gnueabi/core-image-
> > > minimal/1.0-r0/recipe-sysroot-native/usr/include/libfdt.h'
> > > ERROR: core-image-minimal-1.0-r0 do_image_wic: Function failed: extend_recipe_sysroot
> > > $ find tmp/sysroots-components/ -name libfdt.h
> > > tmp/sysroots-components/x86_64/dtc-145-native/usr/include/libfdt.h
> > > tmp/sysroots-components/x86_64/dtc-native/usr/include/libfdt.h
> > 
> > Signed-off-by: Max Krummenacher <max.krummenacher@toradex.com>
> > ---
> >  conf/machine/include/imx-base.inc | 2 --
> >  1 file changed, 2 deletions(-)
> > 
> > diff --git a/conf/machine/include/imx-base.inc b/conf/machine/include/imx-base.inc
> > index bd2a2aa3..24058c45 100644
> > --- a/conf/machine/include/imx-base.inc
> > +++ b/conf/machine/include/imx-base.inc
> > @@ -285,8 +285,6 @@ IMAGE_BOOT_FILES ?= " \
> > 
> >  ### wic default support
> >  WKS_FILE_DEPENDS ?= " \
> > -    virtual/bootloader \
> > -    \
> >      e2fsprogs-native \
> >      bmap-tools-native \
> >  "
> > --
> > 2.13.6
> > 
> > --
> > _______________________________________________
> > meta-freescale mailing list
> > meta-freescale@yoctoproject.org
> > https://lists.yoctoproject.org/listinfo/meta-freescale