microsoft/homebrew-mssql-release

Update install instructions (--no-sandbox is an invalid option)

davidjb opened this issue · 10 comments

davidjb commented

The install instructions at https://docs.microsoft.com/en-us/sql/connect/odbc/linux-mac/installing-the-microsoft-odbc-driver-for-sql-server?view=sql-server-2017#os-x-1011-el-capitan-and-macos-1012-sierra feature the command:

brew install --no-sandbox msodbcsql@13.1.9.2 mssql-tools@14.0.6.0

which fails because --no-sandbox isn't an option that brew install takes.

Usage: brew install [options] formula
[...options and usage text...]
Error: invalid option: --no-sandbox

The install succeeds fine when brew install is called just as is so the solution is just to remove this option.

There was the general all-options-removed change to Homebrew which affected all formulae, so I presume removing options relating to sandboxing were caught up in that too. See also https://discourse.brew.sh/t/since-homebrew-2-0-0-install-params-always-give-invalid-option/4075

v-chojas commented

The documentation was changed to remove --no-sandbox.

David-Engel commented

Looks like it was only removed for v17. I submitted a PR to remove it for v13, too.

andymcarter commented

I found this solution, but the '--no-sandbox' option is still included on this page:

https://www.microsoft.com/en-us/sql-server/developer-get-started/python/mac/

  • 👍2
arkadit commented
David-Engel commented

@jacks0n Thanks. That reminds me I need to find someone to merge my months old PR which fixes that reference. :)

hanhanhan commented

It's also included here:
https://cloudblogs.microsoft.com/sqlserver/2017/05/16/sql-server-command-line-tools-for-macos-released/

Which was the top result for me when I searched for homebrew sqlcmd

  • 👍1
v-chojas commented

Yes, as the URL says, that is from 2017 when the option was still needed. The doc itself has the latest version.