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

I added a cross-platform implementation of pyodbc.drivers() #13

Closed
wants to merge 2 commits into from

Conversation

sergiodlc
Copy link

Hi Michael:

I removed the windows only version of pyodbc.drivers() and added an OS independent version of it, along with its tests. I tested that with iODBC in Ubuntu 12.04 x86.

@sergiodlc
Copy link
Author

Also added a way to detect the installed ODBC driver manager for Linux systems so we can link to the one installed in setup.py.

@ghost ghost assigned mkleehammer Mar 22, 2013
@apatil
Copy link

apatil commented Aug 12, 2015

Hi @mkleehammer, is this PR still in the pipeline?

@mkleehammer
Copy link
Owner

Merged the drivers implementation. Sorry it took so long - I thought this was a different patch.

I did not merge the 2nd commit - I'd need some more time with it, especially with systems that have both installed for testing, etc. Plus I think I'd like a warning if it picks iODBC which seems to be falling out of favor and may have some SQLWCHAR issues.

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.

3 participants