Bug 436551

Summary:	Review Request: perl-LWP-UserAgent-Determined - A virtual browser that retries errors
Product:	[Fedora] Fedora	Reporter:	Robert Rati <rrati>
Component:	Package Review	Assignee:	Robert Rati <rrati>
Status:	CLOSED NEXTRELEASE	QA Contact:	Fedora Extras Quality Assurance <extras-qa>
Severity:	medium	Docs Contact:
Priority:	medium
Version:	rawhide	CC:	fedora-package-review, matt, notting
Target Milestone:	---	Flags:	nsantos: fedora-review+ wtogami: fedora-cvs+
Target Release:	---
Hardware:	All
OS:	Linux
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2008-03-10 19:19:11 UTC	Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

Description Robert Rati 2008-03-07 20:52:58 UTC

Spec URL: http://grid.et.redhat.com/files/perl-LWP-UserAgent-Determined.spec
SRPM URL: http://grid.et.redhat.com/files/perl-LWP-UserAgent-Determined-1.03-1.fc8.src.rpm
Description:
This class works just like LWP::UserAgent (and is based on it, by being a
subclass of it), except that when you use it to get a web page but run into
a possibly-temporary error (like a DNS lookup timeout), it'll wait a few
seconds and retry a few times.

Comment 1 Robert Rati 2008-03-07 20:57:03 UTC

This is my 3rd "first" package and am looking for a sponsor.

Comment 2 Nuno Santos 2008-03-07 21:19:03 UTC

Reviewed according to guidelines at
http://fedoraproject.org/wiki/Packaging/ReviewGuidelines

Builds under mock (on devel):

$ mock perl-LWP-UserAgent-Determined-1.03-1.fc8.src.rpm 
INFO: mock.py version 0.9.7 starting...
State Changed: init plugins
State Changed: start
INFO: Start(perl-LWP-UserAgent-Determined-1.03-1.fc8.src.rpm) 
Config(fedora-development-i386)
State Changed: lock buildroot
State Changed: clean
State Changed: init
State Changed: lock buildroot
INFO: enabled root cache
State Changed: unpacking root cache
INFO: enabled yum cache
State Changed: cleaning yum metadata
INFO: enabled ccache
State Changed: running yum
State Changed: setup
State Changed: build
INFO: Done(perl-LWP-UserAgent-Determined-1.03-1.fc8.src.rpm) Config(default) 0
minutes 45 seconds
INFO: Results and/or logs in: /var/lib/mock//fedora-development-i386/result

$ cd /var/lib/mock//fedora-development-i386/result
$ ls -alF
total 56
drwxrwsr-x 2 nsantos mock  4096 2008-03-07 16:15 ./
drwxrwsr-x 4 root    mock  4096 2008-03-07 16:14 ../
-rw-rw-r-- 1 nsantos mock  5183 2008-03-07 16:15 build.log
-rw-r--r-- 1 nsantos mock 11318 2008-03-07 16:15
perl-LWP-UserAgent-Determined-1.03-1.fc9.noarch.rpm
-rw-r--r-- 1 nsantos mock  9431 2008-03-07 16:15
perl-LWP-UserAgent-Determined-1.03-1.fc9.src.rpm
-rw-rw-r-- 1 nsantos mock 11381 2008-03-07 16:15 root.log
-rw-rw-r-- 1 nsantos mock   272 2008-03-07 16:15 state.log

$ rpmlint *.rpm
(no output)

Specfile looks good too, so I'm marking as fedora-review+

Comment 3 Robert Rati 2008-03-10 15:08:23 UTC

New Package CVS Request
=======================
Package Name: perl-LWP-UserAgent-Determined
Short Description: A virtual browser that retries errors
Owners: rrati
Branches: 
InitialCC: perl-sig
Cvsextras Commits: yes

Comment 4 Ralf Corsepius 2008-03-11 13:16:15 UTC

All perl packages must own all of their subdirectories and files underneath.

Please fix your packages ASAP.

Comment 5 Robert Rati 2008-03-11 19:29:24 UTC

This has been fixed in -3