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

Compatible with isql/osql syntax #530

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

chenall
Copy link
Contributor

@chenall chenall commented Mar 7, 2024

Allows using server,port to specify server address and port number #528

tsql -Usa -Ppassword -S127.0.0.1,2022

This change makes it easy to replace batch scripts written in isql/osql syntax by renaming tsql to isql/osql

…er and port number

tsql -Usa -Ppassword -S127.0.0.1,2022

This change makes it easy to replace batch scripts written in isql/osql syntax by renaming tsql to isql/osql

Signed-off-by: chenall <[email protected]>
@freddy77
Copy link
Contributor

freddy77 commented Mar 7, 2024

Personally I would avoid it. Configuration is already confusing, adding weird settings does not help, the server,port should IMHO something to leave to ODBC.
Why don't you instead use TDSPORT override the port and unify the script (see https://www.freetds.org/userguide/envvar.html) ?

@chenall
Copy link
Contributor Author

chenall commented Mar 8, 2024

It is not convenient to use TDSPORT because the port number is not fixed in different servers.

Modify the batch script to replace the original SERVER,PORT with SERVER:PORT can also be used.
I am lazy, so I think it is more convenient to modify the code directly for compatibility. ^_^

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants