Skip to content

Conversation

@robertmaynard
Copy link
Contributor

Since we don't have the tooling online to validate if ABI changes have occured we will update the SOVERSION to be safe.

Since we don't have the tooling online to validate if ABI changes
have occured we will update the SOVERSION to be safe.
@cjnolet
Copy link
Member

cjnolet commented Jan 6, 2026

Will this mean that code built against cuVS 25.12 will no longer work against 26.02? We've committed to Elastic that we would begin deprecation of breaking ABI changes starting in 26.02, rather than breaking it unexpectedly.

@robertmaynard
Copy link
Contributor Author

Will this mean that code built against cuVS 25.12 will no longer work against 26.02?

Yes any project that linked explicitly against a pre-built standalone version of libcuvs_c.so will now fail to run with 26.02 ( with this change ).

Currently we don't have the infrastructure in place to validate that no ABI change has occured between 25.12 and 26.02. So the safest way is to assume an ABI change has occured.

We could validate that no ABI change has occured during burndown, but after that point we would have to enforce stricter code reviews / manual validation for any PR that modifies the C API.

@robertmaynard
Copy link
Contributor Author

Closing as #1708 replaces this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

breaking Introduces a breaking change feature request New feature or request

Development

Successfully merging this pull request may close these issues.

2 participants