Bug 222597
Summary: | Review Request: php-pear-Crypt-CHAP - Class to generate CHAP packets | ||||||
---|---|---|---|---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Christopher Stone <chris.stone> | ||||
Component: | Package Review | Assignee: | Remi Collet <fedora> | ||||
Status: | CLOSED NEXTRELEASE | QA Contact: | Fedora Package Reviews List <fedora-package-review> | ||||
Severity: | medium | Docs Contact: | |||||
Priority: | medium | ||||||
Version: | rawhide | Flags: | fedora:
fedora-review+
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-03-13 19:16:27 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 | ||||||
Attachments: |
|
Description
Christopher Stone
2007-01-15 02:02:08 UTC
Created attachment 149795 [details]
php-pear-Crypt-CHAP-test.patch (to avoid awful warnings)
I always run (or try to run) tests during review.
This patch allow the simple test to run without any warning or error.
Add it if you think it is usefull (you can even file a bug upstream).
=== REVIEW === :) source files match upstream: b09ba7b851f85528638d9a9dae67b1c9 Crypt_CHAP-1.0.0.tgz :) package meets naming and packaging guidelines. :) specfile is properly named, is cleanly written and uses macros consistently. :) dist tag is present. :) build root is correct (second form). :) license field matches the actual license. :) license is open source-compatible (BSD). :) latest version is being packaged (1.0.0) :) BuildRequires are proper. :) %clean is present. :) package builds in mock (FC6). :) package installs properly :/ rpmlint output acceptable W: php-pear-Crypt-CHAP no-documentation :) final provides are sane: php-pear(Crypt_CHAP) = 1.0.0 php-pear-Crypt-CHAP = 1.0.0-1.fc7 :) final Requires are sane: php-mcrypt php-mhash php-pear(PEAR) :) %check is not present; :) owns the directories it creates :) no duplicates in %files. :) file permissions are appropriate. :) scriplets are ok (pear install) :) code, not content. :/ no documentation. === APPROVED === New Package CVS Request ======================= Package Name: php-pear-Crypt-CHAP Short Description: Class to generate CHAP packets Owners: chris.stone Branches: FC-5 FC-6 devel InitialCC: The pear bug database is seriously messed up. I have spent *way* too much time trying to submit a bug to pear, it is so difficult to report a bug that it simply is not worth the effort. In fact, I think it is actually impossible at this point. Just as an example of how bad it is, try going to: http://pear.php.net/bugs/report.php to report a bug, you cannot even create a new account, nor can I even log in with my old account. This is just the tip of the iceberg! I attempted to report the bug, but I am unable to even confirm my email address because their bug reporting system is so broken right now. That being said, I will just apply your patch and not even bother contacting upstream, the situation is pretty dire if you can no longer report bugs to pear. - Attempted to report bug to pear, but reporting a bug to pear was about ten times more difficult than actually making this package and I still was unsuccessful. - Applied patch to spec file in -2 release - Imported and built on FC-5 FC-6 and devel Thanks for the review! :D Seems you bug finaly reported... Version 1.0.1 is out Changelog : fix some warnings (test-script) ;) Indeed! Remi, can you take a look at bug #232165, I've split the package up into sub-packages for optional dependencies. I'm wondering if we should add extra Provides for these sub-packages and if we should add something to the PHP guidelines concerning optional dependencies on php packages. Branch Package CVS Request ======================= Package Name: php-pear-Crypt-CHAP Short Description: Class to generate CHAP packets Owners: chris.stone Branches: EL-5 InitialCC: |