ofbiz-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jacques Le Roux (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (OFBIZ-10145) Remove the Gradle wrapper from our release packages and add a step to our build notes
Date Wed, 19 Jun 2019 11:15:00 GMT

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

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

I can get the wrapper and its properties from  https://github.com/gradle/gradle/tree/v5.0.0/gradle/wrapper
and https://github.com/gradle/gradle/tree/v3.2.1/gradle/wrapper. But not https://github.com/gradle/gradle/tree/v2.13/gradle/wrapper.
As often external services have an end. That's one of the reasons I copied our files in tools.
After bad experiences (yes you GoOgle) I don't trust much external services life durations.
I can find it at https://repo.spring.io/plugins-release/org/gradle/gradle-wrapper/2.13/gradle-wrapper-2.13.jar
but not the properties. Anyway I can't see how it's better than having our own files copied
somewhere. We need to change things when upgrading Gradle anyway. Copying 2 files is not that
much work.

Also, thinking about it, we only need my tools solution for Buildbot. Because there all start
afresh. So the wrapper is missing each time. But not Gradle, it's there (with different versions)
on the Builbot servers. That's why the wrapper, and only it, should be copied from tools.
On the demo we should use init-gradle-wrapper.sh as in other environments (dev, production,
etc.). Only Buildbot is a transient environment that we have to handle specifically.

So rather than relying on an external service which can disappear in future, I stay with my
proposition. :)

I don't think we have a wiki page about "How to upgrade Gradle". I think we need one now and
I will create it, notably for the tools update thing.

> Remove the Gradle wrapper from our release packages and add a step to our build notes
> -------------------------------------------------------------------------------------
>
>                 Key: OFBIZ-10145
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-10145
>             Project: OFBiz
>          Issue Type: Task
>          Components: Gradle
>    Affects Versions: 17.12.01, 16.11.06, 18.12.01
>            Reporter: Jacques Le Roux
>            Assignee: Nicolas Malin
>            Priority: Blocker
>             Fix For: 17.12.01
>
>         Attachments: OFBIZ-10145-gradlew.patch, init-gradle-wrapper-R16.sh, init-gradle-wrapper-trunk-and-18.sh,
init-gradle-wrapper-trunk-and-18.sh, init-gradle-wrapper-trunk-and-18.sh, init-gradle-wrapper-trunk-and-18.sh,
init-gradle-wrapper.ps1, init-gradle-wrapper.sh, init-gradle-wrapper.sh, init-gradle-wrapper.sh,
init-gradle-wrapper.sh, init-gradle-wrapper.sh, init-gradle-wrapper.sh, init-gradlew-readme-R16.patch,
init-gradlew-readme-R17.1.patch, init-gradlew-readme-R17.1.patch, init-gradlew-readme.patch,
init-gradlew-readme.patch
>
>
> Following the discussion at http://markmail.org/message/nd7grfiyobjkfwae, considering
LEGAL-288 and based on a lazy consensus on dev ML, we want to remove the gradle-wrapper.jar
file from the next packaged releases and  use [~jacopoc]'s related proposition to document
how to have Gradle working in the main README.md file.
> # Extract the archive file to your local directory.
> # Download gradle-wrapper.jar and place it in the OFBiz-root-dir/gradle/wrapper folder.
> I'm not sure if we should recommend a link to download the gradle-wrapper.jar. This might
change in the future (versions, etc.), so indeed maybe simply asking to download is enough,
cf  https://www.google.com/search?q=gradle-wrapper.jar+download&ie=UTF-8
> Also we need to add a point about removing gradle-wrapper.jar in https://cwiki.apache.org/confluence/display/OFBIZ/Release+Management+Guide+for+OFBiz



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

Mime
View raw message