diff mbox series

[v3,1/3] package/feh: Fix build issue with some older cross-compilers

Message ID 20171116100459.5801-1-petr.vorel@gmail.com
State Accepted
Headers show
Series [v3,1/3] package/feh: Fix build issue with some older cross-compilers | expand

Commit Message

Petr Vorel Nov. 16, 2017, 10:04 a.m. UTC
by adding -std=gnu99 to CFLAGS.

Fixes:
http://autobuild.buildroot.net/results/392b58cca2671e206c85a530221ce7b2d42f31fb/
http://autobuild.buildroot.net/results/94527fcd70fa85298c366537013c939534beead0/
http://autobuild.buildroot.net/results/bf31199cdaa18971a9b599c26fbefac73b2d6f30/
http://autobuild.buildroot.net/results/5b65c305b50664e02d5f8022278055c80739342d/
http://autobuild.buildroot.net/results/ed3b9fb8d09cd677af4b8825566877c552926ae5/

Signed-off-by: Petr Vorel <petr.vorel@gmail.com>
---
 package/feh/feh.mk | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Thomas Petazzoni Dec. 1, 2017, 9:23 p.m. UTC | #1
Hello,

On Thu, 16 Nov 2017 11:04:57 +0100, Petr Vorel wrote:
> by adding -std=gnu99 to CFLAGS.
> 
> Fixes:
> http://autobuild.buildroot.net/results/392b58cca2671e206c85a530221ce7b2d42f31fb/
> http://autobuild.buildroot.net/results/94527fcd70fa85298c366537013c939534beead0/
> http://autobuild.buildroot.net/results/bf31199cdaa18971a9b599c26fbefac73b2d6f30/
> http://autobuild.buildroot.net/results/5b65c305b50664e02d5f8022278055c80739342d/
> http://autobuild.buildroot.net/results/ed3b9fb8d09cd677af4b8825566877c552926ae5/
> 
> Signed-off-by: Petr Vorel <petr.vorel@gmail.com>
> ---
>  package/feh/feh.mk | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)

Seems like I had forgotten to notify you that PATCH 1/3 had been merged
to master, it was part of the 2017.11 release.

And I've just applied PATCH 2/3 and 3/3 to master, they will be part of
the next release.

Thanks!

Thomas
diff mbox series

Patch

diff --git a/package/feh/feh.mk b/package/feh/feh.mk
index 3f4c410440..7211dc7bdd 100644
--- a/package/feh/feh.mk
+++ b/package/feh/feh.mk
@@ -12,8 +12,8 @@  FEH_LICENSE = MIT
 FEH_LICENSE_FILES = COPYING
 
 define FEH_BUILD_CMDS
-	$(TARGET_MAKE_ENV) $(MAKE1) CC="$(TARGET_CC) $(TARGET_CFLAGS) $(TARGET_LDFLAGS)" \
-		-C $(@D) all
+	$(TARGET_MAKE_ENV) CFLAGS="$(TARGET_CFLAGS) -std=gnu99" $(MAKE1) \
+		CC="$(TARGET_CC) $(TARGET_LDFLAGS)" -C $(@D) all
 endef
 
 define FEH_INSTALL_TARGET_CMDS