contrib: Do not hardcode compiler variables for macOS
authorDavid Fuhrmann <dfuhrmann@videolan.org>
Sun, 9 Jul 2017 16:53:09 +0000 (18:53 +0200)
committerDavid Fuhrmann <dfuhrmann@videolan.org>
Mon, 17 Jul 2017 17:22:09 +0000 (19:22 +0200)
Removal of those variables is done for multiple reasons:
- In general, the caller should set correct variables for compilers,
  this should not be done by the script itself
- In recent macOS versions, default stubs for compilers exist
  (/usr/bin/clang / gcc / cc), so there is not need to use xcrun
  anymore
- Some contribs fail to build with the multi-command notion given by
  xcrun cc, instead they are looking for a tool named "xcrun cc"
  which does not exist

contrib/src/main.mak

index e95b789..79cf780 100644 (file)
@@ -111,12 +111,6 @@ endif
 
 ifdef HAVE_MACOSX
 MIN_OSX_VERSION=10.7
-CC=xcrun cc
-CXX=xcrun c++
-AR=xcrun ar
-LD=xcrun ld
-STRIP=xcrun strip
-RANLIB=xcrun ranlib
 EXTRA_CFLAGS += -isysroot $(MACOSX_SDK) -mmacosx-version-min=$(MIN_OSX_VERSION) -DMACOSX_DEPLOYMENT_TARGET=$(MIN_OSX_VERSION)
 EXTRA_CXXFLAGS += -stdlib=libc++
 EXTRA_LDFLAGS += -Wl,-syslibroot,$(MACOSX_SDK) -mmacosx-version-min=$(MIN_OSX_VERSION) -isysroot $(MACOSX_SDK) -DMACOSX_DEPLOYMENT_TARGET=$(MIN_OSX_VERSION)