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. |
bebop_driver/CMakeLists.txt
Outdated
| # PATTERN ".svn" EXCLUDE | ||
| # ) | ||
| # Copy ARSDK libraries | ||
| install(DIRECTORY ${ARDRONESDK3_PATH}/lib/ |
There was a problem hiding this comment.
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.