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

2015-09-15 Thread Bystricky, Juro
Thanks, I see what you mean. I'll repost a new patch after I've tested it.

> -Original Message-
> From: Phil Blundell [mailto:p...@pbcl.net]
> Sent: Monday, September 14, 2015 12:53 PM
> To: Khem Raj
> Cc: Bystricky, Juro; Purdie, Richard; openembedded-
> c...@lists.openembedded.org
> Subject: Re: [OE-core] [PATCH] libgcc.inc: package baremetal multilib 
> libraries
> 
> On Mon, 2015-09-14 at 09:38 -0700, Khem Raj wrote:
> > > On Sep 14, 2015, at 9:22 AM, Juro Bystricky <juro.bystri...@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.bystri...@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.
> 

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


[OE-core] [PATCH] libgcc.inc: package baremetal multilib libraries

2015-09-14 Thread Juro Bystricky
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 
---
 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}/* \
+"
+
 FILES_${PN}-dbg += "${base_libdir}/.debug/"
 
 LIBGCCBUILDTREENAME = "gcc-build-internal-"
-- 
1.9.1

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


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

2015-09-14 Thread Khem Raj

> On Sep 14, 2015, at 9:22 AM, Juro Bystricky  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 
> ---
> 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 

> +

> FILES_${PN}-dbg += "${base_libdir}/.debug/"
> 
> LIBGCCBUILDTREENAME = "gcc-build-internal-"
> -- 
> 1.9.1
> 
> -- 
> ___
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


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

2015-09-14 Thread Phil Blundell
On Mon, 2015-09-14 at 09:38 -0700, Khem Raj wrote:
> > On Sep 14, 2015, at 9:22 AM, Juro Bystricky  
> > 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 
> > ---
> > 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.


-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core