Re reversal: yep, sorry. Seems my mercurial tool diff generator can go backwards in time as well as forwards, which I had assumed...
I think exposing these options as connection parameters is a great idea as it gives more transparency/clarity to the user on what mode is actually used.
As far as I gather, these options have an effect on how a user should format his SQL, so this is very relevant info.
I'd suggest adding them to the connection params and setting them to the defaults of the relevant db...
Naughty suggestion: you could also have a TDS version in the db params with choices auto, 4.2, 5.0, 7... Select auto by default.
This can let you change the TDSVER environment variable within the program so users don't have to
)
If auto selected:
- leave code as is for SQL server
- set TDSVER environment variable to 5.0 before connecting if on Sybase. That should elminate the need for a patched dblib.
If non-auto value selected: set it at beginning of code.
If you decide to implement stuff in params, I could have a look at implementing something like that afterwards and you can (dis)approve it... What do you think?