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

Package for ROS Lunar #5

Closed
victorpaleologue opened this issue Nov 24, 2017 · 14 comments
Closed

Package for ROS Lunar #5

victorpaleologue opened this issue Nov 24, 2017 · 14 comments
Assignees

Comments

@victorpaleologue
Copy link
Contributor

Requested by @nlyubova, would be useful to researchers using Pepper and NAO

@suryaambrose suryaambrose self-assigned this Nov 25, 2017
@mikaelarguedas
Copy link
Member

This has been released in ROS Lunar by @suryaambrose ros/rosdistro#16469.
This fails to build on all platforms but Ubuntu Xenial (Ubuntu Zesty and Debian Stretch). Is libqi compatible with Boost 1.62?

@victorpaleologue
Copy link
Contributor Author

victorpaleologue commented Nov 27, 2017

The build error is about an ambiguous call to relative(bfs::path, bfs::path).
In libqi's master it was fixed by prefixing it with ::.
I suggest the project pulls the upstream libqi instead of trying this local quick fix.
So it is not a problem with boost, I'm building libqi master with boost 1.64 successfully, for example.

@suryaambrose
Copy link
Member

Hi,

@mikaelarguedas I noticed the issue this morning. I will cherry-pick the fix tonight and see if it is enough.

@vpaleologue-aldebaran I don't want to be based on master branch. I prefer releasing specific tags (easier for me to be certain of what I am releasing). The last libqi tag on github is 2.5.0.176 so that's the one I used (and will keep using until a new one is available).

@suryaambrose
Copy link
Member

But the way @mikaelarguedas the problem should also be present for yaketty (boost 1.61) but there was no complaint. Is that package not built by the buildfarm ?

@victorpaleologue
Copy link
Contributor Author

I think it is best that you take the tag of the latest release on Pepper, which is 2.5.7.1 according to the title of http://doc.aldebaran.com/2-5/home_pepper.html
I tried it here, and libqi builds successfully with boost 1.64.

@victorpaleologue
Copy link
Contributor Author

@mikaelarguedas
Copy link
Member

mikaelarguedas I noticed the issue this morning. I will cherry-pick the fix tonight and see if it is enough.

@suryaambrose Indeed, we now try to closely match EOL dates from Ubuntu and we stopped building binaries for Yakkety this summer: https://discourse.ros.org/t/suspension-of-debian-packaging-for-eol-ubuntu-yakkety/2444.

I think it is best that you take the tag of the latest release on Pepper, which is 2.5.7.1 according to the title of http://doc.aldebaran.com/2-5/home_pepper.html

👍 As long as that version works for all robots (as it seems to be the case based on the provided link)

@suryaambrose
Copy link
Member

But it does not seem to compile with boost 1.58.. (something about an ambiguous call to boost::bind<<nested template f***>>)

I don't have the possibility to build libqi for all ROS supported platforms, nor the will the rework each versions to make it ROS-compliant. I sticked with version 2.5.0 for now, as it needed (hopefully) less man power. On 2.5.7, the CMakeLists.txt has significantly changed, so I need to re-adapt it to catkin, and I'm not really confident about what I am doing.

@mikaelarguedas
Copy link
Member

maybe sticking with 2.5.0 for lunar is fine. In case it's relevant: Melodic (next LTS ROS version coming out in May) will target only platforms that have at least 1.62 so supporting compatibility for all platforms should be easier.

If you happen to reach a state of 2.5.7 working for all platforms targeted by lunar feel free to release another version anytime ;)

@mikaelarguedas
Copy link
Member

Thanks @suryaambrose. looks like the last version compiles on all platforms 🎉
I guess the next step is to release libqicore? and then we should be able to release the ROS naoqi stack right ?

@nlyubova
Copy link
Member

nlyubova commented Dec 7, 2017

Bravo Surya ! Thank you Mikael!

@nlyubova
Copy link
Member

nlyubova commented Dec 7, 2017

@mikaelarguedas does it compile even for ARM64?

@nlyubova
Copy link
Member

nlyubova commented Dec 7, 2017

@suryaambrose if it compile for Kinetic ARM64 then can we release it as well?

@mikaelarguedas
Copy link
Member

@nlyubova yes it compiles on all platforms / architectures.

if it compile for Kinetic ARM64 then can we release it as well?

It is already released on ARM64 for kinetic: http://build.ros.org/view/Kbin_uxv8_uXv8/job/Kbin_uxv8_uXv8__naoqi_libqi__ubuntu_xenial_arm64__binary/

I think that the one that was not building on arm was naoqi_driver (https://github.com/ros-infrastructure/ros_buildfarm_config/blob/d6741493a907480c886dac4954f93da2bfb49651/kinetic/release-xenial-arm64-build.yaml#L20)

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

No branches or pull requests

4 participants