Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.
Bug 218232 (pear-Math-Stats)
Summary: | Review Request: php-pear-Math-Stats - Classes to calculate statistical parameters | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Christopher Stone <chris.stone> |
Component: | Package Review | Assignee: | Michał Bentkowski <mr.ecik> |
Status: | CLOSED NEXTRELEASE | QA Contact: | Fedora Package Reviews List <fedora-package-review> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | rawhide | Flags: | wtogami:
fedora-cvs+
|
Target Milestone: | --- | ||
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2007-01-03 18:04:29 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: | |||
Bug Depends On: | |||
Bug Blocks: | 163779 |
Description
Christopher Stone
2006-12-03 22:58:00 UTC
[jwhiter@dhcp59-101 ~]$ rpmlint /home/jwhiter/redhat/RPMS/noarch/php-pear-Math-Stats-0.9.0-0.1.beta3.noarch.rpm W: php-pear-Math-Stats non-standard-dir-in-usr doc [jwhiter@dhcp59-101 ~]$ I'm assuming thats ok since you are putting the doc into a php-pear doc directory. Everything else looks good. As this is my first review I'm going to wait on feedback for the rpmlint error before approving this. Ok everything looks good, here is my review check list. review check: -rpmlint output: [jwhiter@dhcp59-101 ~]$ rpmlint /home/jwhiter/redhat/RPMS/noarch/php-pear-Math-Stats-0.9.0-0.1.beta3.noarch.rpm [jwhiter@dhcp59-101 ~]$ - package is named according to the php package naming guidelines - spec file name matches %{name} - package meets package guidlines - package is licensed with appropriate open source license - license feild matches actual license - license is in %doc - license file is written in american english - spec file is legible - sources match upstream ffc0b653e5e2985113262a5299ebe69b Math_Stats-0.9.0beta3.tgz - package successfully compiled on fc6 x86_64 box - buildrequires makes sense and everything required is present - no locales - no shared libraries - not relocatable - owns all directories it creates - no duplicates in %files - permissions set properly - contains a %clean section - uses macros consistently - contains code - no large documentation - files in %doc do not affect runtime - no header files or static libraries - no pkgconfig files - no dynamic libraries provided by package, no need for -devel package - no .la files - not a GUI application - does not own files or directories owned by other packages sorry about the rpmlint problem before, I hadn't run fedora-buildrpmtree on the box I was building on. chris - since you haven't sponsored me yet, is this supposed to be assigned to me? If it is, am i supposed to move this into FE-ACCEPT? If not what do you need me to do? The bug should still block FE-NEW and be assigned to nobody (or whatever the default asignee is) Review: * rpmlint is quiet (and that's good ;-)) * I have no objection to names of package and spec file * sources match upstream (md5: ffc0b653e5e2985113262a5299ebe69b) * package is licensed under PHP open-source compatible license which is properly mentioned in License field and included in %doc * no problems with building on fc6/x86_64 * package isn't designed to be relocatable * %post and %postun are present (like in a template) * package meets every Guidelines it should * everything looks very similar to a php-pear spec template and everything is good :) There's even no need to set a FE-REVIEW blocker. Me and everything else can say that: ___________ < Approved! > ----------- \ ,__, \ (oo)____ (__) )\ ||--|| * - Imported into CVS - Entry added to owners.list - Tagged and Built on devel - FC-5/6 branch request made Thanks for the review! Package Change Request ====================== Package Name: php-pear-Math-Stats New Branches: EL-5 |