Re: [PATCH] libgcc.inc: package baremetal multilib libraries

[Phil Blundell <pb@pbcl.net>]

On Mon, 2015-09-14 at 09:38 -0700, Khem Raj wrote:
> > On Sep 14, 2015, at 9:22 AM, Juro Bystricky <juro.bystricky@intel.com> wrote:
> > 
> > When building libgcc for baremetal cross-compilers, some files and libraries
> > may be built but not packaged. This patch fixes errors such as these:
> > 
> > ERROR: QA Issue: libgcc: Files/directories were installed but not shipped in any package:
> >  /usr/lib/arm-poky-eabi/4.9.3/thumb
> >  /usr/lib/arm-poky-eabi/4.9.3/fpu
> >  /usr/lib/arm-poky-eabi/4.9.3/thumb/libgcov.a
> >  /usr/lib/arm-poky-eabi/4.9.3/thumb/crtn.o
> >  /usr/lib/arm-poky-eabi/4.9.3/thumb/crtend.o
> >  /usr/lib/arm-poky-eabi/4.9.3/thumb/crtbegin.o
> >  /usr/lib/arm-poky-eabi/4.9.3/thumb/libgcc.a
> >  /usr/lib/arm-poky-eabi/4.9.3/thumb/crti.o
> >  /usr/lib/arm-poky-eabi/4.9.3/fpu/libgcov.a
> >  /usr/lib/arm-poky-eabi/4.9.3/fpu/crtn.o
> >  /usr/lib/arm-poky-eabi/4.9.3/fpu/crtend.o
> >  /usr/lib/arm-poky-eabi/4.9.3/fpu/crtbegin.o
> >  /usr/lib/arm-poky-eabi/4.9.3/fpu/libgcc.a
> >  /usr/lib/arm-poky-eabi/4.9.3/fpu/crti.o
> > Please set FILES such that these items are packaged. Alternatively if they are unneeded,
> > avoid installing them or delete them within do_install. [installed-vs-shipped]
> > 
> > Signed-off-by: Juro Bystricky <juro.bystricky@intel.com>
> > ---
> > meta/recipes-devtools/gcc/libgcc.inc | 6 ++++++
> > 1 file changed, 6 insertions(+)
> > 
> > diff --git a/meta/recipes-devtools/gcc/libgcc.inc b/meta/recipes-devtools/gcc/libgcc.inc
> > index 739adbd..95709e4 100644
> > --- a/meta/recipes-devtools/gcc/libgcc.inc
> > +++ b/meta/recipes-devtools/gcc/libgcc.inc
> > @@ -28,6 +28,12 @@ FILES_${PN}-dev = "\
> >     ${libdir}/${TARGET_SYS}/${BINV}/libgcov.a \
> > "
> > 
> > +FILES_${PN}-dev_libc-baremetal = "\
> > +    ${base_libdir}/ \
> > +    ${libdir}/* \
> > +    ${libdir}/${TARGET_SYS}/* \
> > +”
> 
> why not put it into libgcc-dev itself 

Agreed.  I think the real issue here is nothing really to do with
baremetal, it's just that the existing FILES_${PN}-dev pattern doesn't
consider multilib for anything other than IA and mips.

p.