[Bug 821146] Review Request: jruby-rack - Rack adapter for JRuby and Servlet Containers

bugzilla at redhat.com bugzilla at redhat.com
Wed May 22 14:18:39 UTC 2013


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

--- Comment #14 from gil cattaneo <puntogil at libero.it> ---
(In reply to Vít Ondruch from comment #13)
> Gil, I'm still wondering, why don't you package it just as a gem (see
> comment #4). What is the point of keeping the maven stuff and -javadoc
> subpackages around? They are not distributed with the gem version as far as

no this package isn't *just a gem* ...
the java library is required for build others project as sonar

what you fail to understand? when is specified that this package can be used
*only with jruby*?

> I understand. I would like to see this package renamed to
> "rubygem-jruby-rack" and that is it. Is that some Java standard? Should we
> clarify this with Java-SIG?
> 
take a look here
http://fedoraproject.org/wiki/Packaging:Java
http://fedoraproject.org/wiki/Packaging:Java#Pre-built_JAR_files_.2F_Other_bundled_software

> Moreover, you should install the gem into standard Fedora's gem locations
> IMO, using standard gem macros. The
> "%{_datadir}/jruby/lib/ruby/gems/1.8/gems/" was never intended to be used on
> Fedora. I am not sure if that location even works (but you probably tested
> it, so we might want to disable it, to prevent its misuse).
> 

repeat *only for jruby*

> Also, I am not Java, JRuby nor Rack expert, but I would say, that you have
> overspecified requires. From my POV, the gem should depend just on
> "ruby(rubygems)" and nothing else and the reasoning is following: I have
> installed MRI and I do "yum install rubygem-jruby-rack". After that, I can
> do:
> 
> $ irb
> irb(main):001:0> p RUBY_DESCRIPTION
> "ruby 2.0.0p0 (2013-02-24) [x86_64-linux]"
> => "ruby 2.0.0p0 (2013-02-24) [x86_64-linux]"
> irb(main):002:0> require 'jruby-rack'
> => true
> irb(main):003:0> p JRuby::Rack::VERSION
> "1.1.13.2"
> => "1.1.13.2"
> 

irb read also the internal jruby-rack.jar?

> and you see, it "works". It can be required. From the code, I would say that
> this is design decision of upstream. So why you would force me to install
> all the dependencies you have listed in the .spec file right now?

dependencies? are essentially due to the fact that *this package is not like
any other rubygem(XYZ)*
because it *contains a java library*. as a result you should also consider the
dependencies used or
required by this latest, listed in the pom file
hope that my response does not sound irreverent
regard

-- 
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=6tPyhmCw1D&a=cc_unsubscribe


More information about the package-review mailing list