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=644335 --- Comment #2 from Jozef Zigmund <jzigmund@xxxxxxxxxx> 2010-10-20 12:56:09 EDT --- (In reply to comment #1) > Some notes for -1: > > - Source0: http://rubygems.org/gems/%{gemname}-%{version}.gem > > Please this URL to: Source0: http://rubygems.org/downloads/%{}-%{version}.gem > I repair URL to Source0: http://rubygems.org/downloads/%{gemname}-%{version}.gem, because in your URL was typo. > - Summary > > Please remove "Mustache is", it's unnecessarily here. > Removed. > - Require > > Please add rubygem-sinatra to requires: > > ./lib/mustache/sinatra.rb:require 'sinatra/base Add to spec file, Require: rubygem(sinatra) NEW SPEC URL: http://st.fri.uniza.sk/~zigmundj/fedorapkg/rubygem-mustache.spec NEW SRPM URL: http://st.fri.uniza.sk/~zigmundj/fedorapkg/rubygem-mustache-0.11.2-2.fc13.src.rpm -- 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. _______________________________________________ package-review mailing list package-review@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review