Re: [gradle-scm] [gradle/gradle] 0ecc63: GRADLE-638 gradle shell script not marked as execu...

View: New views
2 Messages — Rating Filter:   Alert me  

Parent Message unknown Re: [gradle-scm] [gradle/gradle] 0ecc63: GRADLE-638 gradle shell script not marked as execu...

by Adam Murdoch-2 :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

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.


noreply@... wrote:

> Branch: refs/heads/master
> Home:   http://github.com/gradle/gradle
>
> Commit: 0ecc631bf6738d519ede945adb85b64add42ef88
>     http://github.com/gradle/gradle/commit/0ecc631bf6738d519ede945adb85b64add42ef88
> Author: hansd <mail@...>
> 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



Re: Re: [gradle-scm] [gradle/gradle] 0ecc63: GRADLE-638 gradle shell script not marked as execu...

by hdockter :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message


On Sep 17, 2009, at 10:54 PM, Adam Murdoch wrote:

> Hi,
>
> File.setExecutable() does not exist for Java 5.

Thanks for pointing this out.

>
> You're going to have to fork a chmod process.

I will go this route.

> 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.

Definitely.

- Hans

--
Hans Dockter
Gradle Project Manager
http://www.gradle.org


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

    http://xircles.codehaus.org/manage_email