-
Notifications
You must be signed in to change notification settings - Fork 250
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
librealsense: drop the recipe #457
Conversation
@rojkov I reviewed the commit and it looks good. Currently I do not have access to my build machine until March 13th, as I am on vacation. If there is no need to hurry, I suggest to postpone the merge until I am back from vacation; hopefully, we can also update the upstream status until then. If this pull request is a blocker for intel/intel-iot-refkit#49, we can merge the pull request without me testing it, as it is limited to the realsense recipe. If the dependencies on the layers have changed, we should also mention that on the ROS wiki pages. By the way, I would prefer to see the ros recipes from the intel-iot-refkit repository in the meta-ros repository; two to three weeks ago, some user mentioned his interest in moveit recipes. What are the reasons that you wanted to provide them in the iot-refkit repository? |
@bulwahn No hurry, have your time! My priorities have been changed drastically for the next two weeks at least, so this PR can definitely wait. I put MoveIt to intel-iot-refkit just for the sake of agility because in intel-iot-refkit we can't use libtinyxml and quite many packages including moveit have to be patched to be linkable against libtinyxml2. I wanted to maintain all the patches in one place. But I'd be happy to include the new recipes to meta-ros. |
Okay, when I am back, I will do the usual testing and then merge the pull request. If you find time a pull request with moveit recipes is also always greatly appreciated from my side. |
@bulwahn Any chance to test this PR soon? Now it's getting more importance... |
As the main repository for the librealsense recipe now contains fixes required for linking catkin-based packages against it we can stop maintaining our own version of librealsense recipe. Also this patch adapts realsense-camera recipe to link against the non-catkin version of librealsense. Signed-off-by: Dmitry Rozhkov <[email protected]>
fe4f0c1
to
7f889a8
Compare
Updated the realsense-camera recipe to 1.7.2 and force pushed. |
@rojkov Sorry, I have been busy with other topic, and then used my meta-ros working time on the side track of this boost issue. I have two build machines (one in Fedora, the other is Ubuntu), but on both now fail with an issue on libgphoto2. This libgphoto2 issue is very, very probable not related to this pull request. Hence, I cannot successfully build My build configuration is:
I would suggest to simply merge your pull request, although I could not fully test it. |
After I added
So, I will now merge this pull request with some more confidence 👍 |
As the main repository for the librealsense recipe now contains
fixes required for linking catkin-based packages against it
we can stop maintaining our own version of librealsense recipe.
Also this patch adapts realsense-camera recipe to link against
the non-catkin version of librealsense.