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

Add rosdep key for gazebo7 in Xenial #11288

Merged
merged 6 commits into from
Apr 26, 2016
Merged

Add rosdep key for gazebo7 in Xenial #11288

merged 6 commits into from
Apr 26, 2016

Conversation

jacquelinekay
Copy link
Contributor

@scpeters or @j-rivero or whoever--does this look correct to you?

@wjwwood
Copy link
Member

wjwwood commented Apr 25, 2016

If you do it like this then you also need an entry for Wily.

@jacquelinekay
Copy link
Contributor Author

Mmkay. It looks like gazebo5 is the version in Wily: https://launchpad.net/ubuntu/wily/+source/gazebo

@wjwwood
Copy link
Member

wjwwood commented Apr 25, 2016

We've already discussed the fact that a newer version will have to be back ported: ros-infrastructure/rep#106 (comment)

@jacquelinekay
Copy link
Contributor Author

ahh, ok thanks. I'll change it

@scpeters
Copy link
Contributor

We also need libgazebo7-dev. Should we make new keys or make the gazebo key point to different versions of gazebo? For jade, we made new keys (#8243). Also, I think wily has gazebo6, but @j-rivero is working on back-porting the gazebo7 from xenial to wily.

@jacquelinekay
Copy link
Contributor Author

2f39a49 adds the libgazebo7-dev key.

@scpeters
Copy link
Contributor

Fedora doesn't have gazebo7 yet:

cc @cottsay @jpgr87

@jacquelinekay
Copy link
Contributor Author

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?

@scpeters
Copy link
Contributor

that's what we did for FCL in #10874, sounds ok to me

@jacquelinekay
Copy link
Contributor Author

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...).

@j-rivero
Copy link
Contributor

Gazebo 7 isn't in Debian Jessie yet,

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]
Copy link
Member

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.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done.

Copy link
Contributor

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.

Copy link
Member

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.

Copy link
Contributor

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.

@wjwwood
Copy link
Member

wjwwood commented Apr 26, 2016

Jessie is a released distribution so it won't be in the official distribution.

It could be backported or put into a ROS PPA equivalent. I'm not sure what they've done in the past.

@wjwwood wjwwood merged commit 3b86bad into master Apr 26, 2016
@wjwwood wjwwood deleted the gazebo7 branch April 26, 2016 00:34
@j-rivero
Copy link
Contributor

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).

@richmattes
Copy link
Contributor

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.

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

Successfully merging this pull request may close these issues.

5 participants