-
Notifications
You must be signed in to change notification settings - Fork 883
README improvements, clarity and typo fixes #732
Conversation
|
||
If you don't want FDE, see the note [below](#bypassing-fde-requirement). | ||
|
||
#### Github API Token |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why this section is needed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yosemite does this by default for new machines.
##### OS X < 10.9 (Mavericks) without auto install | ||
|
||
1. Install Xcode from the Mac App Store. | ||
1. In terminal, run `xcode-select --install` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This command doesn't exist before Mavericks.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Roger. Tangentially-related: is there any defined EOL date for 10.8 support?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In Homebrew-land we support the current version and two prior so that feels like a reasonable approach here. I think even after that point it's still worth landing PRs to fix 10.8 and below but arguably can tweak the docs etc. to make them less clunky then.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I had to run this command to get postgres to compile earlier. Beforehand I was getting a warning, which I don't have shell history for anymore, that went away after running this. I found the fix here, Homebrew/legacy-homebrew#21364 (comment)
No description provided.