Commit aac3f315 authored by Baruch Siach's avatar Baruch Siach Committed by Thomas Petazzoni
Browse files

atftp: fix build with gcc5 

gcc5 defaults to -std=gnu11 which changes the inline semantics. This leads to
multiple definition error with 'extern inline'. See
https://gcc.gnu.org/gcc-5/porting_to.html for the details.

Fixes:
http://autobuild.buildroot.net/results/762/762bce2bbc1f994cc796f0442bfa73b7ad64e272/
http://autobuild.buildroot.net/results/28b/28bf6018dcf91ddacfb76f54a92313da8bf182e1/
http://autobuild.buildroot.net/results/fde/fdee23ecd0c5c66fda1ed38efe43048ce934b733/



Signed-off-by: default avatarBaruch Siach <baruch@tkos.co.il>
Signed-off-by: default avatarThomas Petazzoni <thomas.petazzoni@free-electrons.com>
parent 112ab5f4

package/atftp/atftp.mk

+6 −1
Original line number Diff line number Diff line
@@ -11,7 +11,12 @@ ATFTP_LICENSE_FILES = LICENSE 
ATFTP_CONF_OPTS = --disable-libwrap --disable-mtftp

# For static we need to explicitly link against libpthread

ATFTP_LIBS = -lpthread

ATFTP_CONF_ENV = LIBS="$(ATFTP_LIBS)"

# We use CPPFLAGS for -fgnu89-inline even though it's a compiler flag

# because atftp discards configure environment CFLAGS. -fgnu89-inline

# is needed to avoid multiple definition error with gcc 5. See

# https://gcc.gnu.org/gcc-5/porting_to.html.

ATFTP_CONF_ENV = LIBS="$(ATFTP_LIBS)" \

	CPPFLAGS="$(TARGET_CPPFLAGS) -fgnu89-inline"



ifeq ($(BR2_PACKAGE_READLINE),y)

ATFTP_DEPENDENCIES += readline