Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
Summary: Review Request: perl-PlRPC - Interface for building pServer Clients Alias: perl-PlRPC
https://bugzilla.redhat.com/show_bug.cgi?id=493527
Summary: Review Request: perl-PlRPC - Interface for building pServer Clients Product: Fedora Version: rawhide Platform: All URL: http://search.cpan.org/dist/PlRPC OS/Version: Linux Status: NEW Severity: medium Priority: low Component: Package Review AssignedTo: nobody@fedoraproject.org ReportedBy: cweyl@alumni.drew.edu QAContact: extras-qa@fedoraproject.org CC: notting@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora
Spec URL: http://fedorapeople.org/~cweyl/review/perl-PlRPC.spec SRPM URL: http://fedorapeople.org/~cweyl/review/perl-PlRPC-0.2020-1.fc10.src.rpm
Description: PlRPC (Perl RPC) is a package that simplifies the writing of Perl based client/server applications. RPC::PlServer is the package used on the server side, and you guess what RPC::PlClient is for. PlRPC works by defining a set of methods that may be executed by the client.
Koji build: http://koji.fedoraproject.org/koji/taskinfo?taskID=1272115
*rt-0.06
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=493527
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?
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=493527
Parag AN(पराग) panemade@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-review? |fedora-review+
--- Comment #1 from Parag AN(पराग) panemade@gmail.com 2009-04-02 03:08:34 EDT --- Review: + package builds in mock (rawhide i586). koji Build => http://koji.fedoraproject.org/koji/taskinfo?taskID=1272115 + rpmlint is silent for SRPM and for RPM. + source files match upstream url 5361e137e56d037c5e796926ecb5300b PlRPC-0.2020.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. + %doc is present. + BuildRequires are proper. + %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. + no scriptlets present. + no duplicates in %files. + file permissions are appropriate. + make test gave All tests successful. + Package perl-PlRPC-0.2020-1.fc11.noarch => Provides: perl(Bundle::PlRPC) = 0.03 perl(RPC::PlClient) = 0.2020 perl(RPC::PlClient::Comm) = 0.1002 perl(RPC::PlClient::Object) perl(RPC::PlServer) = 0.2020 perl(RPC::PlServer::Comm) = 0.1003 perl(RPC::PlServer::Test) = 0.01 Requires: perl >= 0:5.004 perl(IO::Socket) perl(Net::Daemon) perl(Net::Daemon::Log) perl(Net::Daemon::Test) perl(RPC::PlClient::Comm) perl(RPC::PlServer) perl(RPC::PlServer::Comm) perl(Storable) perl(strict) perl(vars)
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=493527
--- Comment #2 from Parag AN(पराग) panemade@gmail.com 2009-04-02 03:11:38 EDT --- ah sorry forgot to remove make test checkitem from above review as there is no make check for this package used.
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=493527
--- Comment #3 from Chris Weyl cweyl@alumni.drew.edu 2009-04-02 03:13:47 EDT --- New Package CVS Request ======================= Package Name: perl-PlRPC Short Description: Interface for building pServer Clients Owners: cweyl Branches: F-9 F-10 devel InitialCC: perl-sig
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=493527
Chris Weyl cweyl@alumni.drew.edu changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag| |fedora-cvs?
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=493527
Kevin Fenzi kevin@tummy.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-cvs? |fedora-cvs+
--- Comment #4 from Kevin Fenzi kevin@tummy.com 2009-04-03 16:48:14 EDT --- 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=493527
Chris Weyl cweyl@alumni.drew.edu changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |CLOSED Resolution| |NEXTRELEASE
--- Comment #5 from Chris Weyl cweyl@alumni.drew.edu 2009-04-05 01:57:16 EDT --- Thanks for the review! :-)
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=493527
Lubomir Rintel lkundrak@v3.sk changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |lkundrak@v3.sk
--- Comment #6 from Lubomir Rintel lkundrak@v3.sk 2009-05-07 02:53:23 EDT --- Chris: I'll be very thankful if this was in EPEL-5 as well. Could you please request an EL-5 branch (I'd gladly do that if you have a reason for not maintaining it there).
Thanks!
package-review@lists.fedoraproject.org