Bug 582463
Summary: | Review Request: fcitx - Free Chinese Input Toy for X (XIM) | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Chen Lei <supercyper1> |
Component: | Package Review | Assignee: | Robin Lee <robinlee.sysu> |
Status: | CLOSED NEXTRELEASE | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
Severity: | medium | Docs Contact: | |
Priority: | low | ||
Version: | rawhide | CC: | fedora-package-review, notting, robinlee.sysu |
Target Milestone: | --- | Flags: | robinlee.sysu:
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: | 2010-04-18 03:22:46 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: |
Description
Chen Lei
2010-04-15 03:16:23 UTC
This review is using the Tibbs checklist as reference from: http://fedoraproject.org/wiki/User:Tibbs/Review_Template = source files match upstream: Using non-trunk snapshot version. + package meets naming and versioning guidelines. + specfile is properly named, is cleanly written and uses macros consistently. + dist tag is present. *********************** - license field matches the actual license. ./lib/* ./src/IC.c ./src/IC.h are in MIT ./src/uthash.h is in BSD *********************** + license is open source-compatible. + license text included in package. = latest version is being packaged. Using snapshot version. + BuildRequires are proper. + compiler flags are appropriate. + %clean is present. + package builds in Koji. http://koji.fedoraproject.org/koji/taskinfo?taskID=2122111 + package installs properly. + rpmlint outputs $ rpmlint ./fcitx.spec ./fcitx.spec: W: invalid-url Source0: fcitx-20100410.tar.bz2 0 packages and 1 specfiles checked; 0 errors, 1 warnings. $ rpmlint ./fcitx-3.6.3-1.20100410svn_utf8.fc12.src.rpm fcitx.src: W: invalid-url Source0: fcitx-20100410.tar.bz2 1 packages and 0 specfiles checked; 0 errors, 1 warnings. $ rpmlint ./fcitx-3.6.3-1.20100410svn_utf8.fc14.i686.rpm fcitx.i686: W: conffile-without-noreplace-flag /etc/X11/xinit/xinput.d/fcitx.conf 1 packages and 0 specfiles checked; 0 errors, 1 warnings. + final provides and requires are sane: $ rpm -qp --provides fcitx-3.6.3-1.20100410svn_utf8.fc14.i686.rpm config(fcitx) = 3.6.3-1.20100410svn_utf8.fc14 fcitx = 3.6.3-1.20100410svn_utf8.fc14 fcitx(x86-32) = 3.6.3-1.20100410svn_utf8.fc14 $ rpm -qp --requires fcitx-3.6.3-1.20100410svn_utf8.fc14.i686.rpm /bin/sh /bin/sh /usr/bin/python /usr/sbin/alternatives /usr/sbin/alternatives config(fcitx) = 3.6.3-1.20100410svn_utf8.fc14 im-chooser libX11.so.6 libXext.so.6 libXft.so.2 libXpm.so.4 libXrender.so.1 libXtst.so.6 libc.so.6 libc.so.6(GLIBC_2.0) libc.so.6(GLIBC_2.1) libc.so.6(GLIBC_2.3) libc.so.6(GLIBC_2.3.4) libc.so.6(GLIBC_2.4) libc.so.6(GLIBC_2.7) libdbus-1.so.3 libdl.so.2 libdl.so.2(GLIBC_2.0) libdl.so.2(GLIBC_2.1) libfontconfig.so.1 libfreetype.so.6 libpthread.so.0 libpthread.so.0(GLIBC_2.0) libpthread.so.0(GLIBC_2.1) librt.so.1 rpmlib(CompressedFileNames) <= 3.0.4-1 rpmlib(FileDigests) <= 4.6.0-1 rpmlib(PartialHardlinkSets) <= 4.0.4-1 rpmlib(PayloadFilesHavePrefix) <= 4.0-1 rtld(GNU_HASH) rpmlib(PayloadIsXz) <= 5.2-1 + no shared libraries are added to the regular linker search paths. (or, if shared libraries are present, make sure ldconfig is run) + owns the directories it creates. + doesn't own any directories it shouldn't. + no duplicates in %files. + file permissions are appropriate. + Scriptlets are proper. + code, not content. + documentation is small, so no -docs subpackage is necessary. + %docs are not necessary for the proper functioning of the package. + no headers. + no pkgconfig files. + no libtool .la droppings. Update License tag Add more explanation for UTF-8 branch SRPM URL: http://dl.dropbox.com/u/1338197/1/fcitx-3.6.3-2.20100410svn_utf8.fc12.src.rpm SPEC URL: http://dl.dropbox.com/u/1338197/1/fcitx.spec Okay. New Package CVS Request ======================= Package Name: fcitx Short Description: Fcitx is a collection of Simplified Chinese input methods for Linux. Owners: supercyper Branches: F-13 F-12 F-11 InitialCC: cheeselee liangsuilong New Package CVS Request ======================= Package Name: fcitx Short Description: Fcitx is a collection of Simplified Chinese input methods for Linux. Owners: supercyper Branches: F-13 F-12 F-11 InitialCC: cheeselee liangsuilong i18n-team CVS done (by process-cvs-requests.py). Koji build succeed, So I'm closed the report. http://cvs.fedoraproject.org/viewvc/rpms/fcitx |