faithfracture / Apple-Boost-BuildScript

Script for building Boost for Apple platforms (iOS, iOS Simulator, tvOS, tvOS Simulator, OS X)
279 stars 111 forks source link

Allow passing libraries to build as command-line argument. #4

Closed DouglasHeriot closed 8 years ago

DouglasHeriot commented 8 years ago

Since I started building extra libraries, also ran into error where a C/ObjC file was passed cpp specific compiler arg - so, separated passing in CXX args from C/CPP compiler args.

Got this error when building for iPhone: error: invalid argument '-std=c++11' not allowed with 'C/ObjC' This commit fixes it.

DouglasHeriot commented 8 years ago

Also fix actually setting min OS versions before use (previously the command line args were effectively ignored)

faithfracture commented 8 years ago

I'm not going to merge this for a couple reasons:

  1. Since I use this for my own personal use, I don't want to merge your BOOST_LIBS defaults changes
  2. I added more information to the --boost-libs CLI parameter help
  3. I found out that there were a couple other CLI-configurable parameters that weren't making it all the way through to the variables they were supposed to be configuring, so I moved all the CLI-configurable variables so they are set after parseArgs() is called.

I made a separate commit, but I did credit you with these changes. Thanks!