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

Add patch for getopt #257

Closed
wants to merge 1 commit into from
Closed

Add patch for getopt #257

wants to merge 1 commit into from

Conversation

l8nite
Copy link
Contributor

@l8nite l8nite commented Jan 15, 2016

I was trying to get the latest to install on alpine linux, stole this from http://git.alpinelinux.org/cgit/aports/tree/main/freetds/fix-includes.patch

@larskanis
Copy link
Member

Wouldn't it be better to open a pull request on freetds? The patches in tiny_tds are usually only applied for issues, that are very specific to the environment in which tiny_tds is typically used (like the old gcc/mingw version of the RubyInstaller's Devkit).

@l8nite
Copy link
Contributor Author

l8nite commented Jan 15, 2016

@freddy77 what do you think?

@metaskills
Copy link
Contributor

Correct, the patches are there to primarily support our efforts to precompile static libs. They are only ever used when FreeTDS is not installed on your system via some package manager. For example, I use MacPorts and there may be a few patches we do for our platform, TinyTDS should not assume that, I think your package manager and/or relationship to the parent project, in this case FreeTDS should. Could you open an issue on http://github.com/FreeTDS/freetds and link it to here?

@l8nite
Copy link
Contributor Author

l8nite commented Jan 15, 2016

Thanks guys, I've opened FreeTDS/freetds#45

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

3 participants