-
Notifications
You must be signed in to change notification settings - Fork 564
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
Support unixodbc in Mac OSX #19
Conversation
I'm really uncomfortable with this idea. If I remember correctly before 10.7 iODBC was shipped with OS/X and was the obvious choice. Having it link in two different ways based on whether something installs seems like a very bad idea - if you use a requirements.txt file for a project, you want to know that it will always be built the same way. I think adding a command-line option to the build is a great idea, but not doing it by default. Let me think about the different options. Thanks very much for this code. |
Just as a data point this is something I'd be interested in as well. I'm using unixodbc and currently just maintaining my own fork. |
This is really critical for modern OS X - iODBC does not come pre-packaged anymore and the only homebrew package is for unixodbc. ports might still offer iodbc but it's hard to imagine anyone preferring it these days. |
Support unixodbc in Mac OSX
Thanks. I like this, but I hope it doesn't cause a lot of issues with building since unixODBC doesn't actually ship with OS X. |
I should have known better than to merge here. Consider this still open though I'm not sure how to do so. Anyway, I've just installed unixODBC 2.3.2 on a fresh Mavericks system. It builds fine and installs libodbc.dylib and libodbc.2.dylib. Neither of these is found using (Side note: Please don't reformat imports and comments in other people's projects when adding 2-3 lines.) |
I don't think you can reopen a merged PR, perhaps it's better to create a new one. My install of unixodbc was on Mountain Lion using homebrew. On Mavericks I get the following from running
|
This resolved the MemoryError issues that I had experienced with iODBC. It seems that others have this issue as well..
Btw what is the status of the pyodbc project? There's a lot of unanswered issues these days.