-
Notifications
You must be signed in to change notification settings - Fork 280
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
Update the OS X deployment target from 10.8 to 10.11 #577
Conversation
The Travis build failure is on the iOS target, which I didn't change. I don't know what's going on there. |
I did some digging around, because it annoyed me too. It seems like this is a bug in Xcode and should not cause a warning. I am strongly against setting the deployment target up, as it breaks other apps. GitHub Desktop for example is targeting 10.9. Please see this thread: https://forums.developer.apple.com/thread/20024 |
OK, sounds like this isn't the right solution then. Though I'm left wondering why libgit2 is getting built for 10.11. Maybe the build script doesn't specify, so it defaults to the latest. |
@Uncommon Are the warnings coming from the linker phase of the libgit2 target?
These say 10.10 because I captured them last year. |
Based on this SO answer, the fix may be to specify the |
Note that #570 resolves these warnings because libgit2 is built from an Xcode project instead of cmake. |
The warnings show up under the ObjectiveGit-Mac target rather than libgit2, but they look just like that. |
@Uncommon do you still get these errors in Xcode 8? |
Yes, I still see them in Xcode 8 beta 2. |
Also in Xcode 8 final. |
The libgit2 link might be fixable (we're in control of that, we might just need to pass the correct MACOSX_DEPLOYMENT_TARGET through CMake. libssh2 isn't so easy — AFAIK there's no way to ask Homebrew to do that... |
Every time I build my project, I get a slew of warnings because libgit2 was built for 10.11, but ObjGit is targeting 10.8.