Skip to content

Comments

When bundling libnvcomp.so.X only append the major version value#19786

Merged
rapids-bot[bot] merged 3 commits intorapidsai:branch-25.10from
robertmaynard:debug_wheel_load_failures
Aug 25, 2025
Merged

When bundling libnvcomp.so.X only append the major version value#19786
rapids-bot[bot] merged 3 commits intorapidsai:branch-25.10from
robertmaynard:debug_wheel_load_failures

Conversation

@robertmaynard
Copy link
Contributor

@robertmaynard robertmaynard commented Aug 25, 2025

Description

We previously extracted the whole X.Y.Z.P version string of nvcomp and would therefore fail to load the correct version in our wheel packages.

Checklist

  • I am familiar with the Contributing Guidelines.
  • New or existing tests cover these changes.
  • The documentation is up to date with these changes.

@copy-pr-bot
Copy link

copy-pr-bot bot commented Aug 25, 2025

Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

@robertmaynard
Copy link
Contributor Author

/okay to test 60e8205

@robertmaynard robertmaynard added bug Something isn't working non-breaking Non-breaking change labels Aug 25, 2025
@robertmaynard robertmaynard force-pushed the debug_wheel_load_failures branch from 60e8205 to 1c25c27 Compare August 25, 2025 14:01
@robertmaynard
Copy link
Contributor Author

/okay to test 1c25c27

@robertmaynard
Copy link
Contributor Author

/okay to test 23ac00b

@robertmaynard robertmaynard force-pushed the debug_wheel_load_failures branch from 23ac00b to 2d69e5c Compare August 25, 2025 14:34
@robertmaynard
Copy link
Contributor Author

/okay to test 2d69e5c

@robertmaynard
Copy link
Contributor Author

/okay to test 013af0b

We previously extracted the whole X.Y.Z.P version string of nvcomp and would therefore
fail to load the correct version in our wheel packages.
@robertmaynard robertmaynard force-pushed the debug_wheel_load_failures branch from 013af0b to 35347d5 Compare August 25, 2025 17:10
@robertmaynard robertmaynard changed the title Debug load.py When bundling libnvcomp.so.X only append the major version value Aug 25, 2025
@robertmaynard robertmaynard marked this pull request as ready for review August 25, 2025 17:11
@robertmaynard robertmaynard requested a review from a team as a code owner August 25, 2025 17:11
@TomAugspurger
Copy link
Contributor

/merge

@rapids-bot rapids-bot bot merged commit 9c63bcd into rapidsai:branch-25.10 Aug 25, 2025
102 checks passed
KyleFromNVIDIA added a commit to KyleFromNVIDIA/cudf that referenced this pull request Aug 25, 2025
galipremsagar pushed a commit to galipremsagar/cudf that referenced this pull request Aug 27, 2025
…apidsai#19786)

We previously extracted the whole X.Y.Z.P version string of nvcomp and would therefore fail to load the correct version in our wheel packages.

Authors:
  - Robert Maynard (https://github.com/robertmaynard)
  - Bradley Dice (https://github.com/bdice)

Approvers:
  - Kyle Edwards (https://github.com/KyleFromNVIDIA)

URL: rapidsai#19786
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working non-breaking Non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants