Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Include macOS Sierra when checking for SIP #65

Closed
kubark42 opened this issue Oct 4, 2016 · 2 comments
Closed

Include macOS Sierra when checking for SIP #65

kubark42 opened this issue Oct 4, 2016 · 2 comments

Comments

@kubark42
Copy link
Contributor

kubark42 commented Oct 4, 2016 •
edited

SIP is also present on macOS Sierra.

ros-install-osx/install

Line 169 in 77bf795

# Check for SIP if on El Capitan
checks for OSX 10.11, but not the latest 10.12. It might be appropriate to upgrade the check to look for anything >= 10.11.
@kubark42 kubark42 changed the title SIP is also present on macOS Sierra Include macOS Sierra when checking for SIP Oct 4, 2016
@mikepurvis
Copy link
Owner

Sure, could you provide a PR for this please?

@kubark42 kubark42 mentioned this issue Oct 7, 2016
Merged
@kubark42
Copy link
Contributor Author

kubark42 commented Oct 7, 2016

Sure! I'm unfamiliar with bash, so I did what seemed appropriate. Hope it's helpful.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@mikepurvis @kubark42