[Bug 176733] Review Request: php-pear-DB (PEAR database abstraction layer)

[bugzilla at redhat.com]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: php-pear-DB (PEAR database abstraction layer)


https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=176733


ivazquez at ivazquez.net changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|gdk at redhat.com              |ivazquez at ivazquez.net
OtherBugsDependingO|163776                      |163779
              nThis|                            |




------- Additional Comments From ivazquez at ivazquez.net  2006-01-08 06:57 EST -------
- Builds fine in FC4 mock
- Upstream source matches
- rpmlint output:

W: php-pear-DB invalid-license The PHP License

- ignorable, rpmlint problem

E: php-pear-DB non-executable-script /usr/share/pear/tests/DB/tests/run.cvs 0644
E: php-pear-DB non-executable-script
/usr/share/pear/tests/DB/tests/driver/run.cvs 0644

- ignorable; they aren't meant to actually be run per se AFAIK

W: php-pear-DB dangerous-command-in-%post install

- bogus, but add pear to Requires(post) and Requires(postun)

- The rest looks good

The Requires(post{,un}) change can easily enough be made in CVS.

APPROVED

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.