Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=245655
Summary: Review Request: perl-Gearman - Distributed job system Product: Fedora Version: devel Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nobody@fedoraproject.org ReportedBy: ruben@rubenkerkhof.com QAContact: extras-qa@fedoraproject.org CC: fedora-package-review@redhat.com,notting@redhat.com
Spec URL: http://rubenkerkhof.com/packages/perl-Gearman.spec SRPM URL: http://rubenkerkhof.com/packages/perl-Gearman-1.08-1.fc7.src.rpm Description: Gearman is a system to farm out work to other machines, dispatching function calls to machines that are better suited to do work, to do work in parallel, to load balance lots of function calls, or to call functions between languages.
This package contains the common part and the client, I'll also package perl-Gearman-Server, which depends on this.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: perl-Gearman - Distributed job system
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=245655
ruben@rubenkerkhof.com changed:
What |Removed |Added ---------------------------------------------------------------------------- OtherBugsDependingO| |245697 nThis| |
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: perl-Gearman - Distributed job system
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=245655
tibbs@math.uh.edu changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED AssignedTo|nobody@fedoraproject.org |tibbs@math.uh.edu Flag| |fedora-review?
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: perl-Gearman - Distributed job system
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=245655
------- Additional Comments From tibbs@math.uh.edu 2007-06-28 18:36 EST ------- rpmlint has this to say: E: perl-Gearman useless-explicit-provides perl(Gearman::Client) which unfortunately means you have a dependency to filter out.
The URL seems invalid; at least I get a 404 when attempting to download it. http://search.cpan.org/CPAN/authors/id/B/BR/BRADFITZ/Gearman-1.08.tar.gz seems to work for me.
I can find no license statement of any kind in the code, documentation or upstream web page. How did you determine that "GPL or Artistic" applies?
Review: * source files match upstream: bf4fe03657e49cd5109be54d43158d8e3979329f11f1c7de4420b9d4afc1a8b7 Gearman-1.08.tar.gz * package meets naming and versioning guidelines. * specfile is properly named, is cleanly written and uses macros consistently. * summary is OK. * description is OK. * dist tag is present. * build root is OK. ? license field matches the actual license. ? license is open source-compatible. * license text not included upstream. * latest version is being packaged. * BuildRequires are proper. * %clean is present. * package builds in mock (development, x86_64). * package installs properly X rpmlint has a valid complaint. X final provides and requires are sane: perl(Gearman::Client) perl(Gearman::Client) = 1.08 perl(Gearman::Job) perl(Gearman::JobStatus) perl(Gearman::Objects) perl(Gearman::ResponseParser) perl(Gearman::ResponseParser::Taskset) perl(Gearman::Task) perl(Gearman::Taskset) perl(Gearman::Util) perl(Gearman::Worker) perl-Gearman = 1.08-1.fc8 = perl(:MODULE_COMPAT_5.8.8) perl(Carp) perl(Gearman::Client) perl(Gearman::JobStatus) perl(Gearman::Objects) perl(Gearman::ResponseParser::Taskset) perl(Gearman::Task) perl(Gearman::Taskset) perl(Gearman::Util) perl(IO::Socket::INET) perl(Scalar::Util) perl(Socket) perl(String::CRC32) perl(Time::HiRes) perl(base) perl(fields) perl(strict)
* %check is present and all runnable tests pass, but not many of them run because they require a running server to test against. * owns the directories it creates. * doesn't own any directories it shouldn't. * no duplicates in %files. * file permissions are appropriate. * no scriptlets present. * code, not content. * documentation is small, so no -docs subpackage is necessary. * %docs are not necessary for the proper functioning of the package.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: perl-Gearman - Distributed job system
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=245655
------- Additional Comments From ruben@rubenkerkhof.com 2007-06-28 19:26 EST ------- Hi Jason, thanks for the review
rpmlint has this to say: E: perl-Gearman useless-explicit-provides perl(Gearman::Client) which unfortunately means you have a dependency to filter out.
Good catch, fixed
The URL seems invalid
Ah, this is what cpanspec generates. Fixed.
I can find no license statement of any kind in the code, documentation or upstream web page. How did you determine that "GPL or Artistic" applies?
I tried to prove it mathematically, but somehow failed ;-) I've just mailed the author, hope to hear from him soon.
New version: http://rubenkerkhof.com/packages/perl-Gearman.spec http://rubenkerkhof.com/packages/perl-Gearman-1.08-2.fc7.src.rpm
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: perl-Gearman - Distributed job system
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=245655
------- Additional Comments From ruben@rubenkerkhof.com 2007-06-30 05:23 EST ------- The package now includes the license.
New version:
http://rubenkerkhof.com/packages/perl-Gearman.spec http://rubenkerkhof.com/packages/perl-Gearman-1.09-1.fc7.src.rpm
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: perl-Gearman - Distributed job system
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=245655
tibbs@math.uh.edu changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-review? |fedora-review+
------- Additional Comments From tibbs@math.uh.edu 2007-06-30 14:23 EST ------- Great! rpmlint output is now clean, the errant provided symbol is gone, and the license is now obvious.
APPROVED
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: perl-Gearman - Distributed job system
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=245655
ruben@rubenkerkhof.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag| |fedora-cvs+
------- Additional Comments From ruben@rubenkerkhof.com 2007-06-30 14:41 EST ------- New Package CVS Request ======================= Package Name: perl-Gearman Short Description: Distributed job system Owners: ruben@rubenkerkhof.com Branches: F-6 F-7
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: perl-Gearman - Distributed job system
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=245655
------- Additional Comments From kevin@tummy.com 2007-07-02 14:54 EST ------- cvs done. Note that you are supposed to set the fedora-cvs flag to ?, not +. :)
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: perl-Gearman - Distributed job system
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=245655
ruben@rubenkerkhof.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |CLOSED Resolution| |NEXTRELEASE
------- Additional Comments From ruben@rubenkerkhof.com 2007-07-02 17:03 EST ------- Thanks Kev, I must have been sleeping
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: perl-Gearman - Distributed job system
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=245655
ruben@rubenkerkhof.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-cvs+ |fedora-cvs?
------- Additional Comments From ruben@rubenkerkhof.com 2007-08-09 18:04 EST ------- ackage Change Request ====================== Package Name: perl-Gearman New Branches: EL-4 EL-5
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: perl-Gearman - Distributed job system
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=245655
kevin@tummy.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-cvs? |fedora-cvs+
------- Additional Comments From kevin@tummy.com 2007-08-09 18:53 EST ------- cvs 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=245655
Petr Pisar ppisar@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |ppisar@redhat.com Flag|fedora-cvs+ |fedora-cvs?
--- Comment #10 from Petr Pisar ppisar@redhat.com 2011-07-13 04:52:34 EDT --- Package Change Request ====================== Package Name: perl-Gearman Branches: f15 f14 New InitialCC: perl-sig
Please add `perl-sig' to CC list for all Fedora branches as this is Perl package.
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=245655
--- Comment #11 from Jon Ciesla limb@jcomserv.net 2011-07-15 10:03:38 EDT --- This is now done via pkgdb.
package-review@lists.fedoraproject.org