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

Fix to support LLVM/ruby that ships with Xcode 5.1 #52

Merged
merged 1 commit into from
Apr 17, 2014
Merged

Fix to support LLVM/ruby that ships with Xcode 5.1 #52

merged 1 commit into from
Apr 17, 2014

Conversation

seanknox
Copy link
Contributor

#51

  • It currently checks for the exact version of Xcode that introduced the LLVM/Ruby issue. Didn't want to be too overzealous in matching for versions newer than this since the problem is dealt with in newer versions of Ruby.
  • Shell nitpicking: exported the ARCHFLAGS option so it would be in effect for the entire initial run of Boxen, wasn't sure offhand if setting it in the command only was preferable/fine (e.g. ARCHFLAGS=-Wno-error=unused-command-line-argument-hard-error-in-future script/boxen).

ocxo added a commit that referenced this pull request Apr 17, 2014
Fix to support LLVM/ruby that ships with Xcode 5.1
@ocxo ocxo merged commit b169925 into boxen:master Apr 17, 2014
@ocxo
Copy link
Contributor

ocxo commented Apr 17, 2014

Thanks @seanknox!

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

Successfully merging this pull request may close these issues.

2 participants