Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
Summary: Review Request: gettext-ant-tasks
https://bugzilla.redhat.com/show_bug.cgi?id=517776
Summary: Review Request: gettext-ant-tasks Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: low Component: Package Review AssignedTo: nobody@fedoraproject.org ReportedBy: cedric.olivier@free.fr QAContact: extras-qa@fedoraproject.org CC: notting@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora
Spec URL : http://cedric.olivier.free.fr/rpms/gettext-ant-tasks/gettext-ant-tasks.spec SRPM URL : http://cedric.olivier.free.fr/rpms/gettext-ant-tasks/gettext-ant-tasks-0.9.6...
Description: The Gettext Commons project provides Java classes for internationalization (i18n) through GNU gettext.
The lightweight library combines the power of the unix-style gettext tools with the widely used Java ResourceBundles. This makes it possible to use the original text instead of arbitrary property keys, which is less cumbersome and makes programs easier to read.
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=517776
Cédric OLIVIER cedric.olivier@free.fr changed:
What |Removed |Added ---------------------------------------------------------------------------- Blocks| |508351
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=517776
D Haley mycae@yahoo.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |mycae@yahoo.com
--- Comment #1 from D Haley mycae@yahoo.com 2009-08-22 01:46:27 EDT --- Some quick comments:
*In generate-tarball, please specify a specific version in the repository for checkout. I cannot verify the MD5 without this.
*Generate tarball permission bits should be 744. (rpmlint output)
*Using svn export is probably a better idea than using find to remove .svn after an svn co
*If you like (not required, but I like it), you can add this to the top of generate-tarball to guard against misuse of generate-tarball: if [ $# -ne 2 ] ; then echo USAGE: ./generate-tarball NAME_VERSION RELEASE exit 1 fi
*.java files do not appear to have LGPL licence headers, as required by the licence. Please raise upstream bug.
*Any kind of licence files are missing. Please raise upstream bug.
*Please post rpmlint output for reviews, as well as koji builds where possible
*"Requires: jpackage-utils" for javadoc not needed, as javadoc requires main package which also requires jpackage-utils
*Summary should replace the word "library" with "task".
*Should this package not require gettext-commons (bug 515136) ? I assume that the ant-task will fail if it cannot find the gettext jar file.
*Raise bug for upstream to provide tagged source releases in future. This will assist maintaining sync with project releases.
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=517776
--- Comment #2 from D Haley mycae@yahoo.com 2009-08-28 23:33:59 EDT --- Quick additional comment. Ant tasks are required to register themselves in /etc/ant.d/%{name} , see Bug 167525 for more info
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=517776
Cédric OLIVIER cedric.olivier@free.fr changed:
What |Removed |Added ---------------------------------------------------------------------------- Blocks|508351 |
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=517776
Rafael Aquini aquini@linux.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |aquini@linux.com
--- Comment #3 from Rafael Aquini aquini@linux.com 2010-08-19 21:29:30 EDT --- PING
It's been almost a year with no progress; This review should be closed soon if there is no response, shouldn't it?
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=517776
Rafael Aquini aquini@linux.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |CLOSED Blocks| |201449(FE-DEADREVIEW) Resolution| |NOTABUG Last Closed| |2010-09-09 19:49:43
--- Comment #4 from Rafael Aquini aquini@linux.com 2010-09-09 19:49:43 EDT --- It's been more than a year without any progress on this review. Due to this lack of activity and no replies to the latest PING (Comment 3), this review is now considered stalled. I'm closing this bug just as described in Fedora's Policy for stalled package reviews: http://fedoraproject.org/wiki/Policy_for_stalled_package_reviews
package-review@lists.fedoraproject.org