[Bug 435623] New: Review Request: php-pdb - PHP classes for manipulating Palm OS databases

[Red Hat Bugzilla]

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

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

Summary: Review Request: php-pdb - PHP classes for manipulating
                    Palm OS databases
           Product: Fedora
           Version: rawhide
          Platform: All
        OS/Version: Linux
            Status: NEW
          Severity: medium
          Priority: low
         Component: Package Review
        AssignedTo: [e-mail]
        ReportedBy: [e-mail]
         QAContact: [e-mail]
                CC: [e-mail],[e-mail]

Spec URL: http://richardfearn.co.uk/files/fedora/php-pdb-1.3.4-1/php-pdb.spec
SRPM URL: http://richardfearn.co.uk/files/fedora/php-pdb-1.3.4-1/php-pdb-1.3.4-1.fc8.s...
Description:

Hello,

I'm looking for a sponsor for my second package, php-pdb.

PHP-PDB is a set of PHP classes that manipulate Palm OS databases. It lets you
read, write, alter, and easily use data that is meant to be sent to or
retrieved from a handheld.

The SRPM builds using mock against Fedora 8 and development. The resulting RPMs are both rpmlint clean.

[Bug 435623] Review Request: php-pdb - PHP classes for manipulating Palm OS databases

[Red Hat Bugzilla]

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-pdb - PHP classes for manipulating Palm OS databases

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

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
OtherBugsDependingO|                            |177841
              nThis|                            |

[Bug 435623] Review Request: php-pdb - PHP classes for manipulating Palm OS databases

[Red Hat Bugzilla]

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-pdb - PHP classes for manipulating Palm OS databases

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

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
OtherBugsDependingO|177841                      |
              nThis|                            |
               Flag|                            |fedora-review+

------- Additional Comments From [e-mail]  2008-05-01 12:53 EST -------
APPROVED

No run-time testing done.
Only the standard recommendation:

* Prefer "install -p ..." wherever installing pre-built files.

Plus I would move the "-m 644" to the beginning of the install args
and possibly add "-m 0755" when creating the directories.

[Bug 435623] Review Request: php-pdb - PHP classes for manipulating Palm OS databases

[Red Hat Bugzilla]

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-pdb - PHP classes for manipulating Palm OS databases

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

------- Additional Comments From [e-mail]  2008-05-01 15:46 EST -------
> Prefer "install -p ..." wherever installing pre-built files.
>
> Plus I would move the "-m 644" to the beginning of the install args
> and possibly add "-m 0755" when creating the directories.

Thanks for that. I'll put a new version on here soon.

[Bug 435623] Review Request: php-pdb - PHP classes for manipulating Palm OS databases

[Red Hat Bugzilla]

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-pdb - PHP classes for manipulating Palm OS databases

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

------- Additional Comments From [e-mail]  2008-05-01 16:00 EST -------
No need to. :)
http://fedoraproject.org/wiki/PackageMaintainers/CVSAdminProcedure

[Bug 435623] Review Request: php-pdb - PHP classes for manipulating Palm OS databases

[Red Hat Bugzilla]

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-pdb - PHP classes for manipulating Palm OS databases

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

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
               Flag|                            |fedora-cvs?

------- Additional Comments From [e-mail]  2008-05-02 17:21 EST -------
New Package CVS Request
=======================
Package Name: php-pdb
Short Description: PHP classes for manipulating Palm OS databases
Owners: richardfearn
Branches: F-9
InitialCC: 
Cvsextras Commits: yes

Thanks in advance to the CVS admins!

[Bug 435623] Review Request: php-pdb - PHP classes for manipulating Palm OS databases

[Red Hat Bugzilla]

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-pdb - PHP classes for manipulating Palm OS databases

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

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
               Flag|fedora-cvs?                 |fedora-cvs+

------- Additional Comments From [e-mail]  2008-05-02 18:24 EST -------
cvs done.

[Bug 435623] Review Request: php-pdb - PHP classes for manipulating Palm OS databases

[Red Hat Bugzilla]

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-pdb - PHP classes for manipulating Palm OS databases

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

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |CLOSED
         Resolution|                            |NEXTRELEASE