[jira] [Closed] (OFBIZ-7749) iterateOverActiveComponents exists twice

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

[jira] [Closed] (OFBIZ-7749) iterateOverActiveComponents exists twice

Nicolas Malin (Jira)

     [ https://issues.apache.org/jira/browse/OFBIZ-7749?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Jacques Le Roux closed OFBIZ-7749.
----------------------------------
       Resolution: Implemented
    Fix Version/s: Upcoming Branch

Thanks Michael,

Your patch is in trunk at revision: 1752890

All tests pass.

For the sake of history and especially the link Michael provided, Michael asked me yesterday on HipChat:
bq. have you tried this: https://docs.gradle.org/current/userguide/userguide_single.html#sec%3aconfiguring_using_external_scr... in reference to OFBIZ-7749
bq. No I did not try that. Is it new with 2.14 (we use 2.13 because 2.14 is not released yet, right?)
bq. Found this https://discuss.gradle.org/t/how-do-i-include-buildscript-block-from-external-gradle-script/7016/7

I checked this morning, thanks to the Gradle wrapper we use 2.14 which was released on 14th June 2016: https://gradle.org/gradle-download/ The Gradle wrapper is a bliss :)

> iterateOverActiveComponents exists twice
> ----------------------------------------
>
>                 Key: OFBIZ-7749
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-7749
>             Project: OFBiz
>          Issue Type: Sub-task
>          Components: framework
>    Affects Versions: Trunk
>            Reporter: Pierre Smits
>            Assignee: Jacques Le Roux
>            Priority: Minor
>              Labels: gradle
>             Fix For: Upcoming Branch
>
>         Attachments: OFBIZ-7749.patch
>
>
> Currently the iterateOverActiveComponents function or tasks exists both in build.gradle and settings.gradle.
> Do we need both? Which one is superfluous?



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)