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

Fix for Sybase #30

Merged
merged 1 commit into from Jul 12, 2011
Merged

Fix for Sybase #30

merged 1 commit into from Jul 12, 2011

Conversation

darrint
Copy link
Contributor

@darrint darrint commented Jul 9, 2011

This allows TinyTDS to work with Sybase. Sybase sends a message with
severity 10 anytime the database changes. This includes first connect which
makes TinyTDS completely unusable.

Allowing severity level 10 to not throw errors should be ok as documented
here:

http://www.freetds.org/userguide/samplecode.htm

This allows TinyTDS to work with Sybase. Sybase sends a message with
severity 10 anytime the database changes. This includes first connect which
makes TinyTDS completely unusable.

Allowing severity level 10 to not throw errors should be ok as documented
here:

http://www.freetds.org/userguide/samplecode.htm
metaskills added a commit that referenced this pull request Jul 12, 2011
@metaskills metaskills merged commit 0bd4130 into rails-sqlserver:master Jul 12, 2011
@metaskills
Copy link
Contributor

Thanks for the pull request. I also added these tests in too.

a834eff

If you want, you can always build your own 0.5.0 gem before it is released for your platform using Miniportile, see project README for details. Cheers!

@metaskills
Copy link
Contributor

@darrint, can you look at this thread on the FreeTDS mailing list?

http://lists.ibiblio.org/pipermail/freetds/2011q4/027476.html

Here is the diff below, does this keep you from using TinyTDS at all, was wondering how and what version you are using and if you are using a certain :tds_version string option? https://github.com/rails-sqlserver/tiny_tds/blob/master/lib/tiny_tds/client.rb#L4

--- dblib.c.ori Thu Aug 18 02:54:40 2011
+++ dblib.c Thu Nov 10 13:45:56 2011
@@ -936,6 +936,9 @@ dbsetlversion (LOGINREC * login, BYTE ve
    case DBVER60:
        login->tds_login->tds_version = 0x700;
        return SUCCEED;
+   case DBVERSION_100:
+       tds_set_version(login->tds_login, 5, 0);
+       return SUCCEED;
    case DBVERSION_71:
        tds_set_version(login->tds_login, 7, 1);
        return SUCCEED;

@darrint
Copy link
Contributor Author

darrint commented Nov 21, 2011

On Thu, Nov 10, 2011 at 9:07 AM, Ken Collins <
reply@reply.github.com>wrote:

@darrint, can you look at this thread on the FreeTDS mailing list?

http://lists.ibiblio.org/pipermail/freetds/2011q4/027476.html

Here is the diff below, does this keep you from using TinyTDS at all, was
wondering how and what version you are using and if you are using a certain
:tds_version string option?
https://github.com/rails-sqlserver/tiny_tds/blob/master/lib/tiny_tds/client.rb#L4

Sorry, I don't know either. I've not looked at TDS in some time.

Darrin

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