[ 
https://issues.apache.org/jira/browse/OFBIZ-10430?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16519039#comment-16519039
 ] 

Jacques Le Roux commented on OFBIZ-10430:
-----------------------------------------

Hi Mathieu,

As I partly reported at OFBIZ-10428 for the in operator, this does not work on 
Windows. Also I got an error on line 
bq. exec \{ commandLine [gradleRunner, "--no-daemon", *commandList] }
while using
{{gradlew pullPluginSource -PpluginId=ebay}}
with this code
 {code}
def gradlewSubprocess(commandList) {
    def gradleRunner = os.contains('windows') ? 'gradlew.bat' : './gradlew'
    exec { commandLine [gradleRunner, "--no-daemon", *commandList] }
 {code}

 {code}
C:\projectsASF\ofbiz>gradlew pullPluginSource -PpluginId=ebay
:pullPluginFromSvn
:pullPluginSource FAILED

FAILURE: Build failed with an exception.

* Where:
Build file 'C:\projectsASF\ofbiz\build.gradle' line: 1045

* What went wrong:
Execution failed for task ':pullPluginSource'.
> Cannot cast object 'gradlew.bat' with class 'java.lang.String' to class 'int'
 {code}

> Use `in` and spread operator in `gradlewSubprocess`
> ---------------------------------------------------
>
>                 Key: OFBIZ-10430
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-10430
>             Project: OFBiz
>          Issue Type: Improvement
>            Reporter: Mathieu Lirzin
>            Assignee: Jacques Le Roux
>            Priority: Minor
>             Fix For: Trunk
>
>         Attachments: 
> OFBIZ-10430_Use-in-and-spread-operator-in-gradlewSubprocess.patch
>
>
> No functional change.  The idea of using the spread operator is to have less 
> mutation involved.  In Groovy, using operator overloads like {{in}} is more 
> idiomatic  than using explicitly the associated method.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to