[Bug 695819] New: Review Request: php-pecl-rrd - PHP Bindings for rrdtool

[Red Hat Bugzilla]

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

Summary: Review Request: php-pecl-rrd - PHP Bindings for rrdtool

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

Summary: Review Request: php-pecl-rrd - PHP Bindings for
                    rrdtool
           Product: Fedora
           Version: rawhide
          Platform: All
        OS/Version: Linux
            Status: NEW
          Severity: medium
          Priority: medium
         Component: Package Review
        AssignedTo: [e-mail]
        ReportedBy: [e-mail]
         QAContact: [e-mail]
                CC: [e-mail], [e-mail]
   Estimated Hours: 0.0
    Classification: Fedora
      Story Points: ---

Spec URL:
https://github.com/remicollet/remirepo/raw/master/php-pecl-rrd/php-pecl-rrd....
SRPM URL: http://remi.fedorapeople.org/php-pecl-rrd-1.0.0-1.remi.src.rpm
Description: 
Procedural and simple OO wrapper for rrdtool - data logging and graphing
system for time series data.

koji build --scratch dist-rawhide :
http://koji.fedoraproject.org/koji/taskinfo?taskID=2996366

For now, this package is not a drop-in replacement of rrdtool-php, but some
functions have the same name, so I add a "Conflicts" directive.

If rrdtool maintainers want to EOL rrdtool-php in a future fedora release, I
will change this to Obsoletes/Provides.

[Bug 695819] Review Request: php-pecl-rrd - PHP Bindings for rrdtool

[Red Hat Bugzilla]

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=695819

Remi Collet  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]

--- Comment #1 from Remi Collet  2011-04-12 14:48:00 EDT ---
Adding rrdtool-owner to CC: (as discussed with Jaroslav Skarvada)

[Bug 695819] Review Request: php-pecl-rrd - PHP Bindings for rrdtool

[Red Hat Bugzilla]

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=695819

--- Comment #2 from Remi Collet  2011-04-16 11:53:41 EDT ---
Update to 1.0.1 (in fact, no change in code, just to have latest version)

https://github.com/remicollet/remirepo/commit/bcff63fc76cf91aed32d0819bb3986...

SRPM URL: http://rpms.famillecollet.com/SRPMS/php-pecl-rrd-1.0.1-1.remi.src.rpm

[Bug 695819] Review Request: php-pecl-rrd - PHP Bindings for rrdtool

[Red Hat Bugzilla]

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=695819

--- Comment #3 from Remi Collet  2011-04-20 06:33:58 EDT ---
Update to 1.0.2 (in fact, no change in code, just to have latest version)

https://github.com/remicollet/remirepo/commit/322b10d5c7e833506c14fcbdb0c9ab...

SRPM URL: http://rpms.famillecollet.com/SRPMS/php-pecl-rrd-1.0.2-1.remi.src.rpm

[Bug 695819] Review Request: php-pecl-rrd - PHP Bindings for rrdtool

[Red Hat Bugzilla]

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=695819

Jaroslav Škarvada  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
         AssignedTo|[e-mail]    |[e-mail]

[Bug 695819] Review Request: php-pecl-rrd - PHP Bindings for rrdtool

[Red Hat Bugzilla]

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=695819

--- Comment #4 from Remi Collet  2011-04-29 12:32:41 EDT ---
Update to 1.0.2 (in fact, no change in code, just to have latest version)

https://github.com/remicollet/remirepo/commit/4d6e24cdcba0b2fd2389e64a8bd88d...

SRPM: http://rpms.famillecollet.com/SRPMS/php-pecl-rrd-1.0.3-1.remi.src.rpm

[Bug 695819] Review Request: php-pecl-rrd - PHP Bindings for rrdtool

[Red Hat Bugzilla]

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=695819

Jaroslav Škarvada  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |MODIFIED
               Flag|                            |fedora-review+

--- Comment #5 from Jaroslav Škarvada  2011-05-12 11:32:15 EDT ---
Looks good.

MUST items:
[YES] rpmplint is silent (not counting false positives about spelling)
[YES] Package meets naming guidelines.
[YES] Package meets packaging guidelines.
[YES] Spec file matches base package name.
[YES] License file is present, matching with spec file.
[YES] Licensing Guidelines are met.
[YES] Spec file is legible and in American English.
[YES] Sources match upstream.
[YES] Package builds OK.
[YES] BuildRequires is correct.
[YES] Package doesn't bundle copies of system libraries.
[YES] Package has no duplicity in %files.
[YES] Permission on files are set properly.
[YES] %doc doesn't affect runtime.
[YES] Spec file has consistant macro usage.
[YES] Package is code or permissable content.
[YES] No .la libtool archives.
[YES] Package doesn't own files/directories that other packages own.
[YES] Spec file is valid UTF-8.

