-
Notifications
You must be signed in to change notification settings - Fork 615
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
Set rpath at cmake level (do not wait for bundle-wheel) #5323
Conversation
c93e1fb
to
1454756
Compare
@@ -213,6 +213,19 @@ else() | |||
set(LIBTYPE SHARED) | |||
endif() | |||
|
|||
|
|||
foreach(rpath IN ITEMS |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How it affects conda build?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It shouldn't. Conda build doesn't care about those paths anyway, and resets the rpath completely
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OK, we are overwriting them anyway https://github.com/NVIDIA/DALI/blob/main/conda/dali_native_libs/recipe/build.sh#L216.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes
!build |
CI MESSAGE: [12841037]: BUILD STARTED |
CI MESSAGE: [12841037]: BUILD FAILED |
Signed-off-by: Joaquin Anton <janton@nvidia.com>
1454756
to
f2ddd38
Compare
CI MESSAGE: [12842935]: BUILD STARTED |
Signed-off-by: Joaquin Anton <janton@nvidia.com>
CI MESSAGE: [12842935]: BUILD PASSED |
@@ -279,6 +279,7 @@ if(BUILD_NVIMAGECODEC) | |||
nvimgcodec_headers | |||
URL https://developer.download.nvidia.com/compute/nvimgcodec/redist/nvimgcodec/linux-x86_64/nvimgcodec-linux-x86_64-0.2.0.6-archive.tar.xz | |||
URL_HASH SHA512=6577a8ff5589400cdf5767aa4a507245527a96e48065f23f626dfc6ba13b136960cacfe7f61c345dc158ed0352e1e971834aec6fd98038649b08b250c3306aeb | |||
DOWNLOAD_EXTRACT_TIMESTAMP 1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is to silence a warning
Category:
Other
Description:
Improvement: Set rpath during build and not later during bundle-wheel. This allows for doing pip install directly from build directory without the need to build a wheel, and still be able to detect nvimgcodec installed in the python directory.
Additional information:
Affected modules and functionalities:
experimental decoders
Key points relevant for the review:
Tests:
Checklist
Documentation
DALI team only
Requirements
REQ IDs: N/A
JIRA TASK: N/A