Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=551721
Remi Collet <fedora(a)famillecollet.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Flag|fedora-review? |fedora-review+
--- Comment #7 from Remi Collet <fedora(a)famillecollet.com> 2010-01-24 10:53:32 EST
---
REVIEW:
~ no source files (channel.xml ok, but see notice)
* package meets naming
* package must meet 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 (of the packages provided by the
channel).
* license is open source-compatible (BSD).
* BuildRequires are proper.
* %clean is present.
* package builds in mock (rawhide).
* package installs properly
* channel (un)registered properly
* rpmlint (warnings are ok) :
php-channel-pdepend.src: I: checking
php-channel-pdepend.noarch: I: checking
php-channel-pdepend.noarch: W: no-documentation
2 packages and 1 specfiles checked; 0 errors, 1 warnings.
* final provides are sane:
php-channel(pear.pdepend.org) = 1.3
php-channel-pdepend = 1.3-3.fc8
* final requires
/usr/bin/pear
php-pear(PEAR)
* %check is not present; no test suite provide.
* doesn't own any directories it shouldn't.
* no duplicates in %files.
* file permissions are appropriate.
* scriptlets are OK (pear channel..)
* no documentation
* %install start with rm
* %clean ok
Just a minor diff between channel.xml in src.rpm and upstream one (double / in
URL) => you could update it during CVS import.
***** APPROVED *****
--
Configure bugmail:
https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.