An update to @rtyler's commit that looks for -S in jrubyArgs of JRubyExec.

Author: ysb33r

README.md

@@ -136,14 +136,15 @@ All other methods should work.
 ### Running a Ruby PATH command
 
 Because `JRubyExec` checks for the existence of the script, it might look at first whether running Ruby commands from
-`PATH` could be difficult. However, this is totally possible by utilising `jrubyArgs`. Here is an example of running 
+`PATH` could be difficult. However, this is totally possible by utilising `jrubyArgs` and passing `-S` as one would do
+ when using `ruby` or `jruby` on the command-line. Here is an example of running 
 `rake` as task.
 
 ```groovy
-task rake( type :JRubyExec ) {
-    jrubyArgs '-S', 'rake'
-    script '/path/to/Rakefile'
-    scriptArgs 'target1', 'target2'
+task rake( type : JRubyExec ) {
+    jrubyArgs '-S' 
+    script 'rake'
+    scriptArgs '/path/to/Rakefile', 'target1', 'target2'
 }
 ```
 
@@ -153,9 +154,9 @@ or even
 ext {
     rake = { String target ->
         jrubyexec {
-            jrubyArgs '-S', 'rake'
-            script '/path/to/Rakefile'
-            scriptArgs target            
+            jrubyArgs '-S' 
+            script 'rake'
+            scriptArgs '/path/to/Rakefile', target            
         }
     }
 }

build.gradle

@@ -83,6 +83,9 @@ test {
     systemProperties TEST_GEM_DIR : new File(projectDir,'src/integTest/resources/gems').absolutePath
     systemProperties 'logback.configurationFile' : new File(projectDir,'src/test/resources/logback-test.xml').absolutePath
 
+    if(gradle.startParameter.isOffline()) {
+        systemProperties 'TESTS_ARE_OFFLINE' : '1'
+    }
 }
 
 integrationTest {

src/main/groovy/com/github/jrubygradle/JRubyExec.groovy

@@ -1,6 +1,7 @@
 package com.github.jrubygradle
 
 import com.github.jrubygradle.internal.JRubyExecUtils
+import org.gradle.api.InvalidUserDataException
 import org.gradle.api.Project
 import org.gradle.api.artifacts.Configuration
 import org.gradle.api.file.FileCollection
@@ -38,6 +39,7 @@ class JRubyExec extends JavaExec {
     /** Script to execute.
      *
      */
+    @Input
     File script
 
     /** Configuration to copy gems from. If {@code jRubyVersion} has not been set, {@code jRubyExec} will used as
@@ -152,19 +154,38 @@ class JRubyExec extends JavaExec {
         super.exec()
     }
 
-    /** getArgs gets overridden in order to add JRuby options, script name and script argumens in the correct order
+    /** getArgs gets overridden in order to add JRuby options, script name and script arguments in the correct order.
+     *
+     * There are three modes of behaviour
+     * <ul>
+     *   <li> script set. no jrubyArgs, or jrubyArgs does not contain {@code -S} - Normal way to execute script. A check
+     *   whether the script exists will be performed.
+     *   <li> script set. jrubyArgs contains {@code -S} - If script is not absolute, no check will be performed to see
+     *   if the script exists and will be assumed that the script can be found using the default ruby path mechanism.
+     *   <li> script not set, but jrubyArgs set - Set up to execute jruby with no script. This should be a rarely used otion.
+     * </ul>
+     *
+     * @throw {@code org.gradle.api.InvalidUserDataException} if mode of behaviour cannot be determined.
      */
     @Override
     List<String> getArgs() {
         def cmdArgs = []
 
-        if ((script == null) && (jrubyArgs.size() == 0)) {
-            throw new TaskInstantiationException('Cannot instantiate a JRubyExec instance without either `script` or `jrubyArgs` set')
-        }
+        boolean useBinPath = jrubyArgs.contains('-S')
         cmdArgs.addAll(jrubyArgs)
 
-        if (script != null) {
+        if(script!=null && !useBinPath) {
+            if(!script.exists()) {
+                throw new InvalidUserDataException("${script} does not exist")
+            }
             cmdArgs.add(script.absolutePath)
+        } else if(script!=null && useBinPath ) {
+            if(script.isAbsolute() && !script.exists()) {
+                throw new InvalidUserDataException("${script} does not exist")
+            }
+            cmdArgs.add(script.toString())
+        } else if(script==null && jrubyArgs.size() == 0 ) {
+            throw new InvalidUserDataException('Cannot instantiate a JRubyExec instance without either `script` or `jrubyArgs` set')
         }
         cmdArgs.addAll(scriptArgs)
         cmdArgs as List<String>

src/test/groovy/com/github/jrubygradle/JRubyExecSpec.groovy

@@ -1,5 +1,6 @@
 package com.github.jrubygradle
 
+import org.gradle.api.InvalidUserDataException
 import org.gradle.api.tasks.TaskInstantiationException
 import org.gradle.testfixtures.ProjectBuilder
 import spock.lang.*
@@ -135,12 +136,12 @@ class JRubyExecSpec extends Specification {
         when:
             project.configure(execTask) {
                 scriptArgs '-s1','-s2','-s3'
-                jrubyArgs  '-j1','-j2','-j3'
+                jrubyArgs  '-j1','-j2','-j3','-S'
                 script     "${TEST_SCRIPT_DIR}/helloWorld.rb"
             }
 
         then:
-            execTask.getArgs() == ['-j1','-j2','-j3',new File(TEST_SCRIPT_DIR,'helloWorld.rb').absolutePath,'-s1','-s2','-s3']
+            execTask.getArgs() == ['-j1','-j2','-j3','-S',new File(TEST_SCRIPT_DIR,'helloWorld.rb').toString(),'-s1','-s2','-s3']
     }
 
     def "Properly handle the lack of a `script` argument"() {
@@ -160,7 +161,7 @@ class JRubyExecSpec extends Specification {
             execTask.getArgs()
 
         then: "An exception should be thrown"
-            thrown(TaskInstantiationException)
+            thrown(InvalidUserDataException)
     }
 
 }