Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
Summary: Package Review: ibus-table-erbi
https://bugzilla.redhat.com/show_bug.cgi?id=488173
Summary: Package Review: ibus-table-erbi Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Keywords: i18n Severity: medium Priority: low Component: Package Review AssignedTo: petersen@redhat.com ReportedBy: cchance@redhat.com QAContact: extras-qa@fedoraproject.org CC: notting@redhat.com, petersen@redhat.com, fedora-package-review@redhat.com, cchance@redhat.com, phuang@redhat.com, fedora-i18n-bugs@redhat.com Depends on: 488010 Blocks: 484650 Estimated Hours: 0.0 Classification: Fedora Target Release: ---
+++ This bug was initially created as a clone of Bug #488172 +++
A split from ibus-table-chinese, which was sub-package of ibus-table.
Uploading srpm and spec.
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=488173
Jens Petersen petersen@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|petersen@redhat.com |nobody@fedoraproject.org
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=488173
Bug 488173 depends on bug 488010, which changed state.
Bug 488010 Summary: Review Request: ibus-table-cangjie - Cang Jie input method for ibus-table. https://bugzilla.redhat.com/show_bug.cgi?id=488010
What |Old Value |New Value ---------------------------------------------------------------------------- Status|ASSIGNED |CLOSED Resolution| |RAWHIDE
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=488173
--- Comment #1 from Caius "kaio" Chance cchance@redhat.com 2009-04-02 01:48:04 EDT --- SRPM URL: http://cchance.fedorapeople.org/packaging/ibus-table-erbi-1.1.0.20090402-1.f... Spec URL: http://cchance.fedorapeople.org/packaging/ibus-table-erbi.spec Description: A split from ibus-table-chinese, which was sub-package of ibus-table.
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=488173
Parag AN(पराग) panemade@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED CC| |panemade@gmail.com AssignedTo|nobody@fedoraproject.org |panemade@gmail.com Flag| |fedora-review+
--- Comment #2 from Parag AN(पराग) panemade@gmail.com 2009-04-03 01:44:03 EDT --- Review: + package builds in mock (rawhide i586). koji build => http://koji.fedoraproject.org/koji/taskinfo?taskID=1274696 + rpmlint output for SRPM and for RPM. ibus-table-erbi.noarch: E: zero-length /usr/share/doc/ibus-table-erbi-1.1.0.20090402/AUTHORS ==> can be fixed by removing AUTHORS file.
+ source files match upstream url 5b36a300b07cb6eb74be84f74ae63a4b ibus-table-erbi-1.1.0.20090402.tar.gz + 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. + BuildRequires are proper. + defattr usage is correct. + %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. + no -devel subpackage + no .la files. + no translations are available. + Does owns the directories it creates. + ibus-table-createdb scriptlet used. + file permissions are appropriate. + Not a GUI App.
Suggestions: 1) Drop AUTHORS files from %doc as rpmlint message reported as ibus-table-erbi.noarch: E: zero-length /usr/share/doc/ibus-table-erbi-1.1.0.20090402/AUTHORS 2) try adding contents to Changelog file in upstream and add it as %doc
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=488173
--- Comment #3 from Caius "kaio" Chance cchance@redhat.com 2009-04-03 02:50:00 EDT --- SRPM URL: http://cchance.fedorapeople.org/packaging/ibus-table-erbi-1.1.0.20090403-1.f... Spec URL: http://cchance.fedorapeople.org/packaging/ibus-table-erbi.spec Description: A split from ibus-table-chinese, which was sub-package of ibus-table.
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=488173
--- Comment #4 from Caius "kaio" Chance cchance@redhat.com 2009-04-06 01:09:56 EDT --- ping
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=488173
--- Comment #5 from Parag AN(पराग) panemade@gmail.com 2009-04-06 01:26:51 EDT --- Can you add Changelog as %doc as asked already in comment#2 ?
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=488173
--- Comment #6 from Caius "kaio" Chance cchance@redhat.com 2009-04-06 01:37:46 EDT --- SRPM URL: http://cchance.fedorapeople.org/packaging/ibus-table-erbi-1.1.0.20090403-2.f... Spec URL: http://cchance.fedorapeople.org/packaging/ibus-table-erbi.spec Description: A split from ibus-table-chinese, which was sub-package of ibus-table.
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=488173
--- Comment #7 from Parag AN(पराग) panemade@gmail.com 2009-04-06 01:42:21 EDT --- APPROVED.
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=488173
Caius "kaio" Chance cchance@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag| |fedora-cvs?
--- Comment #8 from Caius "kaio" Chance cchance@redhat.com 2009-04-06 02:09:17 EDT --- Package Name: ibus-table-erbi Short Description: Erbi input method for ibus-table Owners: cchance Branches: devel InitialCC: i18n-team
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=488173
--- Comment #9 from Jens Petersen petersen@redhat.com 2009-04-06 03:34:04 EDT --- What about search on license/origin of the Erbi table?
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=488173
--- Comment #10 from Parag AN(पराग) panemade@gmail.com 2009-04-06 06:06:00 EDT --- license is GPLv3+ from COPYING file as erbi_qs.txt file mentions only "GPL" license.
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=488173
Jens Petersen petersen@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-cvs? |fedora-cvs+
--- Comment #11 from Jens Petersen petersen@redhat.com 2009-04-06 22:48:53 EDT --- cvs admin done
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=488173
--- Comment #12 from Parag AN(पराग) panemade@gmail.com 2009-04-06 23:49:15 EDT --- I too had discussion with upstream and he said correct license is GPLv2. Can you correct that in upstream tarball and then build new SRPM and import that in cvs as cvs module is ready now?
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=488173
--- Comment #13 from Caius "kaio" Chance cchance@redhat.com 2009-04-07 01:50:42 EDT --- SRPM URL: http://cchance.fedorapeople.org/packaging/ibus-table-erbi-1.1.0.20090407-1.f... Spec URL: http://cchance.fedorapeople.org/packaging/ibus-table-erbi.spec Description: A split from ibus-table-chinese, which was sub-package of ibus-table.
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=488173
--- Comment #14 from Caius "kaio" Chance cchance@redhat.com 2009-04-07 02:03:12 EDT --- FYI it couldn't be built atm because dependency problems of its depending packages. Tested in mock.
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=488173
--- Comment #15 from Caius "kaio" Chance cchance@redhat.com 2009-04-07 02:28:34 EDT --- SRPM URL: http://cchance.fedorapeople.org/packaging/ibus-table-erbi-1.1.0.20090407-2.f... Spec URL: http://cchance.fedorapeople.org/packaging/ibus-table-erbi.spec Description: A split from ibus-table-chinese, which was sub-package of ibus-table.
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=488173
Caius "kaio" Chance cchance@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |CLOSED Resolution| |RAWHIDE
--- Comment #16 from Caius "kaio" Chance cchance@redhat.com 2009-04-07 04:03:10 EDT --- built to rawhide
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=488173
--- Comment #17 from Caius "kaio" Chance cchance@redhat.com 2009-04-07 20:05:49 EDT --- http://koji.fedoraproject.org/koji/buildinfo?buildID=96832
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=488173
Caius 'kaio' Chance cchance@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-cvs+ |fedora-cvs?
--- Comment #18 from Caius 'kaio' Chance cchance@redhat.com 2009-05-19 20:44:06 EDT --- Package Change Request ====================== Package Name: ibus-table-erbi New Branches: F-10 Owners: cchance
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=488173
Kevin Fenzi kevin@tummy.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-cvs? |fedora-cvs+
--- Comment #19 from Kevin Fenzi kevin@tummy.com 2009-05-20 01:38:10 EDT --- cvs done.
i18n-bugs@lists.fedoraproject.org