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

Shared library related improvements #149

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

Harry-Chen
Copy link

The PR mainly:

  • generates and installs shared library with proper SOVERSION (currently the first digit of the version number)
  • allows installing to user-specified locations and using external CFLAGS
  • hides symbols from the shared library by default, only exposing necessary ones (sqlite3_vec_init and friends)

These improvements make it easier for distributions to package this great library.

This allow distributions to modify install locations. For example,
in Debian on x86_64, INSTALL_PREFIX = /usr, and INSTALL_LIB_DIR =
/usr/x86_64-linux-gnu/lib.

Signed-off-by: Shengqi Chen <[email protected]>
Many definitions are not correctly marked with `static`.
Instead of changing them one-by-one, use `-fvisibility=hidden` is
a quicker future-proof fix.

Signed-off-by: Shengqi Chen <[email protected]>
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.

1 participant