-
Notifications
You must be signed in to change notification settings - Fork 35
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 with clang on OSX #62
base: master-dev
Are you sure you want to change the base?
Conversation
Hi, |
Hi, |
@andryblack I am refactoring a bunch of stuff in the makefile. I will check your modification and merge them with mine instead of merging the pull request. I see you also have a branch with diff patches instead of whole file overwrite. That's good. I will also integrate this feature. Thanks for contributing! I think we can close this, I will pull your branch directly from your repo and cherry pick from it. |
I think this one is an important merge request because it's really struggling to set-up an OSX CrossBridge Dev environment with non-standard Clang compiler. So its more complicated to release any new CB SDK versions. |
I have started merging by cherry picking part of the pull request while trying to update the build system (makefile) so it does not restart from the beginning if there is an error (unless you explicitly clean the repo or tell it to do so). I also made sure it is possible to do "out of tree" build and that the original repo could stay unmodified at all time and to use the default clang that come with osx. I am about halfway through, I haven't touched it since 2 weeks ago though, I had a surge of client, who take priority. I should get back to it soon. |
Agreed. |
Unrelated note since I don't know if you subscribed to the forum, but crossbridge.io is up and running, generated from the repo doc. We push a repo with instruction on how to update the site soon. In the meantime you can ask me if you need it modified. Here is the forum thread on it: |
build sdk on clean OSX, only Apple provided command line tools needed