Spec URL: http://bruce89.fedorapeople.org/mkgmap.spec SRPM URL: http://bruce89.fedorapeople.org/mkgmap-0-0.1.1647svn.fc13.src.rpm Description: I've just packaged mkgmap. This is a reqest for a review. Also, this is my first package, so I also looking for a sponsor. mkgmap is a Java program to convert OSM data for Garmin GPSes.
Builds on koji http://koji.fedoraproject.org/koji/taskinfo?taskID=2201997
Hi Bruce, I just took a quick look. I wonder if you want to update to the latest svn? Also, I wonder if you want to add the ChangeLog to %doc? I am happy to review the package further but I am unable to sponsor.
I'll get to that probably tomorrow, I've been on holiday the last week.
I've updated the package, spec at http://bruce89.fedorapeople.org/mkgmap.spec; srpm at http://bruce89.fedorapeople.org/mkgmap-0-0.2.1653svn.fc13.src.rpm
Bruce, did you perform some unofficial reviews?
Not yet, but I will do so soon.
The spec is now at http://bruce89.fedorapeople.org/specs/mkgmap.spec and the srpm is now at http://bruce89.fedorapeople.org/rpms/mkgmap-0-0.3.1658svn.fc13.src.rpm.
Yet another update, srpm at http://bruce89.fedorapeople.org/rpms/mkgmap-0-0.4.1669svn.fc14.noarch.rpm, scratch build at http://koji.fedoraproject.org/koji/taskinfo?taskID=2413478
The last update isn't a source rpm, it's the already-built package. Since the scratch build has already been garbage collected, there's no way we can access it.
Whoops. New spec at http://bruce89.fedorapeople.org/specs/mkgmap.spec, srpm is at http://bruce89.fedorapeople.org/rpms/mkgmap-0-0.6.1728svn.fc14.src.rpm. Scratch build http://koji.fedoraproject.org/koji/taskinfo?taskID=2612440.
Unfortunately the build fails for me, but I don't know enough about Java to say why. compile-pbf: build: [copy] Copying 388 files to /builddir/build/BUILD/mkgmap-r1728/build/classes dist: [mkdir] Created dir: /builddir/build/BUILD/mkgmap-r1728/dist [mkdir] Created dir: /builddir/build/BUILD/mkgmap-r1728/dist/doc/api [jar] Building jar: /builddir/build/BUILD/mkgmap-r1728/dist/mkgmap.jar [copy] Copying 18 files to /builddir/build/BUILD/mkgmap-r1728/dist/doc BUILD FAILED /builddir/build/BUILD/mkgmap-r1728/build.xml:207: Warning: Could not find resource file "/opt/jars/protobuf-2.3.0/protobuf.jar" to copy. Total time: 4 seconds A scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=2617989
Bruce, do you still need sponsorship?
A quick ".fasinfo bruce89" on IRC shows that he's not in the packager group and so requires a sponsor.
Yes, I'm looking for a sponsor. I also have another package request that I should file some time soon, and I will do some reviews as well.
I'll do this one.
Bruce, your package really tries to use some jars from /opt and they are not there on a stock Fedora system. There is protobuf-java package please make use of it to fix the build.
According to http://www.mkgmap.org.uk/pipermail/mkgmap-dev/2010q4/009248.html, mkgmap requires protobuf.jar (which is in protobuf-java), and also osmprotobuf.jar (which is not packaged). Apparently osmprotobuf is in Osmosis.
Looking at the huge numbers of jars that osmosis bundles, I suspect this going to be far too difficult for me.
Sorry for the spam, but I've found (thanks to #osm on OFTC) the source for osmprotobuf.jar, which is https://github.com/scrosby/OSM-binary. However, I notice that there is no licence for this code. I will e-mail the author to ask him to add/clarify the licence.
Please note that you will have to open a new bugreport for the osmprotobuf as it is a different project released separately. And make the new bug report blocks this one.
Due to having not received a reply yet, not knowing if that source I found will even build, and that I no longer use a Garmin GPS, I'm putting this review on hold.
(In reply to comment #21) > Due to having not received a reply yet, not knowing if that source I found will > even build, and that I no longer use a Garmin GPS, I'm putting this review on > hold. Sorry but I missed your quetion. We can not accept any package with unknown license. Even if there is no license file it would have been ok if the source files contained license headers which they don't.
Sorry, that was badly phrased. I know about Fedora's policy about licences, and I respect that. I was just saying that I have no reply to the e-mail I sent to the author to ask about the licence yet, and so can't work on this at the moment.
(In reply to comment #17) > According to http://www.mkgmap.org.uk/pipermail/mkgmap-dev/2010q4/009248.html, > mkgmap requires protobuf.jar (which is in protobuf-java), and also > osmprotobuf.jar (which is not packaged). Apparently osmprotobuf is in Osmosis. I want to clarify that the protobuf jars enable optional functionality and are not required (any longer) to build mkgmap - the referenced message from the mailing list is no longer relevant as the build process was changed shortly after. Now the build file will detect that the jars are not present and simply not build the protobuf format plugin. Reading protobuf is not particularly useful at the current time as in most cases you will be reading files produced by the splitter program which does not currently write in the protobuf based format yet. So I think that it is entirely acceptable to build without protobuf support until such time as the osmprotobuf jars are more easily available.
(In reply to comment #24) > I want to clarify that the protobuf jars enable optional functionality and > are not required (any longer) to build mkgmap - the referenced message from the > mailing list is no longer relevant as the build process was changed shortly > after. Now the build file will detect that the jars > are not present and simply not build the protobuf format plugin. > > Reading protobuf is not particularly useful at the current time as in most > cases > you will be reading files produced by the splitter program which does not > currently write in the protobuf based format yet. > > So I think that it is entirely acceptable to build without protobuf support > until such time as the osmprotobuf jars are more easily available. Thank you for clarifiying this. I have updated to r1760. Spec : http://bruce89.fedorapeople.org/specs/mkgmap.spec SRPM: http://bruce89.fedorapeople.org/rpms/mkgmap-0-0.7.1760svn.fc14.src.rpm
Build still fails. BUILD FAILED /home/akurtakov/work/rpmeditor-demo/mkgmap/BUILD/mkgmap-r1760/build.xml:207: Warning: Could not find resource file "/opt/jars/protobuf-2.3.0/protobuf.jar" to copy. Total time: 9 seconds
Bruce, I'm closing this bug now. Please reopen once you gain interest again.