[jira] [Commented] (OFBIZ-9972) Find an alternative suitable syntax for OFBiz rule tasks in gradle

2018-12-03 Thread Mathieu Lirzin (JIRA)


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

Mathieu Lirzin commented on OFBIZ-9972:
---

Thanks Taher.

> Find an alternative suitable syntax for OFBiz rule tasks in gradle
> --
>
> Key: OFBIZ-9972
> URL: https://issues.apache.org/jira/browse/OFBIZ-9972
> Project: OFBiz
>  Issue Type: Improvement
>Reporter: Taher Alkhateeb
>Assignee: Taher Alkhateeb
>Priority: Major
> Fix For: Upcoming Branch
>
> Attachments: OFBIZ-9972_Upgrade-to-Gradle-5.0.patch
>
>
> In testing OFBiz with a new version of gradle, I identified a potential 
> problem. The rule based tasks such as "ofbiz --test" will not work in 
> versions 5 and above. The exact error message that I received is:
> The task name 'ofbiz --test' contains at least one of the following 
> characters: [ , /, \, :, <, >, ", ?, *, |]. This has been deprecated and is 
> scheduled to be removed in Gradle 5.0.
> In other words, spaces are not allowed. We need to find an alternative 
> solution.



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


[jira] [Commented] (OFBIZ-9972) Find an alternative suitable syntax for OFBiz rule tasks in gradle

2018-12-03 Thread Taher Alkhateeb (JIRA)


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

Taher Alkhateeb commented on OFBIZ-9972:


I recreated a more clean patch that sustains the Apache header comments and 
removes any unnecessary diffs. Implemented in r1848062.

Thank you Mathieu for giving us the heads up. I hope this doesn't come to hunt 
us back in the future.

> Find an alternative suitable syntax for OFBiz rule tasks in gradle
> --
>
> Key: OFBIZ-9972
> URL: https://issues.apache.org/jira/browse/OFBIZ-9972
> Project: OFBiz
>  Issue Type: Improvement
>Reporter: Taher Alkhateeb
>Assignee: Taher Alkhateeb
>Priority: Major
> Fix For: Upcoming Branch
>
> Attachments: OFBIZ-9972_Upgrade-to-Gradle-5.0.patch
>
>
> In testing OFBiz with a new version of gradle, I identified a potential 
> problem. The rule based tasks such as "ofbiz --test" will not work in 
> versions 5 and above. The exact error message that I received is:
> The task name 'ofbiz --test' contains at least one of the following 
> characters: [ , /, \, :, <, >, ", ?, *, |]. This has been deprecated and is 
> scheduled to be removed in Gradle 5.0.
> In other words, spaces are not allowed. We need to find an alternative 
> solution.



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


[jira] [Commented] (OFBIZ-9972) Find an alternative suitable syntax for OFBiz rule tasks in gradle

2018-11-28 Thread Taher Alkhateeb (JIRA)


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

Taher Alkhateeb commented on OFBIZ-9972:


This is wonderful news, I'll start a thread on the ML to go ahead with the 
upgrade.

> Find an alternative suitable syntax for OFBiz rule tasks in gradle
> --
>
> Key: OFBIZ-9972
> URL: https://issues.apache.org/jira/browse/OFBIZ-9972
> Project: OFBiz
>  Issue Type: Improvement
>Reporter: Taher Alkhateeb
>Priority: Major
> Attachments: OFBIZ-9972_Upgrade-to-Gradle-5.0.patch
>
>
> In testing OFBiz with a new version of gradle, I identified a potential 
> problem. The rule based tasks such as "ofbiz --test" will not work in 
> versions 5 and above. The exact error message that I received is:
> The task name 'ofbiz --test' contains at least one of the following 
> characters: [ , /, \, :, <, >, ", ?, *, |]. This has been deprecated and is 
> scheduled to be removed in Gradle 5.0.
> In other words, spaces are not allowed. We need to find an alternative 
> solution.



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


[jira] [Commented] (OFBIZ-9972) Find an alternative suitable syntax for OFBiz rule tasks in gradle

2018-11-28 Thread Mathieu Lirzin (JIRA)


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

Mathieu Lirzin commented on OFBIZ-9972:
---

I don't know if they change their mind, but I see no warning.

> Find an alternative suitable syntax for OFBiz rule tasks in gradle
> --
>
> Key: OFBIZ-9972
> URL: https://issues.apache.org/jira/browse/OFBIZ-9972
> Project: OFBiz
>  Issue Type: Improvement
>Reporter: Taher Alkhateeb
>Priority: Major
> Attachments: OFBIZ-9972_Upgrade-to-Gradle-5.0.patch
>
>
> In testing OFBiz with a new version of gradle, I identified a potential 
> problem. The rule based tasks such as "ofbiz --test" will not work in 
> versions 5 and above. The exact error message that I received is:
> The task name 'ofbiz --test' contains at least one of the following 
> characters: [ , /, \, :, <, >, ", ?, *, |]. This has been deprecated and is 
> scheduled to be removed in Gradle 5.0.
> In other words, spaces are not allowed. We need to find an alternative 
> solution.



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


[jira] [Commented] (OFBIZ-9972) Find an alternative suitable syntax for OFBiz rule tasks in gradle

2018-11-28 Thread Taher Alkhateeb (JIRA)


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

Taher Alkhateeb commented on OFBIZ-9972:


Hi Mathieu,

This is great news, however, did you get a warning. Did the folks at gradle 
change their mind and now spaces are allowed again? Or are they still putting 
that deprecation warning?

> Find an alternative suitable syntax for OFBiz rule tasks in gradle
> --
>
> Key: OFBIZ-9972
> URL: https://issues.apache.org/jira/browse/OFBIZ-9972
> Project: OFBiz
>  Issue Type: Improvement
>Reporter: Taher Alkhateeb
>Priority: Major
> Attachments: OFBIZ-9972_Upgrade-to-Gradle-5.0.patch
>
>
> In testing OFBiz with a new version of gradle, I identified a potential 
> problem. The rule based tasks such as "ofbiz --test" will not work in 
> versions 5 and above. The exact error message that I received is:
> The task name 'ofbiz --test' contains at least one of the following 
> characters: [ , /, \, :, <, >, ", ?, *, |]. This has been deprecated and is 
> scheduled to be removed in Gradle 5.0.
> In other words, spaces are not allowed. We need to find an alternative 
> solution.



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


[jira] [Commented] (OFBIZ-9972) Find an alternative suitable syntax for OFBiz rule tasks in gradle

2018-11-28 Thread Mathieu Lirzin (JIRA)


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

Mathieu Lirzin commented on OFBIZ-9972:
---

Hello Taher,

As described in your second link, I would advocate adopting the standard 
[“Application” 
plugin|https://docs.gradle.org/current/userguide/application_plugin.html] which 
in recent Gradle version provides a {{--args}} command line option to the 
{{:run}} task. The benefits would be a meaningful reduction of OFBiz specific 
code in the {{build.gradle}} which helps maintainability and a familiar 
interface for people already familiar with common Gradle idioms.

In any case, I have tried to upgrade {{OFBiz}} Gradle wrapper to 5.0 (as done 
in  [^OFBIZ-9972_Upgrade-to-Gradle-5.0.patch]) and everything seems to work 
perfectly even {{./gradlew "ofbiz --test"}}.

> Find an alternative suitable syntax for OFBiz rule tasks in gradle
> --
>
> Key: OFBIZ-9972
> URL: https://issues.apache.org/jira/browse/OFBIZ-9972
> Project: OFBiz
>  Issue Type: Improvement
>Reporter: Taher Alkhateeb
>Priority: Major
> Attachments: OFBIZ-9972_Upgrade-to-Gradle-5.0.patch
>
>
> In testing OFBiz with a new version of gradle, I identified a potential 
> problem. The rule based tasks such as "ofbiz --test" will not work in 
> versions 5 and above. The exact error message that I received is:
> The task name 'ofbiz --test' contains at least one of the following 
> characters: [ , /, \, :, <, >, ", ?, *, |]. This has been deprecated and is 
> scheduled to be removed in Gradle 5.0.
> In other words, spaces are not allowed. We need to find an alternative 
> solution.



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


[jira] [Commented] (OFBIZ-9972) Find an alternative suitable syntax for OFBiz rule tasks in gradle

2018-01-12 Thread Taher Alkhateeb (JIRA)

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

Taher Alkhateeb commented on OFBIZ-9972:


Useful links for ideas:
- 
https://stackoverflow.com/questions/11696521/how-to-pass-arguments-from-command-line-to-gradle
- 
https://stackoverflow.com/questions/27604283/gradle-task-pass-arguments-to-java-application
- https://issues.gradle.org/browse/GRADLE-1804

> Find an alternative suitable syntax for OFBiz rule tasks in gradle
> --
>
> Key: OFBIZ-9972
> URL: https://issues.apache.org/jira/browse/OFBIZ-9972
> Project: OFBiz
>  Issue Type: Improvement
>Reporter: Taher Alkhateeb
>
> In testing OFBiz with a new version of gradle, I identified a potential 
> problem. The rule based tasks such as "ofbiz --test" will not work in 
> versions 5 and above. The exact error message that I received is:
> The task name 'ofbiz --test' contains at least one of the following 
> characters: [ , /, \, :, <, >, ", ?, *, |]. This has been deprecated and is 
> scheduled to be removed in Gradle 5.0.
> In other words, spaces are not allowed. We need to find an alternative 
> solution.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)