Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
Summary: Review Request: wmeyes - Dockapp with moving eyes that follow mouse movement
https://bugzilla.redhat.com/show_bug.cgi?id=722771
Summary: Review Request: wmeyes - Dockapp with moving eyes that follow mouse movement Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nobody@fedoraproject.org ReportedBy: mariobl@freenet.de QAContact: extras-qa@fedoraproject.org CC: notting@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: ---
Spec URL: http://mariobl.fedorapeople.org/Review/SPECS/wmeyes.spec SRPM URL: http://mariobl.fedorapeople.org/Review/SRPMS/wmeyes-1.2-3.fc15.src.rpm Description: wmeyes is a dockapp with moving eyes that follow mouse movement. This version also allows execution of a command by clicking the icon.
$ rpmlint -i -v * wmeyes.i686: I: checking wmeyes.i686: W: spelling-error Summary(en_US) Dockapp -> Dock app, Dock-app, Paddock The value of this tag appears to be misspelled. Please double-check.
wmeyes.i686: W: spelling-error Summary(de) Dockapp -> Doppelpack The value of this tag appears to be misspelled. Please double-check.
wmeyes.i686: W: spelling-error %description -l en_US dockapp -> dock app, dock-app, paddock The value of this tag appears to be misspelled. Please double-check.
wmeyes.i686: I: checking-url http://www.bstern.org/wmeyes/ (timeout 10 seconds) wmeyes.src: I: checking wmeyes.src: W: spelling-error Summary(en_US) Dockapp -> Dock app, Dock-app, Paddock The value of this tag appears to be misspelled. Please double-check.
wmeyes.src: W: spelling-error Summary(de) Dockapp -> Doppelpack The value of this tag appears to be misspelled. Please double-check.
wmeyes.src: W: spelling-error %description -l en_US dockapp -> dock app, dock-app, paddock The value of this tag appears to be misspelled. Please double-check.
wmeyes.src: I: checking-url http://www.bstern.org/wmeyes/ (timeout 10 seconds) wmeyes.src: I: checking-url http://bstern.org/wmeyes/wmeyes-1.2.tar.gz (timeout 10 seconds) wmeyes-debuginfo.i686: I: checking wmeyes-debuginfo.i686: I: checking-url http://www.bstern.org/wmeyes/ (timeout 10 seconds) wmeyes.spec: I: checking-url http://bstern.org/wmeyes/wmeyes-1.2.tar.gz (timeout 10 seconds) 3 packages and 1 specfiles checked; 0 errors, 6 warnings.
Ignoreable spelling errors so far.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=722771
Mario Blättermann mariobl@freenet.de changed:
What |Removed |Added ---------------------------------------------------------------------------- Alias| |wmeyes
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=722771
Ankur Sinha sanjay.ankur@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED CC| |sanjay.ankur@gmail.com AssignedTo|nobody@fedoraproject.org |sanjay.ankur@gmail.com Flag| |fedora-review?
--- Comment #1 from Ankur Sinha sanjay.ankur@gmail.com 2011-07-17 14:49:30 EDT --- I'll review this one. :)
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=722771
Ankur Sinha sanjay.ankur@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-review? |fedora-review+
--- Comment #2 from Ankur Sinha sanjay.ankur@gmail.com 2011-07-18 12:01:58 EDT --- + 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 dump]$ md5sum wmeyes-1.2.tar.gz ~/rpmbuild/SOURCES/wmeyes-1.2.tar.gz bb687b5fea83d49b35552a181083142b wmeyes-1.2.tar.gz bb687b5fea83d49b35552a181083142b /home/ankur/rpmbuild/SOURCES/wmeyes-1.2.tar.gz
- 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.
defattr not required, you can remove it :)
- 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 /var/lib/mock/fedora-rawhide-i386/result/*.rpm ../SPECS/wmeyes.spec wmeyes-1.2-3.fc15.src.rpm wmeyes.i686: W: spelling-error Summary(en_US) Dockapp -> Dock app, Dock-app, Paddock wmeyes.i686: I: enchant-dictionary-not-found de wmeyes.i686: W: spelling-error %description -l en_US dockapp -> dock app, dock-app, paddock wmeyes.src: W: spelling-error Summary(en_US) Dockapp -> Dock app, Dock-app, Paddock wmeyes.src: W: spelling-error %description -l en_US dockapp -> dock app, dock-app, paddock wmeyes.src: W: spelling-error Summary(en_US) Dockapp -> Dock app, Dock-app, Paddock wmeyes.src: W: spelling-error %description -l en_US dockapp -> dock app, dock-app, paddock 4 packages and 1 specfiles checked; 0 errors, 6 warnings.
^^ Ignorable.
+ final provides and requires are sane: == wmeyes-1.2-3.fc16.i686.rpm == Provides: wmeyes = 1.2-3.fc16 wmeyes(x86-32) = 1.2-3.fc16
Requires: libICE.so.6 libSM.so.6 libX11.so.6 libXext.so.6 libXmu.so.6 libXpm.so.4 libXt.so.6 libc.so.6 libc.so.6(GLIBC_2.0) libc.so.6(GLIBC_2.1) libc.so.6(GLIBC_2.3.4) libm.so.6 libm.so.6(GLIBC_2.0) rtld(GNU_HASH)
== wmeyes-1.2-3.fc16.src.rpm == Provides:
Requires: libXext-devel libXmu-devel libXpm-devel xmkmf
== wmeyes-debuginfo-1.2-3.fc16.i686.rpm == Provides: wmeyes-debuginfo = 1.2-3.fc16 wmeyes-debuginfo(x86-32) = 1.2-3.fc16
Requires:
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:
Clean package.
XXX APPROVED XXX
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=722771
--- Comment #3 from Mario Blättermann mariobl@freenet.de 2011-07-18 14:17:04 EDT --- Many thanks for your review. Before committing to the Git, I will drop the "defattr" line.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=722771
Mario Blättermann mariobl@freenet.de changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag| |fedora-cvs?
--- Comment #4 from Mario Blättermann mariobl@freenet.de 2011-07-18 14:21:46 EDT --- New Package SCM Request ======================= Package Name: wmeyes Short Description: Dockapp with moving eyes that follow mouse movement Owners: mariobl Branches: f15
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=722771
--- Comment #5 from Jon Ciesla limb@jcomserv.net 2011-07-18 14:31:34 EDT --- Git done (by process-git-requests).
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=722771
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |MODIFIED
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=722771
--- Comment #6 from Fedora Update System updates@fedoraproject.org 2011-07-18 15:37:25 EDT --- wmeyes-1.2-3.fc15 has been submitted as an update for Fedora 15. https://admin.fedoraproject.org/updates/wmeyes-1.2-3.fc15
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=722771
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|MODIFIED |ON_QA
--- Comment #7 from Fedora Update System updates@fedoraproject.org 2011-07-22 15:41:46 EDT --- wmeyes-1.2-3.fc15 has been pushed to the Fedora 15 testing repository.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=722771
--- Comment #8 from Fedora Update System updates@fedoraproject.org 2011-08-01 21:57:17 EDT --- wmeyes-1.2-3.fc15 has been pushed to the Fedora 15 stable repository.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=722771
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ON_QA |CLOSED Fixed In Version| |wmeyes-1.2-3.fc15 Resolution| |ERRATA Last Closed| |2011-08-01 21:57:23
package-review@lists.fedoraproject.org