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

Change rpath of the cython file #274

Merged
merged 1 commit into from
May 4, 2023

Conversation

powei-lin
Copy link
Contributor

Change the runtime path of cythonized file to $ORIGIN. So user can copy libapriltag.so and apriltag.cpython-310-x86_64-linux-gnu.so to other places and still be able to use them in python.

CMakeLists.txt Outdated Show resolved Hide resolved
@christian-rauch
Copy link
Collaborator

Since you are changing the same section twice, can you just squash your two commits into one?

change to custom_target so it accept VERBATIM
@powei-lin
Copy link
Contributor Author

@christian-rauch Done

@christian-rauch christian-rauch merged commit 48b448b into AprilRobotics:master May 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants