Bug 532382
Summary: | Review Request: fvkbd - Free Virtual Keyboard | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Peter Robinson <pbrobinson> |
Component: | Package Review | Assignee: | Parag AN(पराग) <panemade> |
Status: | CLOSED RAWHIDE | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
Severity: | medium | Docs Contact: | |
Priority: | low | ||
Version: | rawhide | CC: | bnocera, fedora-package-review, i18n-bugs, notting, panemade, petersen |
Target Milestone: | --- | Flags: | panemade:
fedora-review+
kevin: fedora-cvs+ |
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2009-11-19 23:48:17 UTC | Type: | --- |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: | |||
Bug Depends On: | |||
Bug Blocks: | 532337 |
Description
Peter Robinson
2009-11-02 00:07:34 UTC
Review: + package builds in mock (rawhide i686). koji Build =>http://koji.fedoraproject.org/koji/taskinfo?taskID=1781762 + rpmlint output for SRPM and for RPM is silent. + source files match upstream url(sha1sum) 4c270a703ffa3895a992351dbb4768a550e8dd2f fvkbd-0.1.6.tar.bz2 + package meets naming and packaging guidelines. + specfile is properly named, is cleanly written + Spec file is written in American English. + Spec file is legible. + dist tag is present. + build root is correct. + license is open source-compatible. + License text is included in package. + %doc is present. + BuildRequires are proper. + %clean is present. + package installed properly. + Macro use appears rather consistent. + Package contains code, not content. + no headers or static libraries. + no .pc file present. + -devel subpackage + no .la files. + no translations are available + Does owns the directories it creates. + no scriptlets present. + no duplicates in %files. + file permissions are appropriate. + Desktop file installed correctly. + Package fvkbd-0.1.6-1.fc13.i686 => Provides: libfvkbd-0.1.so.0 libfvkbd-gtk-0.1.so.0 Requires: /usr/bin/python libX11.so.6 libXtst.so.6 libatk-1.0.so.0 libc.so.6 libc.so.6(GLIBC_2.0) libc.so.6(GLIBC_2.1.3) libc.so.6(GLIBC_2.3.4) libcairo.so.2 libfakekey.so.0 libfontconfig.so.1 libfreetype.so.6 libfvkbd-0.1.so.0 libfvkbd-gtk-0.1.so.0 libgdk-x11-2.0.so.0 libgdk_pixbuf-2.0.so.0 libgio-2.0.so.0 libglib-2.0.so.0 libgmodule-2.0.so.0 libgobject-2.0.so.0 libgtk-x11-2.0.so.0 libpango-1.0.so.0 libpangocairo-1.0.so.0 libpangoft2-1.0.so.0 libunique-1.0.so.0 libxml2.so.2 libxml2.so.2(LIBXML2_2.4.30) libxml2.so.2(LIBXML2_2.6.0) rtld(GNU_HASH) + Package fvkbd-devel-0.1.6-1.fc13.i686 => Requires: libfvkbd-0.1.so.0 libfvkbd-gtk-0.1.so.0 + GUI application Suggestions 1) I don't think you need Requires: pkgconfig remove it. 2) menu entry in Accessories is conflicting with matchbox-keyboard name "Keyboard" though once see difference in icon But I suggest you change it to "Free Virtual Keyboard" Ping? Sorry, been away from computer and very busy. I hope to have an update shortly. Package updated with the above 2 suggestions. SPEC: http://pbrobinson.fedorapeople.org/fvkbd.spec SRPM: http://pbrobinson.fedorapeople.org/fvkbd-0.1.6-2.fc12.src.rpm thanks. APPROVED. New Package CVS Request ======================= Package Name: fvkbd Short Description: Free Virtual Keyboard Owners: pbrobinson Branches: F-12 InitialCC: Should this replace the matchbox-keyboard? I believe it's supposed to be the successor to the matchbox-keyboard, as it handles CJK. (In reply to comment #7) > Should this replace the matchbox-keyboard? I believe it's supposed to be the > successor to the matchbox-keyboard, as it handles CJK. That's what I believe is suppose to happen but I figured it could obsolete it at a later point once its been tested etc. Thoughts? (In reply to comment #8) > (In reply to comment #7) > > Should this replace the matchbox-keyboard? I believe it's supposed to be the > > successor to the matchbox-keyboard, as it handles CJK. > > That's what I believe is suppose to happen but I figured it could obsolete it > at a later point once its been tested etc. Thoughts? That'd be fine by me. Mind filing a bug to that effect? CVS done. Bastien, will do. Package Change Request ====================== Package Name: fvkbd New Branches: EL-6 Owners: pbrobinson CVS done (by process-cvs-requests.py). |