Message ID | 1567690842-33954-1-git-send-email-gwenj@trabucayre.com |
---|---|
State | Accepted |
Headers | show |
Series | package:gmp: enable C++ support when BR2_INSTALL_LIBSTDCPP=y | expand |
Hello, On Thu, 5 Sep 2019 15:40:40 +0200 Gwenhael Goavec-Merou <gwenj@trabucayre.com> wrote: > From: Gwenhael Goavec-Merou <gwenhael.goavec-merou@trabucayre.com> > > gmp has a optional and disable by default, option to add C++ support. > > Signed-off-by: Gwenhael Goavec-Merou <gwenhael.goavec-merou@trabucayre.com> The correct prefix in the commit title is "package/gmp:", not "package:gmp:". > +ifeq ($(BR2_INSTALL_LIBSTDCPP),y) > +GMP_CONF_OPTS += --enable-cxx > +endif We like to have an explicit --disable-cxx in an else clause. I fixed both issues and applied. Thanks! Thomas
diff --git a/package/gmp/gmp.mk b/package/gmp/gmp.mk index 7236e37bb4..3e375a12e7 100644 --- a/package/gmp/gmp.mk +++ b/package/gmp/gmp.mk @@ -19,5 +19,9 @@ ifeq ($(BR2_m68k_cf)$(BR2_MIPS_CPU_MIPS32R6)$(BR2_MIPS_CPU_MIPS64R6)$(BR2_ARM_CP GMP_CONF_OPTS += --disable-assembly endif +ifeq ($(BR2_INSTALL_LIBSTDCPP),y) +GMP_CONF_OPTS += --enable-cxx +endif + $(eval $(autotools-package)) $(eval $(host-autotools-package))