Lighthouse has a new layout. Prefer the old one? Return to the old layout, and switch back any time from the link at the top of each page.

This project is archived and is in readonly mode.

JRuby platform support for gem tasks.

#2215

In a Rails project that uses JRuby, the rake task gems:install doesn't use the correct rubygems binary to install dependencies. The correct solution is to use the Gem API to run the install command rather than spawning the rubygems command in a shell.

Reported by Zack Hobson · March 11th, 2009 @ 05:47 PM

State: resolved
Milestone: 2.x
Assigned to: Matt Jones Matt Jones
Importance: none

Activity

  1. CancelProfileIsBroken
    CancelProfileIsBroken
    • Assigned user set to Matt Jones

    March 11th, 2009 @ 07:19 PM

  2. Matt Jones
    Matt Jones

    This is trickier than it seems - the GemRunner version of install exits the process after the first gem is installed (see #210 rake gems:install only installs one gem at a time).

    Is it possible to just pick the correct executable in gem_command?

    March 11th, 2009 @ 09:59 PM

  3. Zack Hobson
    Zack Hobson

    Thanks Matt, I wasn't aware of the purpose of the current implementation. A new patch is attached that just checks for the java platform and selects an appropriate command.

    March 12th, 2009 @ 12:55 AM

  4. Zack Hobson
    Zack Hobson
    • Title changed from Use the Gem API to install gem dependencies instead of a shell command. to JRuby platform support for gem tasks.

    March 12th, 2009 @ 12:59 AM

  5. Matt Jones
    Matt Jones

    This looks reasonable to me, but I don't have a JRuby setup to test on. I've posted a request to rails-core to get some feedback.

    March 12th, 2009 @ 07:50 AM

  6. Repository
    Repository
    • State changed from new to resolved

    (from [5b751ae0b3f6c10a1053e4a2e04fd92d228d787f]) Execute the appropriate gem command when using JRuby [#2215 JRuby platform support for gem tasks. state:resolved]

    Signed-off-by: Pratik Naik pratiknaik@gmail.com http://github.com/rails/rails/co...

    March 13th, 2009 @ 10:25 AM