microsoft / go-sqlcmd

The new sqlcmd, CLI for SQL Server and Azure SQL (winget install sqlcmd / sqlcmd create mssql / sqlcmd open ads)
https://learn.microsoft.com/sql/tools/sqlcmd/go-sqlcmd-utility
MIT License
334 stars 58 forks source link

Add LICENSE/NOTICE.md to sqlcmd install folder #289

Closed stuartpa closed 1 year ago

stuartpa commented 1 year ago
  1. Add LICENSE/NOTICE.md to the install directory on Windows .msi installs (winget/choco)
  2. Add arm/arm64 .msi builds (a separate winget repo PR adds these to the winget installer)
  3. Also, align Architecture naming to winget/brew/debian:

So we will have:

x64 (for Windows/Mac) amd64 (for Linux)

x86 (for Windows/Mac) arm (for all Platforms) arm64 (for all Platforms)

Sources:

https://github.com/jedieaston/winget-cli/blob/0384fcd3657942f59bc94aae3dd888742f9ef632/schemas/JSON/settings/settings.schema.0.2.json#L58

        "enum": [
          "neutral",
          "x64",
          "x86",
          "arm64",
          "arm"
        ],

This also seems to be align with brew:

https://github.com/Homebrew/homebrew-core/blob/361429d75bf64c698c3d971f7aeebe01cedb87c2/Formula/sqlcmd.rb

sha256 cellar: :any_skip_relocation, arm64_ventura:  
sha256 cellar: :any_skip_relocation, arm64_monterey:
sha256 cellar: :any_skip_relocation, arm64_big_sur:  
sha256 cellar: :any_skip_relocation, ventura:        
sha256 cellar: :any_skip_relocation, monterey:       
sha256 cellar: :any_skip_relocation, big_sur:        
sha256 cellar: :any_skip_relocation, x86_64_linux:   

For Debian installers, the choices are:

Available options: amd64,arm64,armel,armhf,i386,mips,mips64el,mipsel,ppc64el,s390x