Bug 221672 (pear-Net-POP3)
Summary: | Review Request: php-pear-Net-POP3 - Provides a POP3 class to access POP3 server | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Christopher Stone <chris.stone> |
Component: | Package Review | Assignee: | Kevin Fenzi <kevin> |
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-13 22:27:53 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
2007-01-05 23:07:15 UTC
OK - Package meets naming and packaging guidelines OK - Spec file matches base package name. OK - Spec has consistant macro usage. OK - Meets Packaging Guidelines. OK - License (BSD) OK - License field in spec matches OK - License file included in package OK - Spec in American English OK - Spec is legible. OK - Sources match upstream md5sum: dad6200744d2c8f326fbec1f5e0c8249 Net_POP3-1.3.6.tgz dad6200744d2c8f326fbec1f5e0c8249 Net_POP3-1.3.6.tgz.1 f2c485f9e2b306f5f75c7c71d373a26cad09667b Net_POP3-1.3.6.tgz f2c485f9e2b306f5f75c7c71d373a26cad09667b Net_POP3-1.3.6.tgz.1 OK - BuildRequires correct OK - Package has %defattr and permissions on files is good. OK - Package has a correct %clean section. OK - Package has correct buildroot OK - Package is code or permissible content. OK - Packages %doc files don't affect runtime. OK - Package compiles and builds on at least one arch. OK - Package has no duplicate files in %files. OK - Package doesn't own any directories other packages own. See below - Package owns all the directories it creates. See below - No rpmlint output. OK - final provides and requires are sane: SHOULD Items: OK - Should build in mock. OK - Should build on all supported archs OK - Should have dist tag OK - Should package latest version Issues: 1. I don't think that: Requires(hint): php-pear(Net_Auth_SASL) >= 1.0 is usable with any rpm shipped by Fedora. I suppose it just makes it a hard Requires. Perhaps just change that to a Requires? Also, I can't seem to find that requirement in yet, should that package be added? 2. rpmlint says: W: php-pear-Net-POP3 no-documentation Which I guess can be ignored, since upstream has had no docs. 3. The /usr/share/pear/Net directory doesn't seem to be owned by this package, it seems owned by php-pear-Net-FTP. Should that package be required? or should this package also own that directory? (In reply to comment #1) > Issues: > > 1. I don't think that: > Requires(hint): php-pear(Net_Auth_SASL) >= 1.0 > is usable with any rpm shipped by Fedora. I suppose it just makes it a hard > Requires. Perhaps just change that to a Requires? Also, I can't seem to find > that requirement in yet, should that package be added? Even though (hint) does not do anything right now, I use this for optional packages. I added it because the Summary on the web page: "...using optional Auth_SASL package" > 3. The > /usr/share/pear/Net > directory doesn't seem to be owned by this package, > it seems owned by php-pear-Net-FTP. > Should that package be required? or should this package also > own that directory? This package Requires php-pear(Net_Socket) >= 1.0 which should own the Net directory. > (In reply to comment #2) > (In reply to comment #1) > > Issues: > > > > 1. I don't think that: > > Requires(hint): php-pear(Net_Auth_SASL) >= 1.0 > > is usable with any rpm shipped by Fedora. I suppose it just makes it a hard > > Requires. Perhaps just change that to a Requires? Also, I can't seem to find > > that requirement in yet, should that package be added? > > Even though (hint) does not do anything right now, I use this for optional > packages. I added it because the Summary on the web page: > "...using optional Auth_SASL package" Although this should read "Auth_SASL" not "Net_Auth_SASL". I will fix this. Nice catch. Spec URL: http://tkmame.retrogames.com/fedora-extras/php-pear-Net-POP3.spec SRPM URL: http://tkmame.retrogames.com/fedora-extras/php-pear-Net-POP3-1.3.6-2.src.rpm %changelog * Sat Jan 13 2007 Christopher Stone <chris.stone> 1.3.6-2 - Fix Requires for Auth_SASL ok, that appears to fix up all the blockers I see. This package is APPROVED. Don't forget to close this review request NEXTRELEASE once it's been imported and built. - Imported into CVS - Entry added to owners.list - built successfully for devel - cvs sync request added for FC5/6 THANKS FOR THE REVIEW! Package Change Request ====================== Package Name: php-pear-Net-POP3 New Branches: EL-5 |