Summary
Support newer Xcode versions on Mac
Metadata
- Id: d614f60b37ee846b2de4c2634850890cc9f35c00
- Trac id: 1053
- Type: defect
- Reporter: felix
- Owner:
- Cc:
- Status: closed
- Component: build system
- Estimated difficulty:
- Resolution: fixed
- Priority: major
- Milestone: someday
- Version: 4.8.x
- Changetime: 2014-07-02 08:34:11 UTC
- Created: 2013-09-25 10:23:05 UTC
- Keywords:
Attachments
- d614f60b37ee846b2de4c2634850890cc9f35c00/attachments/0002-Detect-Xcode-sdk-and-toolchain-correctly-for-newer-x.patch
- d614f60b37ee846b2de4c2634850890cc9f35c00/attachments/0001-Add-set-of-extra-options-for-compiler-linker-and-ass.patch
Description
in Xcode 4 and 5 the development tools are now all located under the `Applications/Xcode.app` directory (by default). It should be possible to check this and setup all paths automatically without requiring the user to modify the PATH.
Note that the paths changed between Xcode 4 and 5 (of course).
Additionally `-isysroot` options have to be set when running the compiler. This is particularly important when you want to build for iOS (either the simulator or the device).
Attached are two patches that extend the build-system and some scripts that try to figure out the correct location of Xcode. Note that this hasn't been adapted to Xcode 5 yet (but I can assist if someone is interested).
Changes and comments
[2013-09-25 10:23:29 UTC] felix attached 0001-Add-set-of-extra-options-for-compiler-linker-and-ass.patch (description=#f)
[2013-09-25 10:23:38 UTC] felix attached 0002-Detect-Xcode-sdk-and-toolchain-correctly-for-newer-x.patch (description=#f)
[2013-09-25 11:09:42 UTC] sjamaan wrote:
Why doesn't the patch simply append to C_COMPILER_OPIONS?
This EXTRA_OPTIONS sounds a bit crufty, and it isn't there on all platforms; the patch only adds it to the osx and the mingw makefile (the latter doesn't seem to use it)
[2013-09-28 14:35:33 UTC] felix wrote:
Replying to sjamaan: > Why doesn't the patch simply append to C_COMPILER_OPIONS? > > This EXTRA_OPTIONS sounds a bit crufty, and it isn't there on all platforms; the patch only adds it to the osx and the mingw makefile (the latter doesn't seem to use it)
The patch is probably insufficient, it is just a first attempt. C_COMPILER_OPTIONS doesn't propagate everywhere, I think in particular not to the options stored in `chicken-config.h`. Also, to allow customization of compiler options when building, it is currently necessary to override C_COMPILER_OPTIMIZATION_OPTIONS, possibly nuking those options that are enabled by default (and which we know, work). The build should provide an initially empty setting for user-defined compilation options.
[2014-07-02 08:34:11 UTC] sjamaan changed status from new to closed
[2014-07-02 08:34:11 UTC] sjamaan set resolution to fixed
[2014-07-02 08:34:11 UTC] sjamaan wrote:
Implemented a while ago, by c6a6a26225620b566f8bc487cc4180de55555a2a