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

exporting symbols #8

Closed
jubalh opened this issue Feb 23, 2018 · 5 comments
Closed

exporting symbols #8

jubalh opened this issue Feb 23, 2018 · 5 comments
Assignees

Comments

@jubalh
Copy link
Contributor

jubalh commented Feb 23, 2018

I saw that both Fedora and openSUSE export some more symbols. Maybe this is something that should be upstreamed?

Fedora symbols:
https://src.fedoraproject.org/rpms/unixODBC/blob/master/f/export-symbols.patch
openSUSE exports:
https://build.opensuse.org/package/view_file/server:database/unixODBC/unixODBC-2.3.1-libodbcinst-exports.patch?expand=1

@lurcher
Copy link
Owner

lurcher commented Feb 26, 2018

The Fedora addition looks like it is worth adding, but I would like to know what problem the openSUSE patch fixes, the base its applied to is 7 years old and I have not had a report of the code needing that from anywhere else.

@jubalh
Copy link
Contributor Author

jubalh commented Feb 26, 2018

It's an old bug indeed: https://bugzilla.suse.com/show_bug.cgi?id=131175
Seems that unixODBC-gui-qt needed it to work properly.
However we don't ship the GUI anymore, so it is not really relevant to us.

Thought it best to be reported here anyways.

@jubalh
Copy link
Contributor Author

jubalh commented Feb 26, 2018

@lurcher
Copy link
Owner

lurcher commented Feb 26, 2018

Ok, I am in agreement that leaving it here is fine, if it does come back we can deal with it then.

The second Fedora patch, I am thinking to ignore. Microsoft have removed those defs from ODBC, so if the absence causes a problem building against unixODBC, the same problem will occur latter on in a Windows build. If there are still drivers out there that fail because of that, I would suggest that they add the typedefs to their own code as its their choice.

@praiskup
Copy link

praiskup commented Apr 27, 2018

Thanks a lot for making those symbols upstream. I guess closing this now is fine.
edit: FWIW, Fedora PR dropping the downstream patch: https://src.fedoraproject.org/rpms/unixODBC/pull-request/4

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

No branches or pull requests

3 participants