Bug 566281

Summary: Review Request: roboptim-trajectory - Trajectory optimization for robotics
Product: [Fedora] Fedora Reporter: Thomas Moulard <thomas.moulard>
Component: Package ReviewAssignee: Mamoru TASAKA <mtasaka>
Status: CLOSED ERRATA QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: fedora-package-review, mtasaka, notting
Target Milestone: ---Flags: mtasaka: fedora-review+
j: fedora-cvs+
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: roboptim-trajectory-0.5-2.fc11 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2010-02-27 03:43:34 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Thomas Moulard 2010-02-17 19:08:33 UTC
Spec URL: http://homepages.laas.fr/~tmoulard/fedora/roboptim-trajectory.spec
SRPM URL: http://homepages.laas.fr/~tmoulard/fedora/roboptim-trajectory-0.5-1.fc11.src.rpm
Description: The RobOptim trajectory C++ library

Hi! Here is a robotics package I have developed and packaged.

Structure is similar to roboptim-core (review here https://bugzilla.redhat.com/show_bug.cgi?id=538820).

This package depends on roboptim-core and extends it to provide features dealing with trajectory optimization for robotics.


Koji successful build: http://koji.fedoraproject.org/koji/taskinfo?taskID=1994806

$ rpmlint ~/rpmbuild/RPMS/i586/roboptim-trajectory-*.rpm                               
roboptim-trajectory-devel.i586: W: no-documentation
4 packages and 0 specfiles checked; 0 errors, 1 warnings.

Comment 1 Mamoru TASAKA 2010-02-21 18:22:03 UTC
Some notes:

* Requires
  - Would you explain why -doc subpackage has
    "Requires: roboptim-core-doc" ?

* Directory owership issue
  - %{_includedir}/roboptim is already owned by roboptim-core-devel
    and -trajectory-devel requires -core-devel, so -trajectory-devel
    should not own this directory.


! By the way for F-12/11 roboptim-core is still in -testing and
  currently this package cannot be rebuild on F-12/11.
  Would you request to move -core to stable repository?

Comment 2 Thomas Moulard 2010-02-21 18:48:02 UTC
* Requires
  - Would you explain why -doc subpackage has
    "Requires: roboptim-core-doc" ?

It is due to the Doxygen dependency system (two documentations are linked together so even outside classes documentation can be accessed easily).

* Directory owership issue
  - %{_includedir}/roboptim is already owned by roboptim-core-devel
    and -trajectory-devel requires -core-devel, so -trajectory-devel
    should not own this directory.

I will fix that tomorrow.


! By the way for F-12/11 roboptim-core is still in -testing and
  currently this package cannot be rebuild on F-12/11.
  Would you request to move -core to stable repository?

roboptim-core-0.5-2.fc11 and roboptim-core-0.5-2.fc12 have been submitted today.

BTW, when I click on roboptim-core.0.5.2.fc11 (such as here https://admin.fedoraproject.org/updates/user/thomasmoulard), I get the wrong package (rubygem-RedCloth).

Comment 3 Thomas Moulard 2010-02-22 09:56:00 UTC
Spec URL: http://homepages.laas.fr/~tmoulard/fedora/roboptim-trajectory.spec
SRPM URL:
http://homepages.laas.fr/~tmoulard/fedora/roboptim-trajectory-0.5-2.fc11.src.rpm

New version fixes directory ownership.

Thanks for reporting my problem.

Comment 4 Mamoru TASAKA 2010-02-22 15:58:03 UTC
Okay.

---------------------------------------------------------------
  This package (roboptim-trajectory) is APPROVED by mtasaka
---------------------------------------------------------------

Comment 5 Thomas Moulard 2010-02-22 16:20:58 UTC
New Package CVS Request
=======================
Package Name: roboptim-trajectory
Short Description: The RobOptim trajectory C++ library
Owners: thomasmoulard
Branches: F-11 F-12
InitialCC: roboptim-commit.net

Comment 6 Jason Tibbitts 2010-02-22 21:30:30 UTC
The requested package name "roboptim-trajectory" doesn't match what's in the
bug summary, "RobOptim trajectory".  Please verify that "robotrim-trajectory"
is indeed the proper package name and fix the bug summary if so.

Comment 7 Thomas Moulard 2010-02-22 22:52:37 UTC
My bad, fixed right now.

Comment 8 Jason Tibbitts 2010-02-22 23:38:58 UTC
CVS done (by process-cvs-requests.py).

Note that InitialCC cannot be an email address, so I have removed it.

Comment 9 Fedora Update System 2010-02-23 07:41:08 UTC
roboptim-trajectory-0.5-2.fc12 has been submitted as an update for Fedora 12.
http://admin.fedoraproject.org/updates/roboptim-trajectory-0.5-2.fc12

Comment 10 Fedora Update System 2010-02-23 07:42:11 UTC
roboptim-trajectory-0.5-2.fc11 has been submitted as an update for Fedora 11.
http://admin.fedoraproject.org/updates/roboptim-trajectory-0.5-2.fc11

Comment 11 Fedora Update System 2010-02-24 06:15:06 UTC
roboptim-trajectory-0.5-2.fc12 has been pushed to the Fedora 12 testing repository.  If problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update roboptim-trajectory'.  You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F12/FEDORA-2010-2775

Comment 12 Fedora Update System 2010-02-24 06:15:24 UTC
roboptim-trajectory-0.5-2.fc11 has been pushed to the Fedora 11 testing repository.  If problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update roboptim-trajectory'.  You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F11/FEDORA-2010-2781

Comment 13 Fedora Update System 2010-02-27 03:43:27 UTC
roboptim-trajectory-0.5-2.fc12 has been pushed to the Fedora 12 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 14 Fedora Update System 2010-02-27 03:47:33 UTC
roboptim-trajectory-0.5-2.fc11 has been pushed to the Fedora 11 stable repository.  If problems still persist, please make note of it in this bug report.