[Bug 1131991] Review Request: rubygem-logstash-event - Classes required to create LogStash events

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

 



https://bugzilla.redhat.com/show_bug.cgi?id=1131991

Josef Stribny <jstribny@xxxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
              Flags|                            |fedora-review+



--- Comment #6 from Josef Stribny <jstribny@xxxxxxxxxx> ---
> The package goal is not much bigger, but at least some reformulation was 
> really needed and in the end I was able to add some extra information too. :-)

Great.

> Also I found missing dependency on json in gemfile

Actually, json comes with Ruby on Fedora :), so that dependency is not really
needed, but it's better that it's stated.

Rpmlint doesn't compain, package builds, installs, runs, spec looks sane =>
APPROVED.


[0]
https://github.com/elasticsearch/logstash/commit/e03b67dc7da0c3d654caef9ca1a144d1ab99e580

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