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

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

Open
stuartpa opened this issue Jan 2, 2024 · 1 comment
Labels
enhancement New feature or request sqlcmd switch switch in existing sqlcmd
Milestone

Comments

@stuartpa
Copy link
Collaborator

stuartpa commented Jan 2, 2024

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")

@stuartpa stuartpa changed the title Add a "not yet implemented" error for all back-compat switches so that users get an actionable error message Add a "not yet implemented" error for all back-compat switches so user gets an actionable error message Jan 2, 2024
@dlevy-msft dlevy-msft added enhancement New feature or request sqlcmd switch switch in existing sqlcmd labels Jan 3, 2024
@shueybubbles
Copy link
Collaborator

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

@dlevy-msft dlevy-msft added this to the Backlog milestone Jan 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request sqlcmd switch switch in existing sqlcmd
Projects
None yet
Development

No branches or pull requests

3 participants