Red Hat Bugzilla – Bug 476440
Review Request: latexdiff - Determine and mark up significant differences between latex files
Last modified: 2009-02-04 21:11:36 EST
Spec URL: http://www.cs.technion.ac.il/~danken/latexdiff.spec
SRPM URL: http://www.cs.technion.ac.il/~danken/latexdiff-0.20071020-1.fc10.src.rpm
Description: Latexdiff is a Perl script for visual mark up and revision of significant differences between two latex files. Various options are available for visual markup using standard latex packages such as color. Changes not directly affecting visible text, for example in formatting commands, are still marked in the latex source. A rudimentary revision facility is provided by another Perl script, latexrevise, which accepts or rejects all changes. Manual editing of the difference file can be used to override this default behaviour and accept or reject selected changes only.
I'd say that the version number is not correct. Nowhere in the scripts or on the website do I see any reference to 20071020. OTOH there are references as "Version 0.42 November 06 Bug fixes only" (in the script) and "Version
0.5" (on the main web page of the project)
License is not correct. The website claims GPLv2+ but the scripts themselves include:
# This program is free software; you can redistribute it and/or modify
# it under the terms of the GNU General Public License Version 2 as published by
# the Free Software Foundation.
This clearly makes them GPLv2 (not GPLv2+)
Thanks for the review.
As I understand it, "version 0.42" in latexdiff and "version 0.3" in latexrevise relate to the versions of the separate scripts. Since I did not find any version number for the whole package, I chose the latest update date of files within.
Do you have a better idea (maybe 0.0.20071020, to let upstream choose 0.43 one day in the future ?)
See http://www.cs.technion.ac.il/~danken/latexdiff-0.0.20071020-1.fc10.src.rpm and http://www.cs.technion.ac.il/~danken/latexdiff.spec for updated version and license.
I think it would be better to use the pre-release notation (see kismet in ). By the way, your SRPM link is broken.
That's fine by me
* On the website
I see that version is 0.5
As you said there are different version references within the source. I would try to get in touch with the author to ask about the correct version.
I think it should be OK to go with the version of the latexdiff script for the whole package, because the whole package is named after that script.
* It looks like this package must be noarch.
* rpmlint gives bunch of errors of these types:
They are all easy to fix.
? Why don't you use the existing Makefile facility?
* I think all of doc/support/latexdiff/* files should go to %doc
? The source/latexdiff/example/ can go to %doc too. What do you think?
? Is the wrapper script source/latexdiff/contrib/latexdiff-wrap useful? Should we package it?
* We prefer %defattr(-,root,root,-)
* I don't think
is necessary. rpmbuild itself picks up the perl dependencies. (check with "rpm -qpR <package.rpm>")
Ok, let's use version 0.5.
I hope I got all your (*)s sorted. rpm passes rpmlint cleanly now.
I did not see how the supplied Makefile could work. It relates to non-existing filenames and ignores the tarball directpry structure. avoiding seemed simpler than fixing it.
using latexdiff is so simple that I don't think the example is really useful.
I have never used latexdiff-wrap, and not anxious to try.
Sorry, I've been away for a while. Now I'm back and we can finish up this beast. Here are a few more issues to address.
* Please add the changes you made to the changelog in the future with the correct date(s). It'll be easier to see what is actually changed. That's why we keep a changelog.
* Also please switch to %defattr(-,root,root,-)
* We should preserve the timestamps of all the relevant files we package in the %install section. By relevant files, I mean the non-compiled files.
* Please remove the commented-out line(s) that are unnecessary. Also, in the comments and in the changelog section, we use %% for macros instead of a single %, so that the macros don't expand out during package building.
done. see http://www.cs.technion.ac.il/~danken/latexdiff.spec
This package (latexdiff) is APPROVED by oget
Please supply the SRPM (in addition to the SPEC) whenever you make an update during the review process in the future. It makes life easier for reviewers.
New Package CVS Request
Package Name: latexdiff
Short Description: Determine and mark up significant differences between latex files
latexdiff-0.5-2.fc10 has been submitted as an update for Fedora 10.
latexdiff-0.5-2.fc10 has been pushed to the Fedora 10 stable repository. If problems still persist, please make note of it in this bug report.