-
Notifications
You must be signed in to change notification settings - Fork 121
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
CMake install rules #64
base: indigo-devel
Are you sure you want to change the base?
Conversation
Thanks for the PR. Could you please rebase your changes on top of the new changes in |
dd1ac7b
to
aaa9f32
Compare
Thank you for your reply, the rebase is now ready. |
# PATTERN ".svn" EXCLUDE | ||
# ) | ||
# Copy ARSDK libraries | ||
install(DIRECTORY ${ARDRONESDK3_PATH}/lib/ |
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.
Installing Parrot SDK's libs into ${CATKIN_PACKAGE_LIB_DESTINATION}
might a bit problematic, since it also includes libraries like libcurl.so
and libjson.so
. I have been trying to package the SDK independently with the ultimate goal of removing the inline build step (#2). If you look at the install rules there you see that I copy all the products into a sub-folder. In your opinion, is it a safer approach?
@adynathos Would it be possible to modify/rebase your changes on top of the current |
aaa9f32
to
664029c
Compare
Added rules to copy files necessary for running the module with "make install".
664029c
to
5228e9b
Compare
Ok so I have updated it to remove the references to ARSDK binaries. But when I did
I solved this by symlinking all |
Thanks for rebasing your changes and your efforts on debugging it. The problem might be on my side since the install rules for |
@adynathos I think I've found a workaround for runtime discovery of Could you please test this solution? The change is applied on top of your proposed changes in the following branch: https://github.com/AutonomyLab/bebop_autonomy/tree/adynathos-indigo-devel Could you also please clarify what the commit 0464832 does? |
Added rules to copy files necessary for running the module with "make install".
I need to put the module in a Docker container and therefore determine the files that need to be copied.