[vlc-commits] contrib: fixed libvpx compilation with Xcode 4.3 and later

[Felix Paul Kühne]

vlc | branch: master | Felix Paul Kühne <fkuehne at videolan.org> | Fri Jun  8 10:30:08 2012 +0200| [a26c0ae3ec1609f95a39768f8c06802aa59a2d98] | committer: Felix Paul Kühne

contrib: fixed libvpx compilation with Xcode 4.3 and later

> http://git.videolan.org/gitweb.cgi/vlc.git/?a=commit;h=a26c0ae3ec1609f95a39768f8c06802aa59a2d98
---

 contrib/src/vpx/libvpx-mac.patch |   16 ++++++++++++++++
 contrib/src/vpx/rules.mak        |    9 +++++++++
 2 files changed, 25 insertions(+)

diff --git a/contrib/src/vpx/libvpx-mac.patch b/contrib/src/vpx/libvpx-mac.patch
new file mode 100644
index 0000000..9c1033c
--- /dev/null
+++ b/contrib/src/vpx/libvpx-mac.patch
@@ -0,0 +1,16 @@
+libvpx's configure script hard-codes the SDK location of previous Xcode release in the /Developer folder. However, starting with Xcode 4.3, the SDKs moved to /Applications/Xcode.app/blabla
+VLC's contrib system is clever enough to detect this, but libvpx fails miserably. However, they are providing a work-around for iOS and Android, which is expanded by this patch.
+
+diff -ru libvpx/build/make/configure.sh libvpx/build/make/configure.sh
+--- libvpx/build/make/configure.sh	2012-06-08 10:26:47.000000000 +0200
++++ libvpx-fixed/build/make/configure.sh	2012-06-08 10:26:07.000000000 +0200
+@@ -628,6 +628,9 @@
+     if [ -d "/Developer/SDKs/MacOSX10.7.sdk" ]; then
+         osx_sdk_dir="/Developer/SDKs/MacOSX10.7.sdk"
+     fi
++    if [ -d "${sdk_path}" ]; then
++        osx_sdk_dir=${sdk_path}
++    fi
+ 
+     case ${toolchain} in
+         *-darwin8-*)
diff --git a/contrib/src/vpx/rules.mak b/contrib/src/vpx/rules.mak
index 64f7bbe..133b232 100644
--- a/contrib/src/vpx/rules.mak
+++ b/contrib/src/vpx/rules.mak
@@ -18,6 +18,9 @@ libvpx: libvpx-$(VPX_VERSION).tar.bz2 .sum-vpx
 	$(UNPACK)
 	$(APPLY) $(SRC)/vpx/libvpx-no-cross.patch
 	$(APPLY) $(SRC)/vpx/libvpx-no-abi.patch
+ifdef HAVE_MACOSX
+	$(APPLY) $(SRC)/vpx/libvpx-xcode43.patch
+endif
 	$(PATCH_BASH_LOCATION)
 	$(MOVE)
 
@@ -85,6 +88,12 @@ VPX_CONF := \
 ifndef HAVE_WIN32
 VPX_CONF += --enable-pic
 endif
+ifdef HAVE_MACOSX
+VPX_CONF += --sdk-path=$(MACOSX_SDK)
+endif
+ifdef HAVE_IOS
+VPX_CONF += --sdk-path=$(SDKROOT)
+endif
 
 .vpx: libvpx
 	cd $< && CROSS=$(VPX_CROSS) ./configure --target=$(VPX_TARGET) \