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=245697
Summary: Review Request: perl-Gearman-Server - Function call "router" and load balancer 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-Server.spec SRPM URL: http://rubenkerkhof.com/packages/perl-Gearman-Server-1.09-1.fc7.src.rpm Description: You run a Gearman server (or more likely, many of them for both high- availability and load balancing), then have workers (using Gearman::Worker from the Gearman module, or libraries for other languages) register their ability to do certain functions to all of them, and then clients (using Gearman::Client, Gearman::Client::Async, etc) request work to be done from one of the Gearman servers.
This module needs perl-Gearman, up for review in bug #245655
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-Server - Function call "router" and load balancer Alias: perl-Gearman-Server
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=245697
ruben@rubenkerkhof.com changed:
What |Removed |Added ---------------------------------------------------------------------------- BugsThisDependsOn| |245655 Alias| |perl-Gearman-Server
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-Server - Function call "router" and load balancer Alias: perl-Gearman-Server
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=245697
tibbs@math.uh.edu changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED AssignedTo|nobody@fedoraproject.org |tibbs@math.uh.edu Flag| |fedora-review+
------- Additional Comments From tibbs@math.uh.edu 2007-06-30 14:35 EST ------- OK, perl-Gearman is done and I've put it in a local repo so that I can build this. The license problem with perl-Gearman isn't repeated here.
Review: * source files match upstream: 472c47d154a7168c627aad97fe04abdb3010275315e26382e13982ec4062a8cd Gearman-Server-1.09.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 * rpmlint is silent. * final provides and requires are sane: perl(Gearman::Server) = 1.09 perl(Gearman::Server::Client) perl(Gearman::Server::Job) perl-Gearman-Server = 1.09-1.fc8 = /usr/bin/perl perl(:MODULE_COMPAT_5.8.8) perl(Carp) perl(Danga::Socket) perl(Danga::Socket) >= 1.52 perl(FindBin) perl(Gearman::Server) perl(Gearman::Server::Client) perl(Gearman::Server::Job) perl(Gearman::Util) perl(Getopt::Long) perl(IO::Socket::INET) perl(POSIX) perl(Scalar::Util) perl(Socket) perl(Sys::Hostname) perl(base) perl(fields) perl(lib) perl(strict) perl(vars) perl(warnings) * %check is present, but there's not actually a test suite to run: No tests defined for Gearman::Server extension. * 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.
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-Server - Function call "router" and load balancer Alias: perl-Gearman-Server
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=245697
ruben@rubenkerkhof.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag| |fedora-cvs+
------- Additional Comments From ruben@rubenkerkhof.com 2007-06-30 14:43 EST ------- Thanks for the review Jason,
New Package CVS Request ======================= Package Name: perl-Gearman-Server Short Description: Function call "router" and load balancer Owners: ruben@rubenkerkhof.com Branches: FC-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-Server - Function call "router" and load balancer Alias: perl-Gearman-Server
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=245697
------- Additional Comments From ruben@rubenkerkhof.com 2007-07-02 02:08 EST ------- I added a BuildRequires for perl-Gearman-Server (which is not in the repo yet) and enabled the tests, and it seems to work in mock.
New version:
http://rubenkerkhof.com/packages/perl-Gearman-Client-Async.spec http://rubenkerkhof.com/packages/perl-Gearman-Client-Async-0.94-2.fc7.src.rp...
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-Server - Function call "router" and load balancer Alias: perl-Gearman-Server
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=245697
------- Additional Comments From kevin@tummy.com 2007-07-02 14:58 EST ------- Comment #3 looks like it's in the wrong review request?
cvs done. Note again that you should set fedora-cvs to ? and a cvsadmin will set it to + when done.
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-Server - Function call "router" and load balancer Alias: perl-Gearman-Server
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=245697
Bug 245697 depends on bug 245655, which changed state.
Bug 245655 Summary: Review Request: perl-Gearman - Distributed job system https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=245655
What |Old Value |New Value ---------------------------------------------------------------------------- Resolution| |NEXTRELEASE Status|ASSIGNED |CLOSED
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-Server - Function call "router" and load balancer Alias: perl-Gearman-Server
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=245697
ruben@rubenkerkhof.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |CLOSED Resolution| |NEXTRELEASE
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-Server - Function call "router" and load balancer Alias: perl-Gearman-Server
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=245697
ruben@rubenkerkhof.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-cvs+ |fedora-cvs?
------- Additional Comments From ruben@rubenkerkhof.com 2007-08-09 18:07 EST ------- Package Change Request ====================== Package Name: perl-Gearman-Server 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-Server - Function call "router" and load balancer Alias: perl-Gearman-Server
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=245697
kevin@tummy.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-cvs? |fedora-cvs+
------- Additional Comments From kevin@tummy.com 2007-08-09 18:55 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=245697
Petr Pisar ppisar@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |ppisar@redhat.com Flag|fedora-cvs+ |fedora-cvs?
--- Comment #7 from Petr Pisar ppisar@redhat.com 2011-07-13 04:54:28 EDT --- Package Change Request ====================== Package Name: perl-Gearman-Server 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=245697
Jon Ciesla limb@jcomserv.net changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |limb@jcomserv.net
--- Comment #8 from Jon Ciesla limb@jcomserv.net 2011-07-15 10:04:49 EDT --- This is now done via pkgdb.
package-review@lists.fedoraproject.org