Spec URL: https://raw.github.com/fotioslindiakos/rubygem-all/master/SPECS/rubygem-coffee-script.spec SRPM URL: https://github.com/fotioslindiakos/rubygem-all/raw/master/SRPMS/rubygem-coffee-script-2.2.0-1.fc15.src.rpm Description: Ruby CoffeeScript is a bridge to the JS CoffeeScript compiler.
I don't know very much about ruby, but this package doesn't appear to follow the current guidelines very closely at all. https://fedoraproject.org/wiki/Packaging:Ruby Not knowing ruby I can't comment on how important those differences actually are. Perhaps you could comment on those differences.
Since Fotios has been inactive for quite some time, I'll take this review over.
So here are the updated spec and srpm, I created them completely from scratch: SPEC: http://bkabrda.fedorapeople.org/pkgs/coffee-script/rubygem-coffee-script.spec SRPM: http://bkabrda.fedorapeople.org/pkgs/coffee-script/rubygem-coffee-script-2.2.0-1.fc17.src.rpm Description: Ruby CoffeeScript is a bridge to the JS CoffeeScript compiler. Koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=4246436
* The test should be expanded in %check section - I would prefer if the test suite is expanded in the %check section, since in that case, you will avoid the RPM polution. Although admittedly, the %patch macro does not work there, but you can use something like: cat %{PATCH0} | patch -p1 Since this is just minor nit and the package looks reasonable otherwise, I APPROVE the package.
Thanks, I will fix this before importing. New Package SCM Request ======================= Package Name: rubygem-coffee-script Short Description: Ruby CoffeeScript Compiler Owners: bkabrda Branches: InitialCC:
Git done (by process-git-requests).