Spec URL: http://people.redhat.com/vondruch/rubygem-rack-ssl.spec SRPM URL: http://people.redhat.com/vondruch/rubygem-rack-ssl-1.3.2-1.fc16.src.rpm Description: Rack middleware to force SSL/TLS. Koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=3190854
I'm taking this one.
- License should be MIT, I don't see any references to GPLv2+ or Ruby in any of the included files. - Maybe you could alter the way you run tests by moving the -I%{buildroot}%{geminstdir}/lib parameter to RUBYOPT. I think that once you use RUBYOPT, you should place everything in it not to make mess of things. So after you correct the License tag, this package is APPROVED.
(In reply to comment #2) > - License should be MIT, I don't see any references to GPLv2+ or Ruby in any of > the included files. Good spot! Thank you. > - Maybe you could alter the way you run tests by moving the > -I%{buildroot}%{geminstdir}/lib parameter to RUBYOPT. I think that once you use > RUBYOPT, you should place everything in it not to make mess of things. It does make sense. I'll think about it :) > So after you correct the License tag, this package is APPROVED. Thank you for your review. New Package SCM Request ======================= Package Name: rubygem-rack-ssl Short Description: Force SSL/TLS in your app Owners: vondruch Branches: InitialCC:
Git done (by process-git-requests).