Bug 513869 - Review Request: perl-Beanstalk-Client - Client class to talk to a beanstalkd server
Summary: Review Request: perl-Beanstalk-Client - Client class to talk to a beanstalkd ...
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Parag AN(पराग)
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2009-07-26 18:43 UTC by Ruben Kerkhof
Modified: 2012-07-10 22:15 UTC (History)
4 users (show)

Fixed In Version: 1.05-2.fc11
Clone Of:
Environment:
Last Closed: 2009-08-15 08:24:35 UTC
Type: ---
Embargoed:
panemade: fedora-review+


Attachments (Terms of Use)

Description Ruben Kerkhof 2009-07-26 18:43:26 UTC
Spec URL: http://ruben.fedorapeople.org/perl-Beanstalk-Client.spec
SRPM URL: http://ruben.fedorapeople.org/perl-Beanstalk-Client-1.05-1.fc11.src.rpm
Description:
Beanstalk::Client provides a Perl API of protocol version 1.0 to the
beanstalkd server, a fast, general-purpose, in-memory workqueue service by
Keith Rarick.

Comment 1 Parag AN(पराग) 2009-07-28 10:17:19 UTC
Review:
+ package builds in mock (rawhide i586).
koji Build =>http://koji.fedoraproject.org/koji/taskinfo?taskID=1534328
- rpmlint output messages for SRPM and for RPM.
perl-Beanstalk-Client.src: W: mixed-use-of-spaces-and-tabs (spaces: line 1, tab: line 15)
==> Can be fixed using 
sed -i -e 's|\t| |g' perl-Beanstalk-Client.spec
+ source files match upstream url
8a95bef54fe681f25752849e6fce6f0e3e742e43  Beanstalk-Client-1.05.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.
Files=2, Tests=68,  1 wallclock secs ( 0.03 usr  0.00 sys +  0.10 cusr  0.03 csys =  0.16 CPU)
+ Package perl-Beanstalk-Client-1.05-1.fc12.noarch =>
Provides: perl(Beanstalk::Client) = 1.05 perl(Beanstalk::Job) = 1.02 perl(Beanstalk::Pool) = 0.00 perl(Beanstalk::Stats) = 1.02
Requires: perl(Beanstalk::Job) perl(Beanstalk::Stats) perl(Carp) perl(Error) perl(IO::Socket::INET) perl(Socket) perl(YAML::Syck) perl(base) perl(strict) perl(warnings)

+ Not a GUI application

Should:
1) Working Source URL is http://search.cpan.org/CPAN/authors/id/G/GB/GBARR/Beanstalk-Client-1.05.tar.gz
update in SPEC.

2) you don't need following Requires in spec. Yum will auto install it.
Requires:       perl(Class::Accessor::Fast)
Requires:       perl(Error)
Requires:       perl(YAML::Syck)

Comment 2 Ruben Kerkhof 2009-07-29 10:23:48 UTC
Thanks for the review. All issues are fixed.

New version:
SPEC: http://ruben.fedorapeople.org/perl-Beanstalk-Client.spec
SRPM: http://ruben.fedorapeople.org/perl-Beanstalk-Client-1.05-2.fc11.src.rpm

Comment 3 Parag AN(पराग) 2009-08-05 10:47:59 UTC
APPROVED.

Comment 4 Ruben Kerkhof 2009-08-06 16:55:31 UTC
New Package CVS Request
=======================
Package Name: perl-Beanstalk-Client
Short Description: Client class to talk to a beanstalkd server
Owners: ruben
Branches: F-11

Comment 5 Kevin Fenzi 2009-08-07 20:01:44 UTC
cvs done.

Comment 6 Fedora Update System 2009-08-08 15:18:36 UTC
perl-Beanstalk-Client-1.05-2.fc11 has been submitted as an update for Fedora 11.
http://admin.fedoraproject.org/updates/perl-Beanstalk-Client-1.05-2.fc11

Comment 7 Fedora Update System 2009-08-10 21:49:57 UTC
perl-Beanstalk-Client-1.05-2.fc11 has been pushed to the Fedora 11 testing repository.  If problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update perl-Beanstalk-Client'.  You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F11/FEDORA-2009-8443

Comment 8 Fedora Update System 2009-08-15 08:24:30 UTC
perl-Beanstalk-Client-1.05-2.fc11 has been pushed to the Fedora 11 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 9 Petr Pisar 2012-07-09 13:25:44 UTC
Package Change Request
======================
Package Name: perl-Beanstalk-Client
Branches: f16 f17
Owners: 
InitialCC: perl-sig

Please add `perl-sig' user with watch* permissions only to all Fedora branches.


Note You need to log in before you can comment on or make changes to this bug.