Spec URL: http://red.fedorapeople.org/SRPMS/R-timeSeries.spec SRPM URL: http://red.fedorapeople.org/SRPMS/R-timeSeries-2110.87-1.fc12.src.rpm Description: Environment for teaching Financial Engineering and Computational Finance. rpmlint {SPECS,SRPMS,RPMS/noarch}/R-timeSeries* R-timeSeries.src: W: spelling-error Summary(en_US) Rmetrics -> Metrics, R metrics, Metricizes R-timeSeries.noarch: W: spelling-error Summary(en_US) Rmetrics -> Metrics, R metrics, Metricizes 2 packages and 1 specfiles checked; 0 errors, 2 warnings. All warnings can be safely ignored.
Spec URL: http://red.fedorapeople.org/SRPMS/R-timeSeries.spec SRPM URL: http://red.fedorapeople.org/SRPMS/R-timeSeries-2120.89-1.fc12.src.rpm New version.
Could you update to the last version ? I will do the review.
Spec URL: http://red.fedorapeople.org/SRPMS/R-timeSeries.spec SRPM URL: http://red.fedorapeople.org/SRPMS/R-timeSeries-2130.90-1.fc14.src.rpm New version.
* 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
There is the answer from spot: """ Red Hat legal agrees, as written, the license is non-free, but they doubt that was the intention. You might ask upstream to drop the "No warranty may be created or extended by sales representatives or written sales materials." line. """ There is also a problem with the footer of the pdf which explicitly says: "Please do not distribute" Spot, tibbs and I would advice to ask upstream to remove this pdf from the package, you might also want to just rebuild a clean tarball without the pdf.
I sent upstream an email today, requesting to either remove those clauses from the PDF or to remove the PDF from the release tarball. I was in contact with upstream regarding an issue with another package already a while ago and they are very friendly, but rather slow in reacting :)
Hi Sandro, happy new year ! Do we have any news on this issue ?
nothing ?
There has been no news for 5 months now. I am therefore closing this review. If you ever hear from upstream and want to finish the review please re-open it!