Skip to content

Add missing RPATH entry for rocpd* binaries in the core wheel#4446

Open
lucbruni-amd wants to merge 1 commit intomainfrom
users/lucbruni-amd/add-core-rpath-entry
Open

Add missing RPATH entry for rocpd* binaries in the core wheel#4446
lucbruni-amd wants to merge 1 commit intomainfrom
users/lucbruni-amd/add-core-rpath-entry

Conversation

@lucbruni-amd
Copy link
Copy Markdown
Contributor

Motivation

Fixes (half of) #4424

Technical Details

Added core.rpath_dep(core, "lib/rocm_sysdeps/lib") so that ELFs in the core wheel (like libpyrocpd.cpython-*.so) get a patchelf RPATH entry pointing to lib/rocm_sysdeps/lib/, where librocm_sysdeps_sqlite3.so and friends live.

Test Plan

Build Python packages according to https://github.com/ROCm/TheRock/blob/main/docs/packaging/python_packaging.md, after building TheRock, and invoke rocpd2pftrace to ensure librocm_sysdeps_sqlite3.so is located.

Test Result

To observe

Submission Checklist

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

Labels

None yet

Projects

Status: TODO

Development

Successfully merging this pull request may close these issues.

2 participants