-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
Add rosdep key for gazebo7 in Xenial #11288
Conversation
If you do it like this then you also need an entry for Wily. |
Mmkay. It looks like gazebo5 is the version in Wily: https://launchpad.net/ubuntu/wily/+source/gazebo |
We've already discussed the fact that a newer version will have to be back ported: ros-infrastructure/rep#106 (comment) |
ahh, ok thanks. I'll change it |
We also need |
2f39a49 adds the libgazebo7-dev key. |
Fedora doesn't have gazebo7 yet: cc @cottsay @jpgr87 |
indeed... https://bugzilla.redhat.com/show_bug.cgi?id=1247414 I propose I remove the Fedora entry and add Fedora support later. We need this to start pull request testing gazebo_ros_pkgs on the buildfarm, which doesn't build for Fedora anyway. Does that sound ok? |
that's what we did for FCL in #10874, sounds ok to me |
Gazebo 7 isn't in Debian Jessie yet, either. Not sure what to do about that (@tfoote, is Gazebo important to you for the Turtlebot stack? I assume you're prioritizing the on-robot software...). |
Jessie is a released distribution so it won't be in the official distribution. |
@@ -1509,6 +1511,10 @@ libgazebo5-dev: | |||
fedora: [gazebo-devel] | |||
gentoo: [sci-electronics/gazebo] | |||
ubuntu: [libgazebo5-dev] | |||
libgazebo7-dev: | |||
arch: [gazebo] | |||
gentoo: [sci-electronics/gazebo] |
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.
I'd remove these since we can't ensure they are explicitly Gazebo 7.
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.
Done.
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.
The Gentoo friends are updated with gazebo versions. It lives in the unstable amd64 branch (note the ~amd64 mark).
The Arch Linux friends are a bit behind but also using gazebo7 versions.
These distributions don't add a numbered version to the package but both have gazebo7 available.
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.
Is there a way to ensure it's gazebo7 though? If you depend on libgazebo7-dev
, then rosdep (and by translation the package manager for the OS) better give you version 7. It would be fine to have them for the gazebo
key.
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.
Is there a way to ensure it's gazebo7 though?
In Gentoo, just by using the package name in the installation command, probably no. You can run emerge =sci-electronics/gazebo=7.1.0
but this will be broken as soon as they update the package. Fine grained version election is done in other configuration files.
In Arch the situation is the same AFAIK.
It could be backported or put into a ROS PPA equivalent. I'm not sure what they've done in the past. |
Gazebo is not present into any released Debian distro. it has been present in current testing (will be the next released distro) and unstable (rolling). |
We can probably get Gazebo 7 in Fedora 24 (due out in early June), and definitely get it in Fedora 25+. We stuck with Gazebo 6 in the current releases so we didn't get too far ahead of the Ubuntu packages, but if there's a case for Gazebo 7 in the F24 time frame we can try to make that happen. |
@scpeters or @j-rivero or whoever--does this look correct to you?