[Bug 246782] New: Review Request: postgresql-plr - Procedural language interface between PostgreSQL and R

[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/bugzilla/show_bug.cgi?id=246782

Summary: Review Request: postgresql-plr - Procedural language
                    interface between PostgreSQL and R
           Product: Fedora
           Version: devel
          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]

Spec URL: http://developer.postgresql.org/~devrim/rpms/other/plr/postgresql-plr.spec
SRPM URL: http://developer.postgresql.org/~devrim/rpms/other/plr/postgresql-plr-8.2.0....
Description: Procedural Language Handler for the "R software environment for
statistical computing and graphics".

[Bug 246782] Review Request: postgresql-plr - Procedural language interface between PostgreSQL and R

[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: postgresql-plr - Procedural language interface between PostgreSQL and R

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

[e-mail] changed:

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

[Bug 246782] Review Request: postgresql-plr - Procedural language interface between PostgreSQL and R

[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: postgresql-plr - Procedural language interface between PostgreSQL and R

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

[e-mail] changed:

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

[Bug 246782] Review Request: postgresql-plr - Procedural language interface between PostgreSQL and R

[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: postgresql-plr - Procedural language interface between PostgreSQL and R

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

[e-mail] changed:

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

------- Additional Comments From [e-mail]  2007-08-24 11:18 EST -------
The following comments are based on the Package Review Guidelines and the 
Packaging Guidelines. But I'm not a sponsor of the Fedora project, so these are 
not official review comments.

- You should post the result of rpmlint command. Just I tried to run rpmlint 
for the binary package which I built, so it generate no errors/warnnings.

- You should confirm the license of the package. "BSD" is put on the License: 
tag in the specfile, but plr/README.plr in the source tree says this software 
distributed under GPLv2 or later.

- You should own the directories on which this package creates, such as %
{_datadir}/%{name} and %{_docdir}/%{name}-%{version}

- You should use -p option with install command to preserve timestamps.

- You should add the R package into BuildRequires: tag, because the package 
obviously uses /usr/lib/R/include/R.h to build.

- You should apply common compiler flags defined as $RPM_OPT_FLAGS or %
{optflags}.
(See, http://fedoraproject.org/wiki/Packaging/Guidelines#head-
8b14098227aebff1cf6188939e9d0877295ac448)
-

[Package Review Guidelines]
http://fedoraproject.org/wiki/Packaging/ReviewGuidelines

[The Packaging Guidelines]
http://fedoraproject.org/wiki/Packaging/Guidelines)

[Bug 246782] Review Request: postgresql-plr - Procedural language interface between PostgreSQL and R

[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: postgresql-plr - Procedural language interface between PostgreSQL and R

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

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
            Version|devel                       |rawhide

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]
             Status|NEW                         |NEEDINFO
               Flag|                            |needinfo?(devrim@commandprom
                   |                            |pt.com)

------- Additional Comments From [e-mail]  2008-01-20 15:59 EST -------
Hi Devrim,

Could you please update the package according to comments #1?
I'll do a formal review after that.

[Bug 246782] Review Request: postgresql-plr - Procedural language interface between PostgreSQL and R

[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: postgresql-plr - Procedural language interface between PostgreSQL and R

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

------- Additional Comments From [e-mail]  2008-05-02 14:22 EST -------
This ticket has been set to needinfo for over three months now.  Is it time to
close it?

[Bug 246782] Review Request: postgresql-plr - Procedural language interface between PostgreSQL and R

[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: postgresql-plr - Procedural language interface between PostgreSQL and R

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

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEEDINFO                    |CLOSED
         Resolution|                            |WONTFIX

------- Additional Comments From [e-mail]  2008-05-04 03:28 EST -------
Yes. This package works only for 8.2, so let's skip it.