svn commit: r1862353 - in /ofbiz/ofbiz-framework/trunk: gradle/init-gradle-wrapper.ps1 gradlew gradlew.bat

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

svn commit: r1862353 - in /ofbiz/ofbiz-framework/trunk: gradle/init-gradle-wrapper.ps1 gradlew gradlew.bat

jleroux@apache.org
Author: jleroux
Date: Sun Jun 30 16:32:39 2019
New Revision: 1862353

URL: http://svn.apache.org/viewvc?rev=1862353&view=rev
Log:
Improved: Remove the Gradle wrapper from our release packages and add a step to
our build notes
(OFBIZ-10145)

Applies the last waiting gradlew.bat.patch in OFBIZ-10145
Fixes a trivial issue in Mathieu last commits: repeated "-Xms64m" in gradlew
scripts.
Uses HTTP in init-gradle-wrapper.ps1 instead of HTTP. Not everyone has an HTTPS
access to the tools repo.

Modified:
    ofbiz/ofbiz-framework/trunk/gradle/init-gradle-wrapper.ps1
    ofbiz/ofbiz-framework/trunk/gradlew
    ofbiz/ofbiz-framework/trunk/gradlew.bat

Modified: ofbiz/ofbiz-framework/trunk/gradle/init-gradle-wrapper.ps1
URL: http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/trunk/gradle/init-gradle-wrapper.ps1?rev=1862353&r1=1862352&r2=1862353&view=diff
==============================================================================
--- ofbiz/ofbiz-framework/trunk/gradle/init-gradle-wrapper.ps1 (original)
+++ ofbiz/ofbiz-framework/trunk/gradle/init-gradle-wrapper.ps1 Sun Jun 30 16:32:39 2019
@@ -22,5 +22,5 @@ if (! (Test-Path -Path ((Get-Item -Path
 
 # This uses  PowerShell Invoke-WebRequest command (aliased as wget here)
 # https is not used because we don't want users to be asked for a credential (not sure about that, maybe https is OK)
-wget -outf gradle\wrapper\gradle-wrapper.jar https://svn.apache.org/repos/asf/ofbiz/tools/Buildbot/Gradle/Wrapper/trunk/gradle-wrapper.jar
-wget -outf gradle\wrapper\gradle-wrapper.properties https://svn.apache.org/repos/asf/ofbiz/tools/Buildbot/Gradle/Wrapper/trunk/gradle-wrapper.properties
+wget -outf gradle\wrapper\gradle-wrapper.jar http://svn.apache.org/repos/asf/ofbiz/tools/Buildbot/Gradle/Wrapper/trunk/gradle-wrapper.jar
+wget -outf gradle\wrapper\gradle-wrapper.properties http://svn.apache.org/repos/asf/ofbiz/tools/Buildbot/Gradle/Wrapper/trunk/gradle-wrapper.properties

Modified: ofbiz/ofbiz-framework/trunk/gradlew
URL: http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/trunk/gradlew?rev=1862353&r1=1862352&r2=1862353&view=diff
==============================================================================
--- ofbiz/ofbiz-framework/trunk/gradlew (original)
+++ ofbiz/ofbiz-framework/trunk/gradlew Sun Jun 30 16:32:39 2019
@@ -51,7 +51,7 @@ APP_NAME="Gradle"
 APP_BASE_NAME=`basename "$0"`
 
 # Add default JVM options here. You can also use JAVA_OPTS and GRADLE_OPTS to pass JVM options to this script.
-DEFAULT_JVM_OPTS='"-Xmx64m" "-Xms64m"'
+DEFAULT_JVM_OPTS='"-Xmx64m"
 
 # Use the maximum available, or set MAX_FD != -1 to use that value.
 MAX_FD="maximum"

Modified: ofbiz/ofbiz-framework/trunk/gradlew.bat
URL: http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/trunk/gradlew.bat?rev=1862353&r1=1862352&r2=1862353&view=diff
==============================================================================
--- ofbiz/ofbiz-framework/trunk/gradlew.bat (original)
+++ ofbiz/ofbiz-framework/trunk/gradlew.bat Sun Jun 30 16:32:39 2019
@@ -30,7 +30,7 @@ set APP_BASE_NAME=%~n0
 set APP_HOME=%DIRNAME%
 
 @rem Add default JVM options here. You can also use JAVA_OPTS and GRADLE_OPTS to pass JVM options to this script.
-set DEFAULT_JVM_OPTS="-Xmx64m" "-Xms64m"
+set DEFAULT_JVM_OPTS="-Xmx64m"
 
 @rem Find java.exe
 if defined JAVA_HOME goto findJavaFromJavaHome
@@ -78,16 +78,12 @@ set CMD_LINE_ARGS=%*
 
 :execute
 @rem Setup the command line
-
 set CLASSPATH=%APP_HOME%\gradle\wrapper\gradle-wrapper.jar
 
-@rem check if Gradle is installed
-if exist "%CLASSPATH%" goto GradleOK
-
-md %APP_HOME%\gradle\wrapper\
-Powershell.exe -executionpolicy remotesigned -File  %APP_HOME%\init-gradle-wrapper.ps1
+@rem Overwrites the local Gradle Wrapper files from tools repo in all cases (it's only 55kb).
+@rem This way we are sure to always use the latest version. At least when an Internet connexion is available.
+Powershell.exe -executionpolicy remotesigned -File %APP_HOME%\gradle\init-gradle-wrapper.ps1
 
-:GradleOK
 @rem Execute Gradle
 "%JAVA_EXE%" %DEFAULT_JVM_OPTS% %JAVA_OPTS% %GRADLE_OPTS% "-Dorg.gradle.appname=%APP_BASE_NAME%" -classpath "%CLASSPATH%" org.gradle.wrapper.GradleWrapperMain %CMD_LINE_ARGS%