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

Make biscuit_auth module init function pub #34

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

adamh-oai
Copy link
Contributor

We have python bindings to both the stock biscuit-auth crate, as well
as our own rust crate that depends on biscuit-auth. In order for
these to interoperate they all need to be linked together into a
single shared library. (Or at least, that was the safest way I could
find to do it).

AFAICT python requires that the shared library name match the module
name, so it will not call the module init function in this case. This
PR makes the biscuit_auth module init function pub so it can be
called from another crate's module init function.

We have python bindings to both the stock biscuit-auth crate, as well
as our own rust crate that depends on biscuit-auth.  In order for
these to interoperate they all need to be linked together into a
single shared library.  (Or at least, that was the safest way I could
find to do it).

AFAICT python requires that the shared library name match the module
name, so it will not call the module init function in this case.  This
PR makes the `biscuit_auth` module init function `pub` so it can be
called from another crate's module init function.
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