Spec URL: http://oget.fedorapeople.org/review/rumor.spec SRPM URL: http://oget.fedorapeople.org/review/rumor-1.0.3b-1.fc10.src.rpm Description: Rumor is a realtime monophonic (with chords) MIDI keyboard to Lilypond converter. It receives MIDI events, quantizes them according to its metronome on the fly and outputs handwritten-like corresponding Lilypond notation. Tempo, meter, key and other parameters can be set via command-line options. rpmlint is silent Koji F-11 build: http://koji.fedoraproject.org/koji/taskinfo?taskID=1241995
Hi Orcan, here is my official review, there are only 2 very minor issues: * rpmlint: TODO rpmlint SPECS/rumor.spec RPMS/i386/rumor-* SRPMS/rumor-1.0.3b-1.fc10.src.rpm SPECS/rumor.spec: W: mixed-use-of-spaces-and-tabs (spaces: line 12, tab: line 1) rumor.src: W: mixed-use-of-spaces-and-tabs (spaces: line 12, tab: line 1) 3 packages and 1 specfiles checked; 0 errors, 2 warnings. - just a very minor mixture of spaces and tabs * spec file name: OK * Source0: OK - spectool -g works - URL ok - sources matches upstream: md5sum c95917356659b77bb83505cb2512ad1b rumor-1.0.3b.tar.bz2 * package name: OK, matches upstream * license: OK - GPLv2 is acceptable - rumor.cc explicitely states that "version 2 as published by the Free Software Foundation" should be used - license field matches actual license - COPYING packaged * spec file legible and in American English: OK * package compiles: OK - mock - koji in F9, F10 and F11: https://koji.fedoraproject.org/koji/taskinfo?taskID=1251851 https://koji.fedoraproject.org/koji/taskinfo?taskID=1251856 https://koji.fedoraproject.org/koji/taskinfo?taskID=1251861 * BuildRequires: OK * locale handling: OK (no locale available) * shared libs handling: OK (no shared libs) * directory ownership: OK - no directories created * debuginfo: OK - seems to be complete, debuginfo available in gdb for rumor * files not listed twice: OK * optflags: TODO - I would delete the -O0 in the configure file completely and not substitute it with -O2. If Fedora changes the rpmoptflags, there will be two different -Ox options (the one from rpmoptflags and the one from the configure) (e.g. "sed -i 's|-O0||' configure" could be used) - the only uncommon gcc option is "-ggdb3", but that's no problem, it will only add some more info to the debug infos (e.g. macros) - gdb works still fine with rumor's debuginfo * %clean and rm -rf in %install: OK * file perms: OK * %defattr used: OK * macro usage: OK * code vs. content: OK - only code * large docu in sub-package: OK (n/a) * *.la files, header, static/dynamic libs, pkgconfig: OK (n/a) * GUI application needs desktop file: OK (not a GUI app) * file names UTF8: OK * basic features: OK - I've used rumor together with frescobaldi to record some notes from a keyboard attached via MIDI. It worked without any problems. * scriptlets: OK - texinfo: Requires ok - scriptlets for texinfo files correct - /usr/share/info/dir not part of the package Best regards, Christian
(In reply to comment #1) > * rpmlint: TODO > rpmlint SPECS/rumor.spec RPMS/i386/rumor-* SRPMS/rumor-1.0.3b-1.fc10.src.rpm > SPECS/rumor.spec: W: mixed-use-of-spaces-and-tabs (spaces: line 12, tab: line > 1) > rumor.src: W: mixed-use-of-spaces-and-tabs (spaces: line 12, tab: line 1) > 3 packages and 1 specfiles checked; 0 errors, 2 warnings. > - just a very minor mixture of spaces and tabs > I wonder how this rpmlint survived all my checks. Fixed! > * optflags: TODO > - I would delete the -O0 in the configure file completely and not substitute it > with -O2. If Fedora changes the rpmoptflags, there will be two different -Ox > options (the one from rpmoptflags and the one from the configure) (e.g. "sed -i > 's|-O0||' configure" could be used) > - the only uncommon gcc option is "-ggdb3", but that's no problem, it will only > add some more info to the debug infos (e.g. macros) - gdb works still fine with > rumor's debuginfo > Good point. Thanks for bringing this up. Spec URL: http://oget.fedorapeople.org/review/rumor.spec SRPM URL: http://oget.fedorapeople.org/review/rumor-1.0.3b-2.fc10.src.rpm Thank you Christian, once more :)
all mentioned issues are addressed in the new package: - compiler optimization flags completely removed, so that only the ones from the rpmoptflags are used: g++ -DLOCALEDIR=\"/usr/share/locale\" -DHAVE_CONFIG_H -I. -I. -I.. -O2 -g -pipe -Wall -Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fstack-protector --param=ssp-buffer-size=4 -m32 -march=i386 -mtune=generic -fasynchronous-unwind-tables -D_REENTRANT -ggdb3 -Wall -pedantic -MT options.o -MD -MP -MF ".deps/options.Tpo" \ -c -o options.o - mixes spaces/tabs in spec file fixed: rpmlint SPECS/rumor.spec RPMS/i386/rumor-* SRPMS/rumor-1.0.3b-1.fc10.src.rpm 5 packages and 1 specfiles checked; 0 errors, 0 warnings. APPROVED.
Thanks a lot! New Package CVS Request ======================= Package Name: rumor Short Description: Really Unintelligent Music transcriptOR Owners: oget Branches: F-9 F-10 InitialCC:
CVS Done
rumor-1.0.3b-2.fc10 has been submitted as an update for Fedora 10. http://admin.fedoraproject.org/updates/rumor-1.0.3b-2.fc10
rumor-1.0.3b-2.fc10 has been pushed to the Fedora 10 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 rumor'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F10/FEDORA-2009-3182
rumor-1.0.3b-2.fc10 has been pushed to the Fedora 10 stable repository. If problems still persist, please make note of it in this bug report.