Skip to contentSkip to conversation

vtun: fix gcc5 build

PATCH New Baruch Siach 2 participants 2 messages Oct 1, 2015 – Oct 2, 2015

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 vtun: fix gcc5 build

Add -std=gnu89 to CFLAGS to keep pre gcc5 inline function symbols visibility rules.

Fixes:
http://autobuild.buildroot.net/results/a7e/a7e684174b90a852318a7a8bf41c4f477d159d56/
http://autobuild.buildroot.net/results/ab0/ab01f8e0467e85322c63b81bb7e84fd251e5d556/
http://autobuild.buildroot.net/results/117/117a68bf648b050dabed8088d1165c1d7d9301f0/

and more.

---
 package/vtun/vtun.mk | 2 ++
 1 file changed, 2 insertions(+)
package/vtun/vtun.mk +2 -0
index 2b64ab9efc69..884ebc666431 100644
@@ -14,4 +14,6 @@ VTUN_CONF_OPTS = \
 	--with-lzo-headers=$(STAGING_DIR)/usr/include/lzo \
 	--with-lzo-lib=$(STAGING_DIR)/usr/lib
 
+VTUN_CONF_ENV = CFLAGS="$(TARGET_CFLAGS) -std=gnu89"
+
 $(eval $(autotools-package))
Signature · 1 line
2.5.3
#1 lore raw

PATCH 1/1 vtun: fix gcc5 build

Quoted 22 lines
Quoted 1 line
Quoted 1 line
Quoted 1 line
Quoted 1 line

"Baruch" == Baruch Siach <[email protected]> writes:

Add -std=gnu89 to CFLAGS to keep pre gcc5 inline function symbols visibility rules.

Fixes:
http://autobuild.buildroot.net/results/a7e/a7e684174b90a852318a7a8bf41c4f477d159d56/
http://autobuild.buildroot.net/results/ab0/ab01f8e0467e85322c63b81bb7e84fd251e5d556/
http://autobuild.buildroot.net/results/117/117a68bf648b050dabed8088d1165c1d7d9301f0/

and more.

---
 package/vtun/vtun.mk | 2 ++
 1 file changed, 2 insertions(+)
package/vtun/vtun.mk +0 -0
index 2b64ab9efc69..884ebc666431 100644

package/vtun/vtun.mk VTUN_CONF_OPTS = \ · line 17

Quoted 4 lines
package/vtun/vtun.mk +1 -0
@@ -14,4 +14,6 @@ VTUN_CONF_OPTS = \
 	--with-lzo-headers=$(STAGING_DIR)/usr/include/lzo \
 	--with-lzo-lib=$(STAGING_DIR)/usr/lib
 
+VTUN_CONF_ENV = CFLAGS="$(TARGET_CFLAGS) -std=gnu89"

Committed after adding a comment explaining why this is needed, thanks.

Signature · 1 line
Bye, Peter Korsgaard