Bug 246782 - Review Request: postgresql-plr - Procedural language interface between PostgreSQL and R
Review Request: postgresql-plr - Procedural language interface between Postgr...
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
All Linux
medium Severity medium
: ---
: ---
Assigned To: Nobody's working on this, feel free to take it
Fedora Extras Quality Assurance
Depends On:
  Show dependency treegraph
Reported: 2007-07-04 15:25 EDT by Devrim GUNDUZ
Modified: 2008-08-02 19:40 EDT (History)
4 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2008-05-04 03:28:05 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---

Attachments (Terms of Use)

  None (edit)
Description Devrim GUNDUZ 2007-07-04 15:25:54 EDT
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-
Description: Procedural Language Handler for the "R software environment for
statistical computing and graphics".
Comment 1 KaiGai Kohei 2007-08-24 11:18:26 EDT
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 %
(See, http://fedoraproject.org/wiki/Packaging/Guidelines#head-

[Package Review Guidelines]

[The Packaging Guidelines]
Comment 2 Ruben Kerkhof 2008-01-20 15:59:48 EST
Hi Devrim,

Could you please update the package according to comments #1?
I'll do a formal review after that.
Comment 3 Jason Tibbitts 2008-05-02 14:22:18 EDT
This ticket has been set to needinfo for over three months now.  Is it time to
close it?
Comment 4 Devrim GUNDUZ 2008-05-04 03:28:05 EDT
Yes. This package works only for 8.2, so let's skip it.

Note You need to log in before you can comment on or make changes to this bug.