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

Update documentation to provide table for properly choosing a TDS Ver… #71

Closed
wants to merge 3 commits into from

Conversation

FlipperPA
Copy link
Contributor

…sion to use with SQL Server.

@freddy77
Copy link
Contributor

I don't know why compilation failed... I'll try.
Also I noted there is no documentation for Sybase servers.

@FlipperPA
Copy link
Contributor Author

Thanks, I was wondering about the failues as well, since it was just simple documentation. Apologies for the lack of Sybase, but I have never worked with Sybase, just MS SQL Server. I answer a lot of questions on this topic on Stack Overflow and noticed there was a ton of confusion around TDS versions and SQL Server (most of the questions involved it in one form or another) so I decided a PR might help. If you take a look at a few of my answers, you can see the pattern: http://stackoverflow.com/users/1394697/flipperpa?tab=answers

Thanks for all of your amazing work on FreeTDS.

@freddy77
Copy link
Contributor

I think would be enough to change "SQL Server" to "Microsoft SQL Server" and put a line like

"If you have a Sybase server use version 5.0 otherwide if you have Microsoft SQL Server you can follow following section."

well.. or something better than this.

@FlipperPA
Copy link
Contributor Author

I've updated the text to include the sentence about Sybase. I hope this helps!

@freddy77
Copy link
Contributor

Merged. I removed the "Microsoft Microsoft SQL Server" and the "Microsoft" where "SQL Server" was referring to any SQL Server (Sybase or Microsoft).
I also split some too long line so on next diff they will be smaller.

@freddy77 freddy77 closed this Sep 25, 2016
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.

None yet

2 participants