Spec URL: http://web.davehollis.com:81/packages/php-pear-HTTP-Header.spec SRPM URL: http://web.davehollis.com:81/packages/php-pear-HTTP-Header-1.2.0-1.fc8.src.rpm Description: OO interface to modify and handle HTTP headers and status codes
I can't do a formal review as i'm not a sponsor. Should fix the %file : %{pear_phpdir}/HTTP %{pear_testdir}/HTTP_Header Note HTTP dir is not owned by php-pear-HTTP, so you need to own it. Should requires only php-common to avoid pulling "apache" in dependencies. Or simply remove php >= 4.2.0 which is very old.
Ok, I've made the requested changes. Spec URL: http://web.davehollis.com:81/packages/php-pear-HTTP-Header.spec SRPM URL: http://web.davehollis.com:81/packages/php-pear-HTTP-Header-1.2.0-2.fc8.src.rpm
Actually, Remi is a sponsor now although I really thought someone else was going to sponsor David.
I am in the process of sponsoring David with bug #438804
It's been another six weeks; has anything happened regarding sponsorship?
Just waiting on sponsorship, or if someone wants to take this package under their wing, that's fine by me.
* source files match upstream: 68e0a84fb3aca46e6e0a6ccfe25c2d6b HTTP_Header-1.2.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. * license field matches the actual license (BSD) * license is open source-compatible. * latest version is being packaged (1.2.0) * %clean is present. * package builds in mock (development). * package installs properly * rpmlint is silent. * final provides and requires are sane: php-pear(HTTP_Header) = 1.2.0 php-pear-HTTP-Header = 1.2.0-2.fc8 * %check is not present; test suite cannot run in rpmbuild * owns the directories it creates (HTTP already owned but by non-dependant rpm) * doesn't own any directories it shouldn't. * file permissions are appropriate. * scriptlets are OK (pear install) * code, not content. * documentation is small, so no -docs subpackage is necessary. * %docs are not necessary for the proper functioning of the package. Please remove the TODO comment as file list is ok. Remove php-common from Requires as already required by pear (only need if you want to set a minimal version, which is not the case) This 2 lines are redundant. %dir %{pear_phpdir}/HTTP %{pear_phpdir}/HTTP Missing BR on dos2unix, so doesn't build in mock. Add it or switch to sed.
ping ??
No reponse, I close this bug to allow anynody else to submit this package again.