dbcli / mssql-cli

A command-line client for SQL Server with auto-completion and syntax highlighting
BSD 3-Clause "New" or "Revised" License
1.35k stars 192 forks source link

Changed switch statement to convert string to lowercase #430

Closed ellbosch closed 4 years ago

ellbosch commented 4 years ago

Fixes issue where switch statement would check if MSSQL_CLI_OFFICIAL_BUILD would be equal to the "True" string, without taking casing into account. This was only impacting deb and rpm packaging. Proposed solution uses a command that's supported in Bash 4.0+.

I have validated that this fix is compatible with our Dockerized approach to building Linux packages. This is currently being tested for compatibility with Azure DevOps.