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 210783 (pear-DB-QueryTool)
Summary: | Review Request: php-pear-DB-QueryTool - An OO-interface for easily retrieving and modifying data in a DB | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Christopher Stone <chris.stone> |
Component: | Package Review | Assignee: | Jason Tibbitts <j> |
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: | 2006-10-17 02:55:10 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-10-14 23:13:33 UTC
Would you happen to have any idea why the pear registration bit tries to do network access? It makes things terribly slow to install in a mock chroot (where name resolution doesn't really work). Since this package has three runtime dependencies, it takes about five or so minutes to install them all. Everything looks fine; the only question I have is why the documentation is down in a couple of subdirectories: /usr/share/doc/php-pear-DB-QueryTool-1.0.3 /usr/share/doc/php-pear-DB-QueryTool-1.0.3/docs /usr/share/doc/php-pear-DB-QueryTool-1.0.3/docs/examples /usr/share/doc/php-pear-DB-QueryTool-1.0.3/docs/examples/example.php [...] Perhaps the "docs" subdirectory could be eliminated. In any case, I don't think it's a blocker; it just looks a bit odd. * source files match upstream: 6b93afe58f4e5bdd2f198b2b6d7acefa DB_QueryTool-1.0.3.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. * license field matches the actual license. * license is open source-compatible. License text not separately included upstream (although it is in the source files). * latest version is being packaged. * BuildRequires are proper. * %clean is present. * package builds in mock (development, x86_64). * package installs properly * rpmlint is silent. * final provides and requires are sane: php-pear(DB_QueryTool) = 1.0.3 php-pear-DB-QueryTool = 1.0.3-1.fc6 = /bin/sh /usr/bin/pear php-pear(DB) php-pear(Log) php-pear(PEAR) * %check is not present; there is some form of test suite, but it is not possible to run it at build time. * owns the directories it creates. * doesn't own any directories it shouldn't. * no duplicates in %files. * file permissions are appropriate. * scriptlets are OK (pear module installation) * code, not content. * documentation is small, so no -docs subpackage is necessary. * %docs are not necessary for the proper functioning of the package. APPROVED I've update the spec to not include the extra doc/ dir. Thanks for the review. Package Change Request ====================== Package Name: php-pear-DB-QueryTool New Branches: EL-5 |