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

fix(rust): Allow loading more recent libraries #153

Merged
merged 1 commit into from
Jul 6, 2023

Conversation

aseyboldt
Copy link
Contributor

I think it should be legal to provide a bridgestan library that was compiled with a later version of bridgestan, if the major version still matches. But the rust wrapper expects an exact version match right now. This means that downstream rust libraries have to specify an exact version of the python bridgestan library, or a user could update the python library and libraries built by that new bridgestan couldn't be loaded anymore.

Supporting older libraries would be some work, as we would have to handle the case that symbols might be missing in the library, so we still don't allow that.

This change would also imply that the ABI can't change without a major version bump. I can't really think of reasons why that would be a problem though...

It should be legal to provide a bridgestan library that was compiled
with a later version of bridgestan, providing the major version
still matches.
@WardBrian
Copy link
Collaborator

This change would also imply that the ABI can't change without a major version bump.

We only really guarantee a stable API, but ABI compatibility should be inherited from whatever C compiler is used. This sounds good to me

@WardBrian WardBrian added the rust Pull requests that update Rust code label Jul 6, 2023
@WardBrian WardBrian merged commit ad320eb into roualdes:main Jul 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
rust Pull requests that update Rust code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants