[Bug 498846] Review Request: R-RM2 - Revenue Management and Pricing for R

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


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





--- Comment #9 from Denis Arnaud <denis.arnaud_fedora at m4x.org>  2009-07-06 15:51:28 EDT ---
Thanks for that review!

1. You are absolutely right about the license. The authors specify "GPL >= 3";
so, GPLv3+ must be used (instead of GPLv3).

2. As for the %{packrel} macro, as I now understand, it should be used for the
release of the upstream R package (and not the RPM release).

The updated specification file and source RPM are to be found at the following
URLs:
-----------------------------------------------------
Spec URL: http://denisarnaud.fedorapeople.org/R/RM2/3/R-RM2.spec
SRPM URL: http://denisarnaud.fedorapeople.org/R/RM2/3/R-RM2-0.0-3.fc11.src.rpm
-----------------------------------------------------

As for the package review process, besides a few reviews, I watch almost all
the C++-related packages, so as to try to help where there is no other reviewer
assigned (it is not that much, but I own quite a few upstream open source
projects, which I have to push if I ever want them to reach a critical mass).
However, you are right, and I do fully understand that we are never enough to
perform peer reviews :)

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.