https://bugzilla.redhat.com/show_bug.cgi?id=1315801
Jun Aruga changed:
What|Removed |Added
Status|MODIFIED|CLOSED
https://bugzilla.redhat.com/show_bug.cgi?id=1315801
Jun Aruga changed:
What|Removed |Added
Status|ASSIGNED|MODIFIED
Fixed In
https://bugzilla.redhat.com/show_bug.cgi?id=1315801
--- Comment #9 from Jon Ciesla ---
Package request has been approved:
https://admin.fedoraproject.org/pkgdb/package/rpms/rubygem-nio4r
--
You are receiving this mail because:
You are on the CC list for the bug.
You are
https://bugzilla.redhat.com/show_bug.cgi?id=1315801
--- Comment #8 from Jun Aruga ---
Vit, thanks for your reivew.
I did swap the description and summary, bumped the release, and updated above
files in the URL.
I am going to add this package "bundled libraries" after that.
https://bugzilla.redhat.com/show_bug.cgi?id=1315801
Vít Ondruch changed:
What|Removed |Added
Flags|fedora-review? |fedora-review+
https://bugzilla.redhat.com/show_bug.cgi?id=1315801
--- Comment #5 from Vít Ondruch ---
> > * Bundled library
> > - It seems that this package bundles libev. Is there any chance to use the
> > system version of libev instead [1]? Or in the worst case provide the
> >
https://bugzilla.redhat.com/show_bug.cgi?id=1315801
--- Comment #4 from Jun Aruga ---
Just reminder. Waiting the response. :)
--
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
https://bugzilla.redhat.com/show_bug.cgi?id=1315801
--- Comment #3 from Jun Aruga ---
Hi, Vit
Thank you for your review!! This is in detail, and clear for me.
I updated my spec and srpm files for your review. The URLs is same with
previous review.
Koji:
https://bugzilla.redhat.com/show_bug.cgi?id=1315801
--- Comment #2 from Vít Ondruch ---
* Use virtual rubygem provides
- In you spec file, you are using "BuildRequires: rubygem-rspec", but this
would be nice to replace by "BuildRequires: rubygem(rspec)"
* Simlify
https://bugzilla.redhat.com/show_bug.cgi?id=1315801
Vít Ondruch changed:
What|Removed |Added
Status|NEW |ASSIGNED
10 matches
Mail list logo