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
323 stars 56 forks source link

Add a "not yet implemented" error for all back-compat switches so user gets an actionable error message #496

Open stuartpa opened 6 months ago

stuartpa commented 6 months ago

If there are any switches left from the old ODBC sqlcmd that are not yet implemented in the new go-sqlcmd, please put in a stub implementation for go-sqlcmd now, which gives an actionable error message.

We are seeing some cases where go-sqlcmd is being drop in replaced with the old ODBC sqlcmd and the swithch synatac isn't identical (-N right now is an example, -N in go-sqlcmd needs a parameters, e.g. -N true, whereas in ODBC sqlcmd, it is just "-N")

shueybubbles commented 6 months ago

For -N in particular we can fix it so "no parameter" is the same as -N true