Change allows ember-cli-rails to work under jruby-9.0.3.0

[softwareanimal]

There may be a better fix for this. I doubt that this change should be merged as it is.

Without this change, I get an error after running app.rb line 44:
@pid = exec(cmd, method: :spawn)

Using ruby-2.2.2 there is no problem. With 'rvm use jruby-9.0.3.0' the error is as follows :

Java::JavaLang::ArrayIndexOutOfBoundsException (-2):
org.jruby.util.io.PopenExecutor.run_exec_dup2(PopenExecutor.java:789)
org.jruby.util.io.PopenExecutor.execargRunOptions(PopenExecutor.java:986)
org.jruby.util.io.PopenExecutor.spawnProcess(PopenExecutor.java:113)
org.jruby.util.io.PopenExecutor.spawn(PopenExecutor.java:75)
org.jruby.RubyProcess.spawn(RubyProcess.java:1296)
org.jruby.RubyKernel.spawn(RubyKernel.java:1509)
...

[seanpdoyle]

@softwareanimal thanks for the contribution!

Keep in mind, App#exec isn't the same as Kernel.exec.

Invoking App#exec usually runs system, but it can be configured to run spawn.

Could you verify which invocation of App#exec is causing the issue?

[softwareanimal]

app.rb near line 342 is the changed line, and where the error happens :
Kernel.public_send(method_name, env_hash, cmd, err: :out)

The 'method_name' variable prints out as 'spawn' and 'method: :spawn' is passed in from app.rb near line 52 (not app.rb line 44 as I stated before):
@pid = exec(cmd, method: :spawn)

Not sure if this is related to the question.

Again, I am not sure that you want to merge a change to support jruby-9.0.3.0. Also not sure what change you would merge. I just opened this pull request to report what worked under this environment and what does not work.

[seanpdoyle]

@softwareanimal thanks for the context. I'll try looking into this and I'll ping you if I need more info.

[nadavshatz]

👍

[seanpdoyle]

@softwareanimal thank you so much for bringing this to our attention.

A fix is being tracked in #288.

Closing in favor of #288

[seanpdoyle]

@softwareanimal @nadavshatz could you please checkout the branch on #288 and give it a shot?