Skip to content

Conversation

@fanquake
Copy link
Member

10.5 support has been dropped for some time now.

10.5 support has been dropped for some time now.
@laanwj
Copy link
Member

laanwj commented Nov 25, 2014

Does it really not build anymore on 10.5, or is it just that we stopped supporting it in the released binaries?

@jonasschnelli
Copy link
Contributor

The gitian build uses 10.7 (see https://github.com/bitcoin/bitcoin/blob/master/doc/release-process.md#fetch-and-build-inputs-first-time-or-when-dependency-versions-change).
Maybe it would be worth to define what min OSX version we support (i would set 10.7 as it is the gitians base).

I'm not sure about this pull. Maybe someone likes to build it with 10.6 for any reason.

@theuni
Copy link
Member

theuni commented Nov 25, 2014

10.6 is the current minimum for 0.10. Gitian builds with -mmacosx-version-min=10.6 regardless of which sdk is used. Fwiw, this is confirmed working with SDKs up to 10.9, I haven't tried 10.10.

Which versions should be supported is a separate discussion for after 0.10 is released.

@jonasschnelli
Copy link
Contributor

@theuni Ah. Right. We build against 10.7 but use 10.6 as minimum target which mainly set some macros, etc.

ACK on this pull.

@theuni
Copy link
Member

theuni commented Nov 25, 2014

ACK from me too, but let's make it explicit. If an sdk is too old to build with, it should fail to configure.

I think it should suffice to add a compile-test like

#include Availability.h
#if __MAC_OS_X_VERSION_MIN_REQUIRED < 1060
fail()
#endif
int main(){return 0;}

That should fail to compile on old SDKs, or if the compat version is set <= 10.6

@gmaxwell
Copy link
Contributor

I prefer making the old OSX version fail explictly or no action. (Unsupported doesn't mean we should go break things; but if we do break things better to make it explicit when we know it won't work.)

@laanwj
Copy link
Member

laanwj commented Nov 26, 2014

Agreed @gmaxwell @theuni . I just meant to say that the version that gitian builds against is not necessarily the minimum version that people could build manually against if they wanted. So there are different levels of "support" in play here. At least for Linux it is important that Bitcoin Core can stil be built on older, stable server distributions. For MacOSX this is arguably less of an issue.

@laanwj laanwj merged commit 850c570 into bitcoin:master Feb 18, 2015
laanwj added a commit that referenced this pull request Feb 18, 2015
850c570 No longer check osx compatibility in RenameThread (Michael Ford)
@fanquake fanquake deleted the osx_version_renamethread branch May 12, 2016 14:17
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Sep 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants