[Bug 667997] Review Request: rubygem-rack-mount - Stackable dynamic tree based Rack router

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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

--- Comment #3 from VÃt Ondruch <vondruch@xxxxxxxxxx> 2011-01-24 08:06:40 EST ---
Please see updated package:

Spec URL: http://people.redhat.com/vondruch/rubygem-rack-mount.spec
SRPM URL:
http://people.redhat.com/vondruch/rubygem-rack-mount-0.6.13-2.fc14.src.rpm

(In reply to comment #2)
> Will take this one
> 
> * $ rpmlint rpmbuild/RPMS/noarch/rubygem-rack-mount-*
> rpmbuild/SRPMS/rubygem-rack-mount-0.6.13-1.fc14.src.rpm | grep -v
> unexpanded-macro
>   rubygem-rack-mount.noarch: I: enchant-dictionary-not-found en_US
>   rubygem-rack-mount.src: W: no-cleaning-of-buildroot %clean
>   rubygem-rack-mount.src: W: no-buildroot-tag
>   rubygem-rack-mount.src: W: no-%prep-section
>   rubygem-rack-mount.src: W: no-%build-section
>   rubygem-rack-mount.src: W: no-%clean-section
> 3 packages and 0 specfiles checked; 0 errors, 76 warnings.
> 
> Can you add the missing sections (prep, build, clean), the other warnings can
> be ignored
> 

I have added missing sections

> * Missing dependency, shouldn't rubygem(rack) be a Requires 
> 
> http://rubygems.org/gems/rack-mount
> 

Added missing dependencies

> * MUST: Packages must NOT bundle copies of system libraries, rack-mount
> vendorizes the multimap and regin gems, these need to be separated into their
> own rpms

Bundled gems removed and added dependencies instead.

> 
> * Koji build is green:
> http://koji.fedoraproject.org/koji/taskinfo?taskID=2730211
> 
> * Feel free to tar up the upstream test suite and Rakefile and include and run
> them in the rpm's check section. Not a requirement for approval though.

Test suite is executed now during build. Note, however, that the test suite is
not bundled into resulting package.

> Other than that, looks good, thanks for this

I cannot provide Koji build results as long as the dependencies are not
satisfied. However, if everything goes well, the package should be prepared
already.

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



[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]