[Bug 878421] Review Request: rubygem-bundler_ext - Load system gems via Bundler DSL
bugzilla at redhat.com
bugzilla at redhat.com
Fri Jul 19 05:39:52 UTC 2013
https://bugzilla.redhat.com/show_bug.cgi?id=878421
Bohuslav "Slavek" Kabrda <bkabrda at redhat.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Flags|fedora-review? |fedora-review+
|needinfo?(bkabrda at redhat.co |
|m) |
--- Comment #14 from Bohuslav "Slavek" Kabrda <bkabrda at redhat.com> ---
(In reply to Ken Dreyer from comment #13)
> Hi Slavek, I know you have a lot on your plate, so if you would like to hand
> the review over to Christopher or myself, please un-assign this review.
Huh, sorry, I totally forgot I'm assigned to this.
The package looks fine, just a small comment - is "0%{?rhel:1}" really
necessary? Wouldn't "0%{?rhel}" suffice? (same applies to "0%{fedora:1}")
But this is just a nit and the package is ok otherwise, so APPROVED.
--
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=olBqXMDKZq&a=cc_unsubscribe
More information about the package-review
mailing list