Skip to content
This repository has been archived by the owner on Dec 2, 2020. It is now read-only.

Fix broken Xcode 5.1 w/patch in script/boxen #51

Closed
seanknox opened this issue Apr 14, 2014 · 4 comments
Closed

Fix broken Xcode 5.1 w/patch in script/boxen #51

seanknox opened this issue Apr 14, 2014 · 4 comments

Comments

@seanknox
Copy link
Contributor

Now that clang/LLVM shipped in Xcode 5.1 errors when running Boxen the first time, one suggestion was to patch system Ruby to use the same behavior in Xcode <= 5.0.

Would it be worthwhile to add something like this in script/boxen?

# Fix for system ruby with Xcode 5.1
if [[ $CLT_VERSION =~ ^5\.1 ]]; then
  echo "--> Fixing system Ruby that ships with Xcode 5.1."

  curl --progress-bar https://gist.githubusercontent.com/seanknox/10688629/raw/1e57fbb440d36ca5607d1739cc6151f373b234b6/gistfile1.txt | sudo patch /System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/universal-darwin13/rbconfig.rb
fi
@rafaelfranca
Copy link
Member

I don't think we should patch system Ruby. I think we can just pass ARGSFLAG on script/boostrap

@seanknox
Copy link
Contributor Author

You're right, patching is overzealous. Are you saying that setting ARCHFLAGS in https://github.com/boxen/our-boxen's script/bootstrap might do the trick?

@rafaelfranca
Copy link
Member

I think so.

@seanknox
Copy link
Contributor Author

Fixed in #52.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

No branches or pull requests

2 participants