https://bugzilla.redhat.com/show_bug.cgi?id=1268701
Bug ID: 1268701 Summary: Review Request: rubygem-growl - growlnotify bindings Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nobody@fedoraproject.org Reporter: ilya.gradina@gmail.com QA Contact: extras-qa@fedoraproject.org CC: package-review@lists.fedoraproject.org
Spec URL: https://github.com/ilgrad/fedora-packages/raw/master/rubygems/rubygem-growl.... SRPM URL: https://github.com/ilgrad/fedora-packages/raw/master/rubygems/rubygem-growl-... Description: growlnotify bindings. Fedora Account System Username: ilgrad
https://bugzilla.redhat.com/show_bug.cgi?id=1268701
Ilya Gradina ilya.gradina@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Blocks| |1268765
Referenced Bugs:
https://bugzilla.redhat.com/show_bug.cgi?id=1268765 [Bug 1268765] Review Request: rubygem-grape-entity - A simple facade for managing the relationship between your model and API
https://bugzilla.redhat.com/show_bug.cgi?id=1268701
Vít Ondruch vondruch@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |vondruch@redhat.com
--- Comment #1 from Vít Ondruch vondruch@redhat.com --- Growl is MacOS X stuff, do we really need it on Fedora?
https://bugzilla.redhat.com/show_bug.cgi?id=1268701
--- Comment #2 from Ilya Gradina ilya.gradina@gmail.com --- (In reply to Vít Ondruch from comment #1)
Growl is MacOS X stuff, do we really need it on Fedora?
he's coming as a dependency from the package rubygem-grape-entity ( 1268765 ), don't know whether you can do without this package.
https://bugzilla.redhat.com/show_bug.cgi?id=1268701
--- Comment #3 from Vít Ondruch vondruch@redhat.com --- I would suggest to remove this dependency.
Actually, looking at bug 1268765, it is just build time dependency, isn't it? Then I really can't see reason why we should have this in Fedora and I strongly suggest to remove the dependency, including dependency on Bundler, i.e. this line:
https://github.com/ruby-grape/grape-entity/blob/master/spec/spec_helper.rb#L...
package-review@lists.fedoraproject.org