[Bug 591418] Review Request: R-timeSeries - Rmetrics - financial time series objects

[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=591418

--- Comment #4 from Pierre-YvesChibon <pingou at pingoured.fr> 2010-11-19 10:33:37 EST ---
* Latest version packaged
f55a97164636213af4cca6c24573329af193af85  timeSeries_2130.90.tar.gz
f55a97164636213af4cca6c24573329af193af85 
/home/pierrey/rpmbuild/SOURCES/timeSeries_2130.90.tar.gz
* sha1sum are equals
! Not all mandatory Requires are present, missing ['R-core']
* All required BuildRequires are present
* The macro %check is *not* present -- comment explains why
* There is 1 %dir
  %dir is OK
* There is 7 %doc -- all fine
* The rpm installed in _datadir
* The rpm uses %{_datadir} and is noarch
* The rpm seems to have the required element in %install
* Build properly under 2.6.34.7-61.fc13.x86_64
* rpmlint:
R-timeSeries.src: W: spelling-error Summary(en_US) Rmetrics -> Metrics, R
metrics, Metricates
R-timeSeries.noarch: W: spelling-error Summary(en_US) Rmetrics -> Metrics, R
metrics, Metricates
2 packages and 0 specfiles checked; 0 errors, 2 warnings.
* Koji build: http://koji.fedoraproject.org/koji/taskinfo?taskID=2611275
   0 free  0 open  2 done  0 failed
   2611275 build (dist-f14, R-timeSeries-2130.90-1.fc14.src.rpm) completed
successfully


* Build is clean
* rpmling output can be ignored
* License (GPLv2+) is compatible with Fedora 

You will have to change the "Require R" to "Requires R-core".

Discuting with spot and tibbs, there might be a license issue with the file
TimeSeriesFAQ.pdf in the inst/doc folder.

Waiting for spot's approval before I can approve this package

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