libx264: Fix build errors on x86 targets due to missing ASM features (yasm)

	minor cleanup of CFLAGS, remove COPTS warnings
	yasm needs to be added to x86 toolchain
	(LEDE updated: https://github.com/lede-project/source/commit/c08651226f5645204f91f79d247801033f6015d1)

Signed-off-by: Daniel Golle daniel@makrotopia.org
Signed-off-by: Ted Hess <thess@kitschensync.net>

diff --git a/libs/libx264/Makefile b/libs/libx264/Makefile
index b99e6bd7460f402916a3b05e8f12c21f86daca90..3803808d74402acdd386ee7297ba0778d95d8573 100644 (file)
--- a/libs/libx264/Makefile
+++ b/libs/libx264/Makefile
@@ -24,9 +24,14 @@ PKG_INSTALL:=1
 
 include $(INCLUDE_DIR)/package.mk
 
-TARGET_CFLAGS:=-Wno-maybe-uninitialized -Wshadow -Wall -std=gnu99 -fPIC -O3 -ffast-math -I. 
+TARGET_CFLAGS+=-std=gnu99 -fPIC -O3 -ffast-math -I.
 MAKE_FLAGS+= LD="$(TARGET_CC) -o" 
 
+ifneq ($(CONFIG_TARGET_x86),)
+  CONFIGURE_VARS+= AS=yasm
+  MAKE_FLAGS+= AS=yasm
+endif
+
 CONFIGURE_ARGS += \
                --enable-shared \
                --enable-pic \
@@ -40,7 +45,7 @@ define Package/libx264
   SECTION:=libs
   CATEGORY:=Libraries
   TITLE:=H264/AVC free codec library.
-  DEPENDS:=@BUILD_PATENTED
+  DEPENDS:=@BUILD_PATENTED @!TARGET_x86||YASM
   URL:=http://www.videolan.org/developers/x264.html
 endef