-
Notifications
You must be signed in to change notification settings - Fork 34
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Build from command line #21
Comments
- This is a really basic Makefile to build from command line to make it easy for folks who just want to build and run - Update README - In ref to #21
Is this issue pretty much resolved? |
There is a very basic Makefile that addresses this, but I thought I'd leave this open for now for reference till it's improved. |
Unable to build from command line using
|
Hi @vprasanth! I’m surprised it would have compiled inside of Xcode 6.2 because as far as I recall it doesn’t include the Swift 1.2 compiler which has breaking changes. 6.2 was basically the same as 6.1.1 from a Swift perspective as it was focused essentially all on WatchKit. Thus, looking at the logs, seems to be due to the Xcode version. Xcode 6.3 beta 4 is required, which includes the Swift 1.2 compiler, available here. You can check the Swift compiler version you have like so $ swiftc -v
Apple Swift version 1.2 (swiftlang-602.0.47.4 clang-602.0.48)
Target: x86_64-apple-darwin14.1.0 Once you've downloaded it and put it in place, make sure to run the following to set it as the active Xcode $ sudo xcode-select -s /Applications/Xcode-beta.app/Contents/Developer/ |
Ah ok, I have also downloaded the beta. Will check this out and report back once I get home. Thank you! |
@beltex Awesome. So after setting the beta compiler as the active one, everything worked out. Also, I compiled it within Xcode beta, that was why that had no problems. Thanks and great work! :D |
@vprasanth Great! np, thanks! :) |
Two key changes here. First, is that we now by default init the submodules. This should help alleviate some issues folks have had building from source, in which the recursive switch was not used on repo clone. Shouldn't affect anything if its already been done. Second, running the strip command on the release binary, which will help reduce it's size. Down by 1.6 MB, now at 2.6 MB. Xcode's install action does this as well, but its temp dir install structure is a bit strange, requires cleanup, want to understand it a bit further before using. As for the Makefile itself, we can get fancier by actually letting make determine what needs to be redone (to an extent), its primary feature. Right now, all targets are phony and will run by default. Reference: #21
Done. Sorry it took so damn long! :) |
Ooops sorry, just noticed you up'ed it to Xcode 7 beta 3 |
@vprasanth No worries! :) Yeah I always try and state the required version in the README. For more on why the Swift 2.0 branch was merged to master now, see: #30 (comment) |
xcodebuild
tools and the like?The text was updated successfully, but these errors were encountered: