Hi,

File.setExecutable() does not exist for Java 5.

You're going to have to fork a chmod process. Or use a native solution.

Or drop support for Java 5.

A better alternative might be to change the GUI to launch gradle by execing bash <script> rather than just <script>

Even better would be to change the GUI to not use the scripts at all.


[email protected] wrote:
Branch: refs/heads/master
Home:   http://github.com/gradle/gradle

Commit: 0ecc631bf6738d519ede945adb85b64add42ef88
    
http://github.com/gradle/gradle/commit/0ecc631bf6738d519ede945adb85b64add42ef88
Author: hansd <[email protected]>
Date:   2009-09-17 (Thu, 17 Sep 2009)

Changed paths:
  M subprojects/gradle-wrapper/build.gradle
  M subprojects/gradle-wrapper/src/main/java/org/gradle/wrapper/Install.java
  A 
subprojects/gradle-wrapper/src/test/groovy/org/gradle/wrapper/DownloadTest.groovy
  A 
subprojects/gradle-wrapper/src/test/groovy/org/gradle/wrapper/InstallTest.groovy
  A 
subprojects/gradle-wrapper/src/test/groovy/org/gradle/wrapper/PathAssemblerTest.java
  A 
subprojects/gradle-wrapper/src/test/groovy/org/gradle/wrapper/WrapperTest.java
  R 
subprojects/gradle-wrapper/src/test/java/org/gradle/wrapper/DownloadTest.groovy
  R 
subprojects/gradle-wrapper/src/test/java/org/gradle/wrapper/InstallTest.groovy
  R 
subprojects/gradle-wrapper/src/test/java/org/gradle/wrapper/PathAssemblerTest.java
  R subprojects/gradle-wrapper/src/test/java/org/gradle/wrapper/WrapperTest.java

Log Message:
-----------
GRADLE-638 gradle shell script not marked as executable.
- Set the gradle command of the unzipped wrapper dist to executable using the 
File.setExecutable method. If this was successful a message is printed out to 
the console. If not successful nothing is printed out (e.g. on windows).
- Turned wrapper from Java into Groovy project, as tests are written in Groovy. 
The tests had been ignored since our move to the multi-project build.



<hr/>
<p>
To unsubscribe from this list please visit:
</p>
<p>
    <a 
href="http://xircles.codehaus.org/manage_email";>http://xircles.codehaus.org/manage_email</a>


---------------------------------------------------------------------
To unsubscribe from this list, please visit:

   http://xircles.codehaus.org/manage_email


Reply via email to