Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
Summary: Review Request: gazebo - 3D multiple robot simulator with dynamics
https://bugzilla.redhat.com/show_bug.cgi?id=725752
Summary: Review Request: gazebo - 3D multiple robot simulator with dynamics Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: unspecified Component: Package Review AssignedTo: nobody@fedoraproject.org ReportedBy: sjt@roamingzebra.co.uk QAContact: extras-qa@fedoraproject.org CC: notting@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Type: ---
Spec URL: http://www.cs.bham.ac.uk/~sjt/rpm/gazebo.spec SRPM URL: http://www.cs.bham.ac.uk/~sjt/rpm/gazebo-0.9.0-4.el6.src.rpm Description: Gazebo is a multi-robot simulator for outdoor environments. Like Stage, it is capable of simulating a population of robots, sensors and objects, but does so in a three-dimensional world. It generates both realistic sensor feedback and physically plausible interactions between objects (it includes an accurate simulation of rigid-body physics).
(I'm actually building on an SL6 machine with some extra packages from Fedora which aren't in EPEL, but should build on Fedora).
Note: this is my first package, so I am seeking a sponsor.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=725752
Simon Thompson sjt@roamingzebra.co.uk changed:
What |Removed |Added ---------------------------------------------------------------------------- Blocks| |177841(FE-NEEDSPONSOR)
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=725752
--- Comment #1 from Simon Thompson sjt@roamingzebra.co.uk 2011-07-26 09:34:42 EDT --- I'll add a pre-emptive note on why its 0.9.0 rather than the latest 0.10.0 release. According to one of our researchers who works in robotics:
"0.10.0 is not really very stable, as it is the first release after a major internal code restructure"
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=725752
Rich Mattes richmattes@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |richmattes@gmail.com
--- Comment #2 from Rich Mattes richmattes@gmail.com 2011-07-29 22:55:40 EDT --- I have a couple of comments for you. I'm not a sponsor, so I can't formally review this package, but I would like to help you along.
1) Gazebo bundles assimp, which is against the packaging guidelines at [1]. You'll have to remove assimp and use a system version, which is under review as bug 635511.
2) Fedora packages can't depend on packages from external sources. The ffmpeg-devel requirement cannot be satisfied by any Fedora packages. You'll have to build gazebo without ffmpeg support, somehow make ffmpeg a runtime requirement as opposed to a build time requirement, or submit gazebo to a third-party repository for consideration.
[1] http://fedoraproject.org/wiki/Packaging:No_Bundled_Libraries
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=725752
Denis Arnaud denis.arnaud_fedora@m4x.org changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |denis.arnaud_fedora@m4x.org
--- Comment #3 from Denis Arnaud denis.arnaud_fedora@m4x.org 2011-07-30 09:25:49 EDT --- (In reply to comment #2)
I have a couple of comments for you. I'm not a sponsor, so I can't formally review this package, but I would like to help you along.
Hi Rich! You can do a formal review. The only thing you cannot do is sponsoring Simon.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=725752
--- Comment #4 from Simon Thompson sjt@roamingzebra.co.uk 2011-08-01 05:25:22 EDT --- Thanks for taking a look.
As far as I can see, 0.9.0 (which is what I've packaged) of Gazebo doesn't include assimp - 0.10.0 does.
Maybe there's a debate to be had around 0.9.0 vs 0.10.0 though? I noted in comment #1 that its 0.9.0 rather than the latest release. That's based on the comments from some of the researchers I work with who use this stuff.
I'll take a look at the ffmpeg requirement and see if I can resolve that.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=725752
--- Comment #5 from Simon Thompson sjt@roamingzebra.co.uk 2011-08-01 08:08:31 EDT ---
I've changed it so that its possible to build with ffmpeg with "with ffmpeg", but that is disabled by default. It means there's no audio support for gazebo now.
Spec URL: http://www.cs.bham.ac.uk/~sjt/rpm/gazebo-0.9.0-5.spec SRPM URL: http://www.cs.bham.ac.uk/~sjt/rpm/gazebo-0.9.0-5.el6.src.rpm
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=725752
Simon Thompson sjt@roamingzebra.co.uk changed:
What |Removed |Added ---------------------------------------------------------------------------- Depends on| |635511
--- Comment #6 from Simon Thompson sjt@roamingzebra.co.uk 2011-08-11 07:23:27 EDT ---
I found that 0.9.0 wouldn't build against OGRE 1.7.x so have bumped version to 0.10.0.
Spec URL: http://www.cs.bham.ac.uk/~sjt/rpm/gazebo-0.10.0-1.spec SRPM URL: http://www.cs.bham.ac.uk/~sjt/rpm/gazebo-0.10.0-1.el6.src.rpm
Its patched to use the external assimp library. There's options for --with ffmpeg and --with bullet - it currently won't build with the Fedora bullet package though as it requires changes: OPTION(USE_DOUBLE_PRECISION "Use double precision" ON) OPTION(INSTALL_LIBS "Set when you want to install libraries" ON) in bullet's CMakeLists.txt
There's a couple of patches in there to make it build and to install unversioned .so files into /usr/lib/package style as well a patch to make it work on x86_64 style machines (based on the architecture detection used in player) so that lib64 is substituted appropriately.
Adding bug 635511 (assimp) as blocking this.
Any comments on this?
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=725752
--- Comment #7 from Rich Mattes richmattes@gmail.com 2011-12-19 20:45:08 EST --- Created attachment 548728 --> https://bugzilla.redhat.com/attachment.cgi?id=548728 Buildsystem fixes for 1.0.0-RC1 on f16
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=725752
--- Comment #8 from Rich Mattes richmattes@gmail.com 2011-12-19 20:45:49 EST --- Gazebo 1.0.0-RC1 has been released at gazebosim.org. It looks like the 1.0 release doesn't have any dependencies on bullet or assimp at the moment. It's not too hard to get to build, I've attached a patch that covers all of the stuff that needed to be changed for f16. The library install directories need to be fixed again.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=725752
--- Comment #9 from Rich Mattes richmattes@gmail.com 2012-01-14 13:45:00 EST --- It's been a little while without any reply, are you still interested in completing this review? If not, I can re-submit the review request and take it over.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=725752
Bug 725752 depends on bug 635511, which changed state.
Bug 635511 Summary: Review Request: assimp - Library to import various 3D model formats into applications https://bugzilla.redhat.com/show_bug.cgi?id=635511
What |Old Value |New Value ---------------------------------------------------------------------------- Status|NEW |MODIFIED Status|MODIFIED |ON_QA Resolution| |ERRATA Status|ON_QA |CLOSED
--- Comment #10 from Rich Mattes richmattes@gmail.com 2012-04-27 19:11:00 EDT --- Are you still interested in finishing this review? I've been working on gazebo-1.0.0 spec. I'm happy to share my spec file with you, or submit it myself if you'd prefer.
https://bugzilla.redhat.com/show_bug.cgi?id=725752
Rich Mattes richmattes@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |CLOSED Resolution|--- |DUPLICATE Last Closed| |2012-05-25 20:56:30
--- Comment #11 from Rich Mattes richmattes@gmail.com ---
*** This bug has been marked as a duplicate of bug 825409 ***
https://bugzilla.redhat.com/show_bug.cgi?id=725752
Björn "besser82" Esser bjoern.esser@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Blocks|177841 (FE-NEEDSPONSOR) |
Referenced Bugs:
https://bugzilla.redhat.com/show_bug.cgi?id=177841 [Bug 177841] Tracker: Review requests from new Fedora packagers who need a sponsor
package-review@lists.fedoraproject.org