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.

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

sandbox: sandbox all taps by default. #2898

Merged
merged 1 commit into from
Jul 30, 2017
Merged

sandbox: sandbox all taps by default. #2898

merged 1 commit into from
Jul 30, 2017

Conversation

MikeMcQuaid
Copy link
Member

We've been doing this in brew test-bot, for our CI and for homebrew/core long enough that this is a reasonable default that provides more protection to our users of non-homebrew/core taps.

@MikeMcQuaid
Copy link
Member Author

I'll hold off merging this until we plan on shipping 1.3.0.

We've been doing this in `brew test-bot`, for our CI and for
homebrew/core long enough that this is a reasonable default that
provides more protection to our users of non-homebrew/core taps.
Copy link
Contributor

@apjanke apjanke left a comment

Choose a reason for hiding this comment

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

Well, this will be exciting. But welcome.

@MikeMcQuaid MikeMcQuaid merged commit 3821132 into Homebrew:master Jul 30, 2017
@MikeMcQuaid MikeMcQuaid deleted the sandbox-all-taps branch July 30, 2017 15:02
chapskev added a commit to chapskev/msphpsql that referenced this pull request Sep 13, 2017
I have determined that the install failure is due to another "security feature" recently introduced into Homebrew --- as of 1.3.0 --- which runs all installers in a restricted sandbox environment: Homebrew/brew#2898

However, you can still disable that with an option, and it will register the driver in odbcinst.ini correctly without having to perform the manual workaround above:

microsoft/homebrew-mssql-release#1 (comment)
@Homebrew Homebrew locked and limited conversation to collaborators May 4, 2018
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.

4 participants