Bug 699168 - Review Request: routino - Router for OpenStreetMap Data
Summary: Review Request: routino - Router for OpenStreetMap Data
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Volker Fröhlich
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2011-04-23 21:15 UTC by Kevin Kofler
Modified: 2011-04-26 22:03 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2011-04-26 22:03:38 UTC
Type: ---
volker27: fedora-review+
j: fedora-cvs+


Attachments (Terms of Use)

Description Kevin Kofler 2011-04-23 21:15:37 UTC
Spec URL: http://repo.calcforge.org/f16/routino.spec
SRPM URL: http://repo.calcforge.org/f16/routino-1.5.1-1.fc14.src.rpm
Description:
Routino is a command-line application for finding a route between two points
using the dataset of topographical information collected by OpenStreetMap. It
can be used as a routing tool in Marble.


Koji scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=3021403

Testing instructions:

1. Run these commands (WARNING: This takes ~3 hours in total. The download is over 100 MiB (compressed). If you're in a hurry, try something like liechtenstein.osm.bz2. ;-) ):
wget http://download.geofabrik.de/osm/europe/austria.osm.bz2
bunzip2 austria.osm.bz2
mkdir ~/.local/share/marble/maps/earth/routino
routino-planetsplitter --dir=$HOME/.local/share/marble/maps/earth/routino \
  austria.osm

2. In the Marble preferences, in the route planning tab, edit the profiles to disable all the other routing programs (especially the online services), keep only Routino checked.

3. Then use the route planning applet on the left, select 2 places in Austria (or in whatever map section you downloaded and processed in step 1) and a locomotion method and see the route being computed.

Comment 1 Kevin Kofler 2011-04-23 21:16:37 UTC
CCing Volker Fröhlich as a potential reviewer for this package.

Comment 2 Kevin Kofler 2011-04-24 21:00:24 UTC
Spec URL: http://repo.calcforge.org/f16/routino.spec
SRPM URL: http://repo.calcforge.org/f16/routino-1.5.1-2.fc14.src.rpm
Koji scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=3022895

* Sun Apr 24 2011 Kevin Kofler <…> - 1.5.1-2
- Install ChangeLog

(The upstream makefiles install all other docs, but not the license or the changelog. This is probably due to Debian policies, which say not to install those files. In Fedora, we want them installed. I had already fixed it for the license in -1, I also added the changelog in -2.)

Comment 3 Kevin Kofler 2011-04-25 15:51:17 UTC
Spec URL: http://repo.calcforge.org/f16/routino.spec
SRPM URL: http://repo.calcforge.org/f16/routino-1.5.1-3.fc14.src.rpm
Koji scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=3024047

* Mon Apr 25 2011 Kevin Kofler <…> - 1.5.1-3
- Use install -p instead of cp -pf to install executables, to ensure 755 perms

(Volker told me on IRC that the binaries got installed with 775 permissions when building locally with a 022 umask. This should fix that.)

Comment 4 Kevin Kofler 2011-04-25 16:10:16 UTC
> when building locally with a 022 umask

Oops, I meant "when building locally with a 002 umask", as opposed to the 022 umask Koji is applying.

Comment 5 Kevin Kofler 2011-04-25 18:06:27 UTC
Spec URL: http://repo.calcforge.org/f16/routino.spec
SRPM URL: http://repo.calcforge.org/f16/routino-1.5.1-4.fc14.src.rpm
Koji scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=3024477

* Mon Apr 25 2011 Kevin Kofler <…> - 1.5.1-4
- Better Group and BuildRoot tags (Volker Fröhlich)
- Add -b to the patch invocation (suggested by Volker Fröhlich)
- Add a README-MARBLE.txt documenting how to set up Routino for use with Marble
- Get rid of installation documentation which is not applicable to the RPM

Comment 6 Volker Fröhlich 2011-04-25 19:56:21 UTC
[+] Good
[x] Needs work
[0] Does not apply

MUST:
=====

[+] rpmlint:

routino.x86_64: W: spelling-error %description -l en_US dataset -> data set, data-set, database
routino.x86_64: W: invalid-license AGPLv3+
routino.x86_64: W: no-manual-page-for-binary routino-filedumper
routino.x86_64: W: no-manual-page-for-binary routino-planetsplitter
routino.x86_64: W: no-manual-page-for-binary routino-planetsplitter-slim
routino.x86_64: W: no-manual-page-for-binary routino-tagmodifier
routino.x86_64: W: no-manual-page-for-binary routino-router
routino.x86_64: W: no-manual-page-for-binary routino-filedumper-slim
routino.x86_64: W: no-manual-page-for-binary routino-router-slim
routino-debuginfo.x86_64: W: invalid-license AGPLv3+
routino.src: W: spelling-error %description -l en_US dataset -> data set, data-set, database
routino.src: W: invalid-license AGPLv3+
3 packages and 0 specfiles checked; 0 errors, 12 warnings.

Upstream doesn't provide man-pages. AGPLv3+ was forgotten to be listed as a valid license. That's solved in the wiki so far (Spot).

[+] Naming according to the Package Naming Guidelines
[+] Spec file matches base package name
[+] Packaging guidelines met
[+] License approved for Fedora
[+] License field in spec matches code
[+] License file included, if source package includes it
[+] Spec in American English
[+] Spec is legible
[+] Sources match upstream md5sum: a664772cfa7ba413cd16eae59321c644
[+] Compiles and builds into binary RPMs on at least one primary architecture:

See KK above!

[0] ExcludeArch is specified and commented
[0] Locales are handled correctly
[+] All build dependencies listed
[0] Calls ldconfig for its shared libraries
[0] No bundled system libraries
[0] Stated as relocatable package
[+] Owns all its directories or requires package that does
[+] No file listing duplicates
[+] File permissions correct
[+] Consistent use of macros
[+] Code or permissible content
[0] Large documentation in -doc subpackage
[+] No runtime dependency of files listed as %doc
[0] Header files in -devel subpackage
[0] Static files in -static subpackage
[0] Library files without suffix in -devel subpackage
[0] Devel-package requires base package
[0] No .la libtool archives
[0] GUI application includes properly installed %{name}.desktop file
[+] No files or directories owned, that other packages own
[+] Filenames in packages are UTF-8

SHOULD:
=======

[0] Query upstream if no license text is included
[+] Package builds in mock: Rawhide, EPEL 6
[?] Package works as described -- Haven't tried
[0] Scriptlets are sane, if used
[0] Subpackages other than -devel should require base package (versioned)
[0] pkgconfig files in -devel subpackage
[0] Dependencies outside of /etc, /bin, /sbin, /usr/bin, or /usr/sbin consider
requiring the package which provides the file instead of the file itself
[x] Contain man pages, where they make sense

Probably query upstream on this issue.

--------
APPROVED
--------

Comment 7 Kevin Kofler 2011-04-25 21:35:52 UTC
FYI, I've tested the package and it works for me.

Comment 8 Kevin Kofler 2011-04-25 21:41:32 UTC
New Package SCM Request
=======================
Package Name: routino
Short Description: Router for OpenStreetMap Data
Owners: kkofler
Branches: f15 f14 f13
InitialCC:

Volker, if you want to request branches for el6 and possibly el5 with you as the owner, feel free, that is hereby approved by me.

Comment 9 Volker Fröhlich 2011-04-25 22:02:47 UTC
Package Change Request
======================
Package Name: routino
New Branches: el6
Owners: volter

Comment 10 Jason Tibbitts 2011-04-26 02:39:33 UTC
This package does not appear to be in pkgdb currently; we can't add an EL6
branch for it.

Comment 11 Kevin Kofler 2011-04-26 02:43:30 UTC
That's because the comment above requests the creation of the package. :-)

Comment 12 Jason Tibbitts 2011-04-26 15:39:09 UTC
These things are processed by scripts.  The scripts look only at the last
request in the ticket.

Comment 13 Kevin Kofler 2011-04-26 15:42:40 UTC
New Package SCM Request
=======================
Package Name: routino
Short Description: Router for OpenStreetMap Data
Owners: kkofler
Branches: f15 f14 f13 el6
InitialCC:

We can reassign ownership in pkgdb afterwards…

Comment 14 Jason Tibbitts 2011-04-26 19:35:49 UTC
Git done (by process-git-requests).

Comment 15 Kevin Kofler 2011-04-26 22:03:38 UTC
Imported, built for Rawhide, builds and Bodhi updates for the releases coming soon.


Note You need to log in before you can comment on or make changes to this bug.