Bug 463138 - Review Request: perl-Test-HTTP-Server-Simple - Test::More functions for HTTP::Server::Simple
Summary: Review Request: perl-Test-HTTP-Server-Simple - Test::More functions for HTTP:...
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: manuel wolfshant
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: 459536
TreeView+ depends on / blocked
 
Reported: 2008-09-22 07:45 UTC by Ralf Corsepius
Modified: 2010-06-18 16:02 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2008-09-25 00:27:02 UTC
Type: ---
Embargoed:
manuel.wolfshant: fedora-review+
j: fedora-cvs+


Attachments (Terms of Use)

Description Ralf Corsepius 2008-09-22 07:45:34 UTC
Spec URL: http://corsepiu.fedorapeople.org/packages/perl-Test-HTTP-Server-Simple.spec
SRPM URL: http://corsepiu.fedorapeople.org/packages/perl-Test-HTTP-Server-Simple-0.09-1.fc10.src.rpm

Description:
This mixin class provides methods to test an HTTP::Server::Simple-based web
server. Currently, it provides only one such method: started_ok.

This package is a dependency of rt-3.8's testsuite.

Comment 2 manuel wolfshant 2008-09-22 09:53:25 UTC
The build log says 
t/pod-coverage....skipped: Test::Pod::Coverage 1.04 required for testing POD coverage
t/pod.............skipped: Test::Pod 1.14 required for testing POD

Is this really intended ? Both are in rawhide already

Comment 3 Ralf Corsepius 2008-09-22 10:06:56 UTC
(In reply to comment #2)
> Is this really intended ? Both are in rawhide already
No, this wasn't intended - Seems as if I am a bit swamped with Fedora's bureaucracy and infrastructure defects ;)

However, in general, these two are special, because these only check for a package's integrity/cleanness and don't actually check a package for functionality. It's the reason why some packagers prefer to systematically ignore them, while others prefer to add them.

I am in the latter group, and will add them ASAP.

Comment 5 manuel wolfshant 2008-09-22 11:32:39 UTC
Package Review
==============

Key:
 - = N/A
 x = Check
 ! = Problem
 ? = Not evaluated

=== REQUIRED ITEMS ===
 [x] Package is named according to the Package Naming Guidelines.
 [x] Spec file name must match the base package %{name}, in the format %{name}.spec.
 [x] Package meets the Packaging Guidelines.
 [x] Package successfully compiles and builds into binary rpms on at least one supported architecture.
     Tested on: devel/x86_64
 [x] Rpmlint output:
source RPM:perl-Test-HTTP-Server-Simple.src: E: unknown-key GPG#8ff214b4
--> ignorable
binary RPM:empty
 [x] Package is not relocatable.
 [x] Buildroot is correct (%{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n))
 [x] Package is licensed with an open-source compatible license and meets other legal requirements as defined in the legal section of Pack
aging Guidelines.
 [x] License field in the package spec file matches the actual license.
     License type: GPL+ or Artistic
 [x] If (and only if) the source package includes the text of the license(s) in its own file, then that file, containing the text of the l
icense(s) for the package is included in %doc.
 [x] Spec file is legible and written in American English.
 [x] Sources used to build the package matches the upstream source, as provided in the spec URL.
     SHA1SUM of package: d80afc773737dc40e19024dedf455b04f3ce021b IPC-Run-SafeHandles-0.02.tar.gz
 [x] Package is not known to require ExcludeArch
 [x] All build dependencies are listed in BuildRequires, except for any that are listed in the exceptions section of Packaging Guidelines.
 [-] The spec file handles locales properly.
 [-] ldconfig called in %post and %postun if required.
 [x] Package must own all directories that it creates.
 [x] Package requires other packages for directories it uses.
 [x] Package does not contain duplicates in %files.
 [x] Permissions on files are set properly.
 [x] Package has a %clean section, which contains rm -rf %{buildroot}.
 [x] Package consistently uses macros.
 [-] The spec file handles locales properly.
 [-] ldconfig called in %post and %postun if required.
 [x] Package must own all directories that it creates.
 [x] Package requires other packages for directories it uses.
 [x] Package does not contain duplicates in %files.
 [x] Permissions on files are set properly.
 [x] Package has a %clean section, which contains rm -rf %{buildroot}.
 [x] Package consistently uses macros.
 [x] Package contains code, or permissable content.
 [-] Large documentation files are in a -doc subpackage, if required.
 [x] Package uses nothing in %doc for runtime.
 [-] Header files in -devel subpackage, if present.
 [-] Static libraries in -devel subpackage, if present.
 [-] Package requires pkgconfig, if .pc files are present.
 [-] Development .so files in -devel subpackage, if present.
 [-] Fully versioned dependency in subpackages, if present.
 [x] Package does not contain any libtool archives (.la).
 [-] Package contains a properly installed %{name}.desktop file if it is a GUI application.
 [x] Package does not own files or directories owned by other packages.

=== SUGGESTED ITEMS ===
 [x] Latest version is packaged.
 [x] Package does not include license text files separate from upstream.
 [-] Description and summary sections in the package spec file contains translations for supported Non-English languages, if available.
 [x] Reviewer should test that the package builds in mock.
     Tested on: devel/x86_64
 [-] Package should compile and build into binary rpms on all supported architectures.
-> package is noarch
 [?] Package functions as described.
 [-] Scriptlets must be sane, if used.
 [-] The placement of pkgconfig(.pc) files is correct.
 [-] File based requires are sane.

 All tests (%make check) pass


================
*** APPROVED ***
================

Comment 6 Ralf Corsepius 2008-09-22 11:54:44 UTC
New Package CVS Request
=======================
Package Name: perl-Test-HTTP-Server-Simple
Short Description: Test::More functions for HTTP::Server::Simple
Owners: corsepiu
Branches: F-9 F-8
InitialCC: perl-sig

Comment 7 Kevin Fenzi 2008-09-22 15:54:46 UTC
cvs done.

Comment 8 Fedora Update System 2008-09-22 19:47:50 UTC
perl-Test-HTTP-Server-Simple-0.09-3.fc9 has been submitted as an update for Fedora 9.
http://admin.fedoraproject.org/updates/perl-Test-HTTP-Server-Simple-0.09-3.fc9

Comment 9 Fedora Update System 2008-09-22 19:48:34 UTC
perl-Test-HTTP-Server-Simple-0.09-3.fc8 has been submitted as an update for Fedora 8.
http://admin.fedoraproject.org/updates/perl-Test-HTTP-Server-Simple-0.09-3.fc8

Comment 10 Fedora Update System 2008-09-25 00:26:59 UTC
perl-Test-HTTP-Server-Simple-0.09-3.fc9 has been pushed to the Fedora 9 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 11 Fedora Update System 2008-09-25 00:27:15 UTC
perl-Test-HTTP-Server-Simple-0.09-3.fc8 has been pushed to the Fedora 8 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 12 Mark Chappell 2010-06-18 15:39:25 UTC
corsepiu is listed as not interested in EPEL


Package Change Request
======================
Package Name: perl-Test-HTTP-Server-Simple
New Branches: EL-6
Owners: tremble

Comment 13 Jason Tibbitts 2010-06-18 16:02:48 UTC
CVS done.


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