[Bug 246793] Review Request: postgresql-plruby - PostgreSQL Ruby Procedural Language

[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: postgresql-plruby - PostgreSQL Ruby Procedural Language


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


devrim at commandprompt.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEEDINFO                    |ASSIGNED
               Flag|needinfo?(devrim at commandprom|
                   |pt.com)                     |




------- Additional Comments From devrim at commandprompt.com  2007-07-17 03:34 EST -------
Hi,

(In reply to comment #1)

> * the name of ruby modules must be ruby-<modules>

This is not a ruby module. This is PL/Ruby for PostgreSQL. So, I am inclined to
leave the package name as it is now.

> * the package must have: Requires: ruby(abi) = <version>

Done.

>   (and I think the package should have: BuildRequires: ruby(abi) = <version>)

Why not ruby-devel ?

> * the package must provide ruby(<package>) = <version>-<release>

But it does not provide it?

> * Usually ruby module packages have Group : Development/Languages

As I said, this is a PL for PostgreSQL, so let's leave as Applications/Databases.

Thanks for the review. I will post the updated spec and srpm shortly.

Regards, Devrim

-- 
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.