[Bug 563000] Review Request: rubygem-compass - A Sass-based CSS Meta-Framework

bugzilla at redhat.com bugzilla at redhat.com
Wed Apr 14 17:02:06 UTC 2010


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=563000

--- Comment #4 from Mohammed Morsi <mmorsi at redhat.com> 2010-04-14 13:02:03 EDT ---
(In reply to comment #3)
> Just comment for this:
> 
> (In reply to comment #2)
> > (In reply to comment #1)
> > > File listed twice:
> > > /usr/lib/ruby/gems/1.8/gems/compass-0.8.17/LICENSE.markdown
> > > /usr/lib/ruby/gems/1.8/gems/compass-0.8.17/README.markdown
> > 
> > These two files are different (LICENSE, README) and rpmlint isn't complaining
> > about any duplicate files. Can you elaborate if this is still an issue?
> 
> Note that %files entry
> -------------------------------------------------
> %files
> foo/
> -------------------------------------------------
> contains the directory foo/ itself _and_ all files/directories/etc
> under the directory foo, and %{gemdir}/gems/%{gemname}-%{version}/
> (in %files entry) is the same as %{geminstdir} 
> (by the as you defined %geminstdir, please use this macro in %files),
> while
> -------------------------------------------------
> %files
> %dir foo/
> -------------------------------------------------
> contains the directory foo/ only.    

Fixed.

Spec updated based on feedback and new version uploaded:
Spec URL: http://mo.morsi.org/files/gems/rubygem-compass.spec
SRPM URL: http://mo.morsi.org/files/gems/rubygem-compass-0.8.17-3.fc11.src.rpm 

rpmlint rpmbuild/SPECS/rubygem-compass.spec
rpmbuild/SRPMS/rubygem-compass-0.8.17-3.fc11.src.rpm
rpmbuild/RPMS/noarch/rubygem-compass-0.8.17-3.fc11.noarch.rpm 
2 packages and 1 specfiles checked; 0 errors, 0 warnings.

Koji build: http://koji.fedoraproject.org/koji/taskinfo?taskID=2115523

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



More information about the package-review mailing list