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

Update minimum requirement info for macOS to 10.8 in download pages #562

Merged
merged 1 commit into from May 5, 2017

Conversation

carlobaldassi
Copy link
Member

The macOS binaries don't work correctly on 10.7, see JuliaLang/julia#20651 (which doesn't seem to be about to be fixed any time soon an most likely won't be fixed at all).

Although the binary would still have 10.7+ in the name, at least with this change the information on the website would be correct (I had a puzzled coworker wondering why Pkg.add was failing).

Copy link
Member

@ararslan ararslan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd be surprised if anyone cared enough to ensure things still work on a macOS version 5 generations behind the current one, so this seems like a good idea to me.

@ararslan
Copy link
Member

ararslan commented May 4, 2017

Do they work on Mountain Lion (10.8)?

@carlobaldassi
Copy link
Member Author

Do they work on Mountain Lion (10.8)?

I have no way to test that, however from what I understand the binaries seemed to be working on 10.7 except for the issues with libgit2, so possibly yes?

@ararslan
Copy link
Member

ararslan commented May 4, 2017

Okay, good to know. It'd probably be worth having a larger discussion on the minimum macOS version we want to claim to support (cc @staticfloat and @tkelman) but in the meantime this LGTM.

@staticfloat
Copy link
Sponsor Member

I would support moving the minimum supported mac version up. While 10.8 may be the technically lowest we can run modern versions of Julia on, we may want to bump the actually "supported" version up a little higher to 10.9, as that's what our buildbots and tests actually run on.

@ViralBShah
Copy link
Member

I agree that we may want to go to 10.9, but let's at least make it 10.8. Perhaps we should also adjust that in the build scripts.

@ViralBShah ViralBShah merged commit 722bc6e into master May 5, 2017
@tkelman tkelman deleted the cb/osx108 branch May 5, 2017 16:53
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants