imagemagick: refresh GCC options in Makefile
authorTony Butler <spudz76@gmail.com>
Thu, 22 Dec 2022 23:24:45 +0000 (15:24 -0800)
committerRosen Penev <rosenp@gmail.com>
Fri, 23 Dec 2022 02:11:51 +0000 (18:11 -0800)
this Makefile still used `CONFIG_GCC_USE_VERSION_*` to select various
compilation options, for GCC versions that are antiquated

convert to parsing the major from the `CONFIG_GCC_VERSION` which will
always exist and can also be used with range logic

intent seemed to be:
* `-flto` for "not =10" (or newer, probably)
* no additional options for "=10" (and newer, probably)

GCC 11 or 12 would likely revert to the default (not =10) option,
because 10 was the newest at the time, and 11 and 12 are "not 10"

unsure of what actually works, perhaps `-flto` works in all versions by
now (possibly early gcc 10 bug workaround?)

GCC 11 will have been using `-flto` anyway by the current logic and I
guess it must be working or there would have been changes

Signed-off-by: Tony Butler <spudz76@gmail.com>
multimedia/imagemagick/Makefile

index 53f69720eb43100aff17517c7bc628994c89ba21..8c4e9777559e9d8880a41acec0eae7ddbd6e5ca6 100644 (file)
@@ -110,7 +110,7 @@ CONFIGURE_ARGS += \
        --with-png \
        --with-tiff
 
-TARGET_CFLAGS += $(if $(CONFIG_GCC_USE_VERSION_10),,-flto)
+TARGET_CFLAGS += $(if $(filter-out 5 6 7 8 9,$(word 1,$(subst ., ,$(call qstrip,$(CONFIG_GCC_VERSION))))),,-flto)
 
 define Build/InstallDev
        $(INSTALL_DIR) $(1)/usr/include