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 version compatibility checks #83

Open
gavv opened this issue Apr 5, 2023 · 0 comments
Open

Add version compatibility checks #83

gavv opened this issue Apr 5, 2023 · 0 comments
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Contributions are welcome

Comments

@gavv
Copy link
Member

gavv commented Apr 5, 2023

This should be done after #57

When the library is initialized, we need to retrieve versions of native lib and go bindings, and check the following invariant:

  • major versions are equal
  • minor version of bindings is greater than or equal to minor version of native lib

(See "Versioning" section in Readme for explanation)

If the invariant is not satisfied, we should throw an exception with a message including the retrieved versions and explanation of what is wrong.

We can do this check in RocLibrary.loadLibrary.

@gavv gavv added enhancement New feature or request good first issue Good for newcomers help wanted Contributions are welcome labels Apr 5, 2023
@gavv gavv changed the title Add version checking Add version compatibility checks Apr 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Contributions are welcome
Projects
None yet
Development

No branches or pull requests

1 participant