[Buildroot] [PATCH 1/1] package/imagemagick: utilities need c++

[Buildroot] [PATCH 1/1] package/imagemagick: utilities need c++

[Bernd Kuhls]

See upstream commit:
https://github.com/ImageMagick/ImageMagick/commit/07f3b487f9860fd4eb9422f1a906d0fe83b6fd1c

Fixes:
http://autobuild.buildroot.net/results/29f/29ff3e568ddf40105aec1eded042388df138b7f9/

Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
---
 package/imagemagick/imagemagick.mk | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/package/imagemagick/imagemagick.mk b/package/imagemagick/imagemagick.mk
index 6bdf24c1e8..e40ef56347 100644
--- a/package/imagemagick/imagemagick.mk
+++ b/package/imagemagick/imagemagick.mk
@@ -44,6 +44,10 @@ IMAGEMAGICK_CONF_OPTS = \
 	--without-x \
 	--with-gs-font-dir=/usr/share/fonts/gs
 
+ifeq ($(BR2_INSTALL_LIBSTDCPP),)
+IMAGEMAGICK_CONF_OPTS += --with-utilities=no
+endif
+
 IMAGEMAGICK_DEPENDENCIES = host-pkgconf
 
 ifeq ($(BR2_TOOLCHAIN_USES_UCLIBC),y)
-- 
2.34.1

_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

Re: [Buildroot] [PATCH 1/1] package/imagemagick: utilities need c++

[Thomas Petazzoni via buildroot]

On Tue,  1 Nov 2022 08:21:55 +0100
Bernd Kuhls <bernd.kuhls@t-online.de> wrote:

> See upstream commit:
> https://github.com/ImageMagick/ImageMagick/commit/07f3b487f9860fd4eb9422f1a906d0fe83b6fd1c

What a weird commit. Everything is in C, but they force using a C++
compiler for ... no reason?

> +ifeq ($(BR2_INSTALL_LIBSTDCPP),)
> +IMAGEMAGICK_CONF_OPTS += --with-utilities=no

Could you test if --without-utilities works? If it does, it's
preferable as that's the standard practice in Buildroot.

Thanks!

Thomas
-- 
Thomas Petazzoni, co-owner and CEO, Bootlin
Embedded Linux and Kernel engineering and training
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot