[OE-core] [PATCH 2/5] gmp: also generate the libgmpcxx library

[Kamble, Nitin A]

> -----Original Message-----
> From: openembedded-core-bounces at lists.openembedded.org
> [mailto:openembedded-core-bounces at lists.openembedded.org] On Behalf Of
> Khem Raj
> Sent: Tuesday, October 18, 2011 1:54 PM
> To: Patches and discussions about the oe-core layer
> Subject: Re: [OE-core] [PATCH 2/5] gmp: also generate the libgmpcxx
> library
> 
> On (18/10/11 13:14), nitin.a.kamble at intel.com wrote:
> > From: Nitin A Kamble <nitin.a.kamble at intel.com>
> >
> > Signed-off-by: Nitin A Kamble <nitin.a.kamble at intel.com>
> > ---
> >  meta/recipes-support/gmp/gmp.inc      |    2 ++
> >  meta/recipes-support/gmp/gmp_4.2.1.bb |    2 +-
> >  meta/recipes-support/gmp/gmp_5.0.2.bb |    2 +-
> >  3 files changed, 4 insertions(+), 2 deletions(-)
> >
> > diff --git a/meta/recipes-support/gmp/gmp.inc b/meta/recipes-
> support/gmp/gmp.inc
> > index 66349e6..77ce10d 100644
> > --- a/meta/recipes-support/gmp/gmp.inc
> > +++ b/meta/recipes-support/gmp/gmp.inc
> > @@ -14,3 +14,5 @@ ARM_INSTRUCTION_SET = "arm"
> >  acpaths = ""
> >
> >  BBCLASSEXTEND = "native nativesdk"
> > +
> > +EXTRA_OECONF += " --enable-cxx=yes"
> 
> if you want 'yes' then just --enable-cxx is enough have you tried with
> --enable-cxx=detect that would make sure that C++ compiler and runtime
> is ok. Secondly is this option needed for native recipes too ?

Thanks Khem for your comments. This option is also needed for native recipe to avoid having gmp-devel installed on the build system. "--enable-cxx=detect" can be used, but I am not sure if it is the best way yet, I think we always have c++ as part of gcc, not having c++ is not a case to worry. And if "detect" tests if c++ & runtime are not working as expected, then it should not disable the enable-cxx option with that reason; better would be to get a recipe build failure. So I still favor keeping the option as "--enable-cxx" until I can see the merit of "detect".

Thanks,
Nitin

> 
> > diff --git a/meta/recipes-support/gmp/gmp_4.2.1.bb b/meta/recipes-
> support/gmp/gmp_4.2.1.bb
> > index 74da6b8..97ac4b2 100644
> > --- a/meta/recipes-support/gmp/gmp_4.2.1.bb
> > +++ b/meta/recipes-support/gmp/gmp_4.2.1.bb
> > @@ -6,7 +6,7 @@ LICENSE = "LGPLv2.1+"
> >  LIC_FILES_CHKSUM =
> "file://COPYING;md5=892f569a555ba9c07a568a7c0c4fa63a \
> >
> file://COPYING.LIB;md5=fbc093901857fcd118f065f900982c24 \
> >                      file://gmp-
> h.in;startline=6;endline=21;md5=5e25ffd16996faba8c1cd27b04b16099"
> > -PR = "r0"
> > +PR = "r1"
> >
> >  SRC_URI = "${GNU_MIRROR}/gmp/${BP}.tar.bz2 \
> >             file://disable-stdc.patch"
> > diff --git a/meta/recipes-support/gmp/gmp_5.0.2.bb b/meta/recipes-
> support/gmp/gmp_5.0.2.bb
> > index 03fef45..f80971e 100644
> > --- a/meta/recipes-support/gmp/gmp_5.0.2.bb
> > +++ b/meta/recipes-support/gmp/gmp_5.0.2.bb
> > @@ -2,7 +2,7 @@ require gmp.inc
> >  LICENSE="LGPLv3&GPLv3"
> >  LIC_FILES_CHKSUM =
> "file://COPYING;md5=d32239bcb673463ab874e80d47fae504 \
> >
> file://version.c;endline=18;md5=d8c56b52b9092346b9f93b4da65ef790"
> > -PR = "r0"
> > +PR = "r1"
> >
> >  SRC_URI_append += "file://sh4-asmfix.patch \
> >                     file://use-includedir.patch "
> > --
> > 1.7.4.4
> >
> >
> > _______________________________________________
> > Openembedded-core mailing list
> > Openembedded-core at lists.openembedded.org
> > http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
> 
> --
> -Khem
> 
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core at lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core