SHOULD items:
[YES] Package builds in mock.
[YES] Package should build on all supported architectures.
[YES] Package passed simple functional test.
[YES] Used scriptlets are sane.

[Bug 695819] Review Request: php-pecl-rrd - PHP Bindings for rrdtool

[Red Hat Bugzilla]

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=695819

Remi Collet  changed:

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

--- Comment #6 from Remi Collet  2011-05-12 12:09:30 EDT ---
Thanks for the review

New Package SCM Request
=======================
Package Name: php-pecl-rrd
Short Description: PHP Bindings for rrdtool
Owners: remi
Branches: f14 f15 el6
InitialCC:

[Bug 695819] Review Request: php-pecl-rrd - PHP Bindings for rrdtool

[Red Hat Bugzilla]

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=695819

--- Comment #7 from Jason Tibbitts  2011-05-12 12:38:29 EDT ---
Git done (by process-git-requests).

[Bug 695819] Review Request: php-pecl-rrd - PHP Bindings for rrdtool

[Red Hat Bugzilla]

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=695819

--- Comment #8 from Fedora Update System  2011-05-14 02:24:49 EDT ---
php-pecl-rrd-1.0.3-1.fc15 has been submitted as an update for Fedora 15.
https://admin.fedoraproject.org/updates/php-pecl-rrd-1.0.3-1.fc15

[Bug 695819] Review Request: php-pecl-rrd - PHP Bindings for rrdtool

[Red Hat Bugzilla]

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=695819

--- Comment #9 from Fedora Update System  2011-05-14 02:25:14 EDT ---
php-pecl-rrd-1.0.3-1.fc14 has been submitted as an update for Fedora 14.
https://admin.fedoraproject.org/updates/php-pecl-rrd-1.0.3-1.fc14

[Bug 695819] Review Request: php-pecl-rrd - PHP Bindings for rrdtool

[Red Hat Bugzilla]

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=695819

--- Comment #10 from Fedora Update System  2011-05-14 02:25:39 EDT ---
php-pecl-rrd-1.0.3-1.el6 has been submitted as an update for Fedora EPEL 6.
https://admin.fedoraproject.org/updates/php-pecl-rrd-1.0.3-1.el6

[Bug 695819] Review Request: php-pecl-rrd - PHP Bindings for rrdtool

[Red Hat Bugzilla]

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=695819

Fedora Update System  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|MODIFIED                    |ON_QA

--- Comment #11 from Fedora Update System  2011-05-16 17:15:57 EDT ---
php-pecl-rrd-1.0.3-1.el6 has been pushed to the Fedora EPEL 6 testing
repository.

[Bug 695819] Review Request: php-pecl-rrd - PHP Bindings for rrdtool

[Red Hat Bugzilla]

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=695819

--- Comment #12 from Fedora Update System  2011-05-24 23:24:25 EDT ---
php-pecl-rrd-1.0.3-1.fc15 has been pushed to the Fedora 15 stable repository.

[Bug 695819] Review Request: php-pecl-rrd - PHP Bindings for rrdtool

[Red Hat Bugzilla]

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=695819

Fedora Update System  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ON_QA                       |CLOSED
   Fixed In Version|                            |php-pecl-rrd-1.0.3-1.fc15
         Resolution|                            |ERRATA
        Last Closed|                            |2011-05-24 23:24:30

[Bug 695819] Review Request: php-pecl-rrd - PHP Bindings for rrdtool

[Red Hat Bugzilla]

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=695819

--- Comment #13 from Fedora Update System  2011-05-28 19:56:12 EDT ---
php-pecl-rrd-1.0.3-1.fc14 has been pushed to the Fedora 14 stable repository.

[Bug 695819] Review Request: php-pecl-rrd - PHP Bindings for rrdtool

[Red Hat Bugzilla]

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=695819

Fedora Update System  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
   Fixed In Version|php-pecl-rrd-1.0.3-1.fc15   |php-pecl-rrd-1.0.3-1.fc14

[Bug 695819] Review Request: php-pecl-rrd - PHP Bindings for rrdtool

[Red Hat Bugzilla]

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=695819

--- Comment #14 from Fedora Update System  2011-06-02 15:23:23 EDT ---
php-pecl-rrd-1.0.3-1.el6 has been pushed to the Fedora EPEL 6 stable
repository.

[Bug 695819] Review Request: php-pecl-rrd - PHP Bindings for rrdtool

[Red Hat Bugzilla]

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=695819

Fedora Update System  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
   Fixed In Version|php-pecl-rrd-1.0.3-1.fc14   |php-pecl-rrd-1.0.3-1.el6