Spec URL: http://tadej.fedorapeople.org/techne.spec SRPM URL: http://tadej.fedorapeople.org/techne-0.2-1.fc13.src.rpm Description: Techne is a general purpose, programmable physical simulator and renderer. It reads in a set of scripts wherein every aspect of a physical system is specified and then proceeds to simulate and render the system onscreen. A showcase for Techne are the Billiards cue sports simulator: http://www.nongnu.org/billiards/ and the Aviation flight simulator: http://www.nongnu.org/aviation/ Techne is a prerequisite for packaging both Billiards and Aviation for Fedora. A few notes to the reviewer: 1) A version of ode >= 0:0.11.1-5 is required by Techne, because it includes the ode-double subpackage (see: https://bugzilla.redhat.com/show_bug.cgi?id=574034) It is included in rawhide, but it is not available in F14 and F13 yet. The koji builds for F14 and F13 are here: http://koji.fedoraproject.org/koji/buildinfo?buildID=204178 (F14) http://koji.fedoraproject.org/koji/taskinfo?taskID=2639094 (F13) 2) Techne 0.2 hasn't been officially released yet, but I'm contact with its author, who provided me with the unofficial tarball. The unofficial tarball is included in the SRPM. 3) rpmlint gives 4 warnings: techne.spec: W: no-cleaning-of-buildroot %clean techne.spec: W: no-buildroot-tag techne.spec: W: no-%clean-section techne.spec: W: invalid-url Source0: http://download.savannah.gnu.org/releases/techne/techne-0.2.tar.gz HTTP Error 404: Not Found The first 3 are invalid, because the Fedora Packaging Guidelines states that BuildRoot tag and %clean section are not required for F13 and above. The last warning is the consequence of Techne 0.2 not being officially released yet.
One thing I forgot to mention, the koji scratch build of Techne against rawhide is available here: https://koji.fedoraproject.org/koji/taskinfo?taskID=2648939
Full review done: Good: - rpmlint checks return: techne.src: W: no-cleaning-of-buildroot %clean techne.src: W: no-buildroot-tag techne.src: W: no-%clean-section techne.src: W: invalid-url Source0: http://download.savannah.gnu.org/releases/techne/techne-0.2.tar.gz HTTP Error 404: Not Found 3 packages and 0 specfiles checked; 0 errors, 4 warnings. The first 3 can be ignored, for the last one see below - package meets naming guidelines - package meets packaging guidelines - license (GPLv3+) OK, text in %doc, matches source - spec file legible, in am. english - package compiles on devel (x86) - no missing BR - no unnecessary BR - no locales - not relocatable - no duplicate files - permissions ok - %clean ok - macro use consistent - code, not content - no need for -docs - nothing in %doc affects runtime - no need for .desktop file MUST FIX / needs work: - sources match upstream I cannot verify this, please ask upstream to put a pre-release / alpha / beta tarbal somewhere. - owns all directories that it creates Currently the package does not own the following dirs: %{_libdir}/%{name} %{_datadir}/%{name} %{_datadir}/%{name}/common This can be fixed by replacing (in %files) this: %{_libdir}/%{name}/*.so %{_libdir}/%{name}/*.bin %{_datadir}/%{name}/common/* With: %{_libdir}/%{name} %{_datadir}/%{name} - release field not properly used. Since this is a pre release version being used the release field should be something like: Release: 0.1%{?dist} or: Release: 0.1.beta1%{?dist}
Hans, thanks for the review! (In reply to comment #2) > MUST FIX / needs work: > - sources match upstream > I cannot verify this, please ask upstream to put a pre-release / alpha > / beta tarbal somewhere. I'm in contact with the upstream author and he'll put a tarball out shortly. > - owns all directories that it creates > Currently the package does not own the following dirs: > %{_libdir}/%{name} > %{_datadir}/%{name} > %{_datadir}/%{name}/common > This can be fixed by replacing (in %files) this: > %{_libdir}/%{name}/*.so > %{_libdir}/%{name}/*.bin > %{_datadir}/%{name}/common/* > With: > %{_libdir}/%{name} > %{_datadir}/%{name} Thanks for pointing this out, fixed. > - release field not properly used. > Since this is a pre release version being used the release field > should be something like: > Release: 0.1%{?dist} > or: > Release: 0.1.beta1%{?dist} Fixed. The updated SPEC and SRPM files are available at: http://tadej.fedorapeople.org/techne.spec http://tadej.fedorapeople.org/techne-0.2-0.2.fc13.src.rpm
(In reply to comment #3) > > MUST FIX / needs work: > > - sources match upstream > > I cannot verify this, please ask upstream to put a pre-release / alpha > > / beta tarbal somewhere. > > I'm in contact with the upstream author and he'll put a tarball out shortly. Techne 0.2 has been officially released and is available from http://download.savannah.gnu.org/releases/techne/techne-0.2.tar.gz I've put the updated SPEC and SRPMS files at: http://tadej.fedorapeople.org/techne.spec http://tadej.fedorapeople.org/techne-0.2-1.fc13.src.rpm
Looks good now, approved!
New Package SCM Request ======================= Package Name: techne Short Description: A general purpose, programmable physical simulator and renderer Owners: tadej Branches: f13 f14 InitialCC:
Git done (by process-git-requests).
techne-0.2-1.fc14,ode-0.11.1-5.fc14 has been submitted as an update for Fedora 14. https://admin.fedoraproject.org/updates/techne-0.2-1.fc14,ode-0.11.1-5.fc14
techne-0.2-1.fc13,ode-0.11.1-5.fc13 has been submitted as an update for Fedora 13. https://admin.fedoraproject.org/updates/techne-0.2-1.fc13,ode-0.11.1-5.fc13
techne-0.2-1.fc14, ode-0.11.1-5.fc14 has been pushed to the Fedora 14 stable repository. If problems still persist, please make note of it in this bug report.
techne-0.2-1.fc13, ode-0.11.1-5.fc13 has been pushed to the Fedora 13 stable repository. If problems still persist, please make note of it in this bug report.