https://bugzilla.redhat.com/show_bug.cgi?id=866358
Bug ID: 866358 QA Contact: extras-qa@fedoraproject.org Severity: unspecified Version: rawhide Priority: unspecified CC: notting@redhat.com, package-review@lists.fedoraproject.org Assignee: nobody@fedoraproject.org Summary: Review Request: ams - ALSA Modular synth Regression: --- Story Points: --- Classification: Fedora OS: Unspecified Reporter: brendan.jones.it@gmail.com Type: Bug Documentation: --- Hardware: Unspecified Mount Type: --- Status: NEW Component: Package Review Product: Fedora
ams is a virtual modular synthesizer for JACK/ALSA.
This package was previuously hosted in the Planet CCRMA repository and is a requirement for the Fedora Audio spin.
SPEC: http://bsjones.fedorapeople.org/ams.spec SPEC: http://bsjones.fedorapeople.org/ams-2.0.1-5.fc18.src.rpm
Upstream has been notified of FSF address changes
rpmlint /home/bsjones/rpmbuild/SRPMS/ams-2.0.1-5.fc18.src.rpm /home/bsjones/rpmbuild/RPMS/x86_64/ams-2.0.1-5.fc18.x86_64.rpm /home/bsjones/rpmbuild/RPMS/x86_64/ams-debuginfo-2.0.1-5.fc18.x86_64.rpm ams.src: W: spelling-error Summary(en_US) Alsa -> Alas, Ala, Alisa ams.src: W: spelling-error Summary(en_US) realtime -> mealtime, real time, real-time ams.src: W: spelling-error %description -l en_US realtime -> mealtime, real time, real-time ams.src: W: spelling-error %description -l en_US usr -> use, us, user ams.x86_64: W: spelling-error Summary(en_US) Alsa -> Alas, Ala, Alisa ams.x86_64: W: spelling-error Summary(en_US) realtime -> mealtime, real time, real-time ams.x86_64: W: spelling-error %description -l en_US realtime -> mealtime, real time, real-time ams.x86_64: E: incorrect-fsf-address /usr/share/doc/ams-2.0.1/COPYING 3 packages and 0 specfiles checked; 1 errors, 7 warnings.
https://bugzilla.redhat.com/show_bug.cgi?id=866358
Brendan Jones brendan.jones.it@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Blocks| |805236 (FedoraAudio)
https://bugzilla.redhat.com/show_bug.cgi?id=866358
Ankur Sinha (FranciscoD) sanjay.ankur@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED CC| |sanjay.ankur@gmail.com Assignee|nobody@fedoraproject.org |sanjay.ankur@gmail.com Flags| |fedora-review?
--- Comment #1 from Ankur Sinha (FranciscoD) sanjay.ankur@gmail.com --- I'll review this one.
https://bugzilla.redhat.com/show_bug.cgi?id=866358
Ankur Sinha (FranciscoD) sanjay.ankur@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flags|fedora-review? | Flags| |fedora-review+
--- Comment #2 from Ankur Sinha (FranciscoD) sanjay.ankur@gmail.com --- Hi!
REVIEW:
[+] OK [-] NA [?] Issue
[+] Package meets naming and packaging guidelines [+] Spec file matches base package name. [+] Spec has consistant macro usage. [+] Meets Packaging Guidelines. [+] License [+] License field in spec matches [+] License file included in package [+] Spec in American English [+] Spec is legible. [+] Sources match upstream md5sum: [ankur@ankur SPECS]$ review-md5check.sh ams.spec Getting http://downloads.sourceforge.net/project/alsamodular/alsamodular/2.0.1/ams-2... to /tmp/review/ams-2.0.1.tar.bz2 % Total % Received % Xferd Average Speed Time Time Time Current Dload Upload Total Spent Left Speed 0 0 0 0 0 0 0 0 --:--:-- 0:00:01 --:--:-- 0 100 279k 100 279k 0 0 195k 0 0:00:01 0:00:01 --:--:-- 195k 0d41bd5aac066aa98be45fd7ab12d35f /tmp/review/ams-2.0.1.tar.bz2 0d41bd5aac066aa98be45fd7ab12d35f /home/ankur/rpmbuild/SOURCES/ams-2.0.1.tar.bz2 removed `/tmp/review/ams-2.0.1.tar.bz2' removed directory: `/tmp/review' [ankur@ankur SPECS]$
[-] Package needs ExcludeArch [+] BuildRequires correct [-] Spec handles locales/find_lang [-] Package is relocatable and has a reason to be. [+] Package is code or permissible content. [-] Doc subpackage needed/used. [+] Packages %doc files don't affect runtime.
[-] Headers/static libs in -devel subpackage. [-] Spec has needed ldconfig in post and postun [-] .pc files in -devel subpackage/requires pkgconfig [-] .so files in -devel subpackage. [-] -devel package Requires: %{name} = %{version}-%{release} [-] .la files are removed.
[+] Package is a GUI app and has a .desktop file
[+] Package compiles and builds on at least one arch. [+] Package has no duplicate files in %files. [+] Package doesn't own any directories other packages own. [+] Package owns all the directories it creates. [+] No rpmlint output. [ankur@ankur SRPMS]$ rpmlint ../SPECS/ams.spec ./ams-2.0.1-5.fc18.src.rpm /var/lib/mock/fedora-rawhide-x86_64/result/*.rpm ams.src: W: spelling-error Summary(en_US) Alsa -> Alas, Ala, Alisa ams.src: W: spelling-error Summary(en_US) realtime -> mealtime, real time, real-time ams.src: W: spelling-error %description -l en_US realtime -> mealtime, real time, real-time ams.src: W: spelling-error %description -l en_US usr -> use, us, user ams.src: W: spelling-error Summary(en_US) Alsa -> Alas, Ala, Alisa ams.src: W: spelling-error Summary(en_US) realtime -> mealtime, real time, real-time ams.src: W: spelling-error %description -l en_US realtime -> mealtime, real time, real-time ams.src: W: spelling-error %description -l en_US usr -> use, us, user ams.x86_64: W: spelling-error Summary(en_US) Alsa -> Alas, Ala, Alisa ams.x86_64: W: spelling-error Summary(en_US) realtime -> mealtime, real time, real-time ams.x86_64: W: spelling-error %description -l en_US realtime -> mealtime, real time, real-time ams.x86_64: E: incorrect-fsf-address /usr/share/doc/ams-2.0.1/COPYING 4 packages and 1 specfiles checked; 1 errors, 11 warnings. [ankur@ankur SRPMS]$
^^ Looks okay
[+] final provides and requires are sane: (include output of for i in *rpm; do echo $i; rpm [-]qp --provides $i; echo =; rpm -qp --requires $i; echo; done manually indented after checking each line. I also remove the rpmlib junk and anything provided by glibc.) [ankur@ankur result]$ review-req-check == ams-2.0.1-5.fc19.src.rpm == Provides:
Requires: desktop-file-utils alsa-lib-devel clalsadrv-devel jack-audio-connection-kit-devel ladspa-devel fftw2-devel qt-devel libtool automake autoconf
== ams-2.0.1-5.fc19.x86_64.rpm == Provides: ams = 2.0.1-5.fc19 ams(x86-64) = 2.0.1-5.fc19
Requires: ladspa-cmt-plugins ladspa-mcp-plugins ladspa-rev-plugins ladspa-swh-plugins ladspa-vco-plugins libQtCore.so.4()(64bit) libQtGui.so.4()(64bit) libQtOpenGL.so.4()(64bit) libasound.so.2()(64bit) libasound.so.2(ALSA_0.9)(64bit) libc.so.6()(64bit) libc.so.6(GLIBC_2.14)(64bit) libc.so.6(GLIBC_2.2.5)(64bit) libc.so.6(GLIBC_2.3.4)(64bit) libc.so.6(GLIBC_2.4)(64bit) libclalsadrv.so.2()(64bit) libdl.so.2()(64bit) libdl.so.2(GLIBC_2.2.5)(64bit) libgcc_s.so.1()(64bit) libgcc_s.so.1(GCC_3.0)(64bit) libjack.so.0()(64bit) libm.so.6()(64bit) libm.so.6(GLIBC_2.15)(64bit) libm.so.6(GLIBC_2.2.5)(64bit) libpthread.so.0()(64bit) libpthread.so.0(GLIBC_2.2.5)(64bit) libstdc++.so.6()(64bit) libstdc++.so.6(CXXABI_1.3)(64bit) libstdc++.so.6(GLIBCXX_3.4)(64bit) libstdc++.so.6(GLIBCXX_3.4.11)(64bit) libstdc++.so.6(GLIBCXX_3.4.9)(64bit) rtld(GNU_HASH)
== ams-debuginfo-2.0.1-5.fc19.x86_64.rpm == Provides: ams-debuginfo = 2.0.1-5.fc19 ams-debuginfo(x86-64) = 2.0.1-5.fc19
Requires: ^^ Looks okay
SHOULD Items:
[+] Should build in mock. [+] Should build on all supported archs [-] Should function as described. [+] Should have sane scriptlets. [-] Should have subpackages require base package with fully versioned depend. [+] Should have dist tag [+] Should package latest version [-] check for outstanding bugs on package. (For core merge reviews)
Issues:
None. Package looks good to go!
XXX APPROVED XXX
https://bugzilla.redhat.com/show_bug.cgi?id=866358
Brendan Jones brendan.jones.it@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flags| |fedora-cvs?
--- Comment #3 from Brendan Jones brendan.jones.it@gmail.com --- Thanks for the review!
New Package SCM Request ======================= Package Name: ams Short Description: A virtual modular synthesizer for JACK and ALSA Owners: bsjones Branches:f16 f17 f18 InitialCC:
https://bugzilla.redhat.com/show_bug.cgi?id=866358
--- Comment #4 from Jon Ciesla limburgher@gmail.com --- Git done (by process-git-requests).
https://bugzilla.redhat.com/show_bug.cgi?id=866358
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |MODIFIED
https://bugzilla.redhat.com/show_bug.cgi?id=866358
--- Comment #5 from Fedora Update System updates@fedoraproject.org --- ams-2.0.1-5.fc18 has been submitted as an update for Fedora 18. https://admin.fedoraproject.org/updates/ams-2.0.1-5.fc18
https://bugzilla.redhat.com/show_bug.cgi?id=866358
--- Comment #6 from Fedora Update System updates@fedoraproject.org --- ams-2.0.1-5.fc17 has been submitted as an update for Fedora 17. https://admin.fedoraproject.org/updates/ams-2.0.1-5.fc17
https://bugzilla.redhat.com/show_bug.cgi?id=866358
--- Comment #7 from Fedora Update System updates@fedoraproject.org --- ams-2.0.1-5.fc16 has been submitted as an update for Fedora 16. https://admin.fedoraproject.org/updates/ams-2.0.1-5.fc16
https://bugzilla.redhat.com/show_bug.cgi?id=866358
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|MODIFIED |ON_QA
https://bugzilla.redhat.com/show_bug.cgi?id=866358
--- Comment #8 from Fedora Update System updates@fedoraproject.org --- ams-2.0.1-5.fc17 has been pushed to the Fedora 17 testing repository.
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=866358
Brendan Jones brendan.jones.it@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ON_QA |CLOSED Resolution|--- |CURRENTRELEASE Last Closed| |2012-11-18 23:29:59
package-review@lists.fedoraproject.org