You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@geode.apache.org by ud...@apache.org on 2018/04/16 21:47:21 UTC

[geode] branch feature/GEODE-4791 updated (e09c409 -> 22905ee)

This is an automated email from the ASF dual-hosted git repository.

udo pushed a change to branch feature/GEODE-4791
in repository https://gitbox.apache.org/repos/asf/geode.git.


    from e09c409  GEODE-4791: backing out upgrade to 4.5.1 due to an incompatibility with dockerized-test library, which only supports gradle 4.2
     new 9c60b5f  GEODE-4791: fixing extensions module project name references
     new 3c00fd0  WIP fixing up more gradle 4.x issues
     new 22905ee  GEODE-4791: WIP

The 3 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.


Summary of changes:
 .gitignore                                         |  2 ++
 build.gradle                                       |  8 ++---
 extensions/geode-modules-assembly/build.gradle     | 40 ++++++++++------------
 .../geode-modules-session-internal/build.gradle    |  2 +-
 extensions/geode-modules-session/build.gradle      |  3 +-
 extensions/geode-modules-tomcat7/build.gradle      |  4 +--
 extensions/geode-modules-tomcat8/build.gradle      |  6 ++--
 geode-assembly/build.gradle                        | 12 +++----
 gradle.properties                                  |  2 +-
 gradle/rat.gradle                                  |  1 +
 gradle/wrapper/gradle-wrapper.properties           |  2 +-
 settings.gradle                                    | 15 ++++----
 12 files changed, 48 insertions(+), 49 deletions(-)

-- 
To stop receiving notification emails like this one, please contact
udo@apache.org.

[geode] 02/03: WIP fixing up more gradle 4.x issues

Posted by ud...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

udo pushed a commit to branch feature/GEODE-4791
in repository https://gitbox.apache.org/repos/asf/geode.git

commit 3c00fd0c2a402327fde21c13c3790306d284bd4a
Author: Udo Kohlmeyer <uk...@pivotal.io>
AuthorDate: Wed Apr 11 11:03:29 2018 -0700

    WIP fixing up more gradle 4.x issues
---
 .gitignore                                     | 2 ++
 build.gradle                                   | 6 +++---
 extensions/geode-modules-assembly/build.gradle | 8 +++-----
 geode-assembly/build.gradle                    | 8 ++++----
 gradle.properties                              | 2 +-
 gradle/rat.gradle                              | 1 +
 gradle/wrapper/gradle-wrapper.properties       | 2 +-
 7 files changed, 15 insertions(+), 14 deletions(-)

diff --git a/.gitignore b/.gitignore
index af317b8..0bda1ca 100644
--- a/.gitignore
+++ b/.gitignore
@@ -27,3 +27,5 @@ out/
 *.orig
 geode-pulse/screenshots/
 /jpf.properties
+
+.gradletasknamecache
diff --git a/build.gradle b/build.gradle
index 1850d89..cbacc43 100755
--- a/build.gradle
+++ b/build.gradle
@@ -22,12 +22,12 @@ buildscript {
   }
 
   dependencies {
-    classpath "gradle.plugin.org.nosphere.apache:creadur-rat-gradle:0.2.0"
+    classpath "gradle.plugin.org.nosphere.apache:creadur-rat-gradle:0.3.1"
     classpath 'com.bmuschko:gradle-nexus-plugin:2.3.1'
     classpath 'org.sonarsource.scanner.gradle:sonarqube-gradle-plugin:2.6.1'
     classpath "com.diffplug.gradle.spotless:spotless:2.2.0"
-    classpath "me.champeau.gradle:jmh-gradle-plugin:0.3.1"
-    classpath "com.pedjak.gradle.plugins:dockerized-test:0.5.5"
+    classpath "me.champeau.gradle:jmh-gradle-plugin:0.4.4"
+    classpath "com.pedjak.gradle.plugins:dockerized-test:0.5.6"
     classpath 'com.github.ben-manes:gradle-versions-plugin:0.17.0'
   }
 }
diff --git a/extensions/geode-modules-assembly/build.gradle b/extensions/geode-modules-assembly/build.gradle
index 2bd2d3d..48d54f8 100644
--- a/extensions/geode-modules-assembly/build.gradle
+++ b/extensions/geode-modules-assembly/build.gradle
@@ -17,11 +17,6 @@
 
 import org.apache.tools.ant.filters.ReplaceTokens
 
-configurations {
-  slf4jDeps
-  moduleDistOutputs
-}
-
 jar.enabled = true
 extraArchive {
   sources = false
@@ -32,6 +27,9 @@ extraArchive {
 disableMavenPublishing()
 disableSigning()
 
+project.configurations.create('slf4jDeps')
+project.configurations.create('moduleDistOutputs')
+
 def getJarArtifact(module) {
   project(module).configurations.archives.artifacts.findAll {
     it instanceof PublishArtifact && it.type == 'jar' && it.classifier == ''
diff --git a/geode-assembly/build.gradle b/geode-assembly/build.gradle
index 9b4ed67..09665c8 100755
--- a/geode-assembly/build.gradle
+++ b/geode-assembly/build.gradle
@@ -419,9 +419,9 @@ distributions {
         from (project(":geode-pulse").configurations.archives.allArtifacts.files)
       }
 
-//      into ('tools/Modules') {
-//        from (project(':extensions:geode-modules-assembly').configurations.moduleDistOutputs.files)
-//      }
+      into ('tools/Modules') {
+        from (project(':extensions:geode-modules-assembly').configurations.moduleDistOutputs.files)
+      }
     }
   }
 }
@@ -444,7 +444,7 @@ flakyTest dependOnInstalledProduct
 build.dependsOn installDist
 
 installDist.dependsOn ':extensions:geode-modules-assembly:dist'
-distributedTest.dependsOn ':extensions/session-testing-war:war'
+distributedTest.dependsOn ':extensions:session-testing-war:war'
 distributedTest.dependsOn ':geode-old-versions:build'
 
 /**Print the names of all jar files in a fileTree */
diff --git a/gradle.properties b/gradle.properties
index 30c92af..c8d8ed9 100755
--- a/gradle.properties
+++ b/gradle.properties
@@ -46,7 +46,7 @@ org.gradle.parallel=true
 org.gradle.configureondemand=true
 org.gradle.workers.max=4
 
-minimumGradleVersion = 4.2.1
+minimumGradleVersion = 4.6
 # Set this on the command line with -P or in ~/.gradle/gradle.properties
 # to change the buildDir location.  Use an absolute path.
 buildRoot=
diff --git a/gradle/rat.gradle b/gradle/rat.gradle
index b0012af..74ae2f9 100644
--- a/gradle/rat.gradle
+++ b/gradle/rat.gradle
@@ -39,6 +39,7 @@ rat {
     '**/build/**',
     '**/build-*/**',
     '.buildinfo',
+    '.gradletasknamecache',
 
     // Geode examples
     'geode-examples/.idea/**',
diff --git a/gradle/wrapper/gradle-wrapper.properties b/gradle/wrapper/gradle-wrapper.properties
index 4516ad8..dcb0cb1 100644
--- a/gradle/wrapper/gradle-wrapper.properties
+++ b/gradle/wrapper/gradle-wrapper.properties
@@ -3,4 +3,4 @@ distributionBase=GRADLE_USER_HOME
 distributionPath=wrapper/dists
 zipStoreBase=GRADLE_USER_HOME
 zipStorePath=wrapper/dists
-distributionUrl=https\://services.gradle.org/distributions/gradle-4.2.1-bin.zip
+distributionUrl=https\://services.gradle.org/distributions/gradle-4.6-bin.zip

-- 
To stop receiving notification emails like this one, please contact
udo@apache.org.

[geode] 01/03: GEODE-4791: fixing extensions module project name references

Posted by ud...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

udo pushed a commit to branch feature/GEODE-4791
in repository https://gitbox.apache.org/repos/asf/geode.git

commit 9c60b5ff7fb49e235f4a19e6fff241ccfb5be888
Author: Udo Kohlmeyer <uk...@pivotal.io>
AuthorDate: Tue Apr 10 15:39:13 2018 -0700

    GEODE-4791: fixing extensions module project name references
---
 build.gradle                                       |  2 +-
 extensions/geode-modules-assembly/build.gradle     | 32 +++++++++++-----------
 .../geode-modules-session-internal/build.gradle    |  2 +-
 extensions/geode-modules-session/build.gradle      |  3 +-
 extensions/geode-modules-tomcat7/build.gradle      |  4 +--
 extensions/geode-modules-tomcat8/build.gradle      |  6 ++--
 geode-assembly/build.gradle                        | 14 +++++-----
 settings.gradle                                    | 15 +++++-----
 8 files changed, 38 insertions(+), 40 deletions(-)

diff --git a/build.gradle b/build.gradle
index 56005a2..1850d89 100755
--- a/build.gradle
+++ b/build.gradle
@@ -88,7 +88,7 @@ apply from: "${scriptDir}/dependency-resolution.gradle"
 apply from: "${scriptDir}/test.gradle"
 apply from: "${scriptDir}/publish.gradle"
 apply from: "${scriptDir}/code-analysis.gradle"
-apply from: "${scriptDir}/sonar.gradle"
+//apply from: "${scriptDir}/sonar.gradle"
 apply from: "${scriptDir}/ide.gradle"
 apply from: "${scriptDir}/rat.gradle"
 apply from: "${scriptDir}/docker.gradle"
diff --git a/extensions/geode-modules-assembly/build.gradle b/extensions/geode-modules-assembly/build.gradle
index 6abd47b..2bd2d3d 100644
--- a/extensions/geode-modules-assembly/build.gradle
+++ b/extensions/geode-modules-assembly/build.gradle
@@ -46,9 +46,9 @@ def configureTcServerAssembly = {
 
   // All client-server files
   into('geode-cs/lib') {
-    from getJarArtifact(':extensions/geode-modules')
-    from getJarArtifact(':extensions/geode-modules-tomcat7')
-    from getJarArtifact(':extensions/geode-modules-tomcat8')
+    from getJarArtifact(':extensions:geode-modules')
+    from getJarArtifact(':extensions:geode-modules-tomcat7')
+    from getJarArtifact(':extensions:geode-modules-tomcat8')
     from configurations.slf4jDeps
   }
   into('geode-cs/bin') {
@@ -79,9 +79,9 @@ def configureTcServerAssembly = {
   }
   // All peer-to-peer files
   into('geode-p2p/lib') {
-    from getJarArtifact(':extensions/geode-modules')
-    from getJarArtifact(':extensions/geode-modules-tomcat7')
-    from getJarArtifact(':extensions/geode-modules-tomcat8')
+    from getJarArtifact(':extensions:geode-modules')
+    from getJarArtifact(':extensions:geode-modules-tomcat7')
+    from getJarArtifact(':extensions:geode-modules-tomcat8')
     from configurations.slf4jDeps
   }
   into('geode-p2p/bin') {
@@ -130,15 +130,15 @@ def configureTcServer30Assembly = {
   }
 }
 
-task distTomcat(type: Zip, dependsOn: ':extensions/geode-modules:assemble') {
+task distTomcat(type: Zip, dependsOn: ':extensions:geode-modules:assemble') {
   baseName = moduleBaseName
   classifier = "Tomcat"
 
   // All client-server files
   into('lib') {
-    from getJarArtifact(':extensions/geode-modules')
-    from getJarArtifact(':extensions/geode-modules-tomcat7')
-    from getJarArtifact(':extensions/geode-modules-tomcat8')
+    from getJarArtifact(':extensions:geode-modules')
+    from getJarArtifact(':extensions:geode-modules-tomcat7')
+    from getJarArtifact(':extensions:geode-modules-tomcat8')
     from configurations.slf4jDeps
   }
   into('bin') {
@@ -154,14 +154,14 @@ task distTomcat(type: Zip, dependsOn: ':extensions/geode-modules:assemble') {
   }
 }
 
-task distAppServer(type: Zip, dependsOn: ':extensions/geode-modules-session:assemble') {
+task distAppServer(type: Zip, dependsOn: ':extensions:geode-modules-session:assemble') {
   baseName = moduleBaseName
   classifier = "AppServer"
 
   into('lib') {
-    from getJarArtifact(':extensions/geode-modules-session')
-    from getJarArtifact(':extensions/geode-modules-session-internal')
-    from getJarArtifact(':extensions/geode-modules')
+    from getJarArtifact(':extensions:geode-modules-session')
+    from getJarArtifact(':extensions:geode-modules-session-internal')
+    from getJarArtifact(':extensions:geode-modules')
     from configurations.slf4jDeps
   }
 
@@ -187,11 +187,11 @@ task distAppServer(type: Zip, dependsOn: ':extensions/geode-modules-session:asse
   }
 }
 
-task distTcServer(type: Zip, dependsOn: [':extensions/geode-modules:assemble', ':extensions/geode-modules-tomcat7:assemble', ':extensions/geode-modules-tomcat8:assemble'] ) {
+task distTcServer(type: Zip, dependsOn: [':extensions:geode-modules:assemble', ':extensions:geode-modules-tomcat7:assemble', ':extensions:geode-modules-tomcat8:assemble'] ) {
   configure(configureTcServerAssembly)
 }
 
-task distTcServer30(type: Zip, dependsOn: [':extensions/geode-modules:assemble', ':extensions/geode-modules-tomcat7:assemble', ':extensions/geode-modules-tomcat8:assemble']) {
+task distTcServer30(type: Zip, dependsOn: [':extensions:geode-modules:assemble', ':extensions:geode-modules-tomcat7:assemble', ':extensions:geode-modules-tomcat8:assemble']) {
   configure(configureTcServerAssembly)
   configure(configureTcServer30Assembly)
 }
diff --git a/extensions/geode-modules-session-internal/build.gradle b/extensions/geode-modules-session-internal/build.gradle
index 39c6e55..e1dcc1c 100644
--- a/extensions/geode-modules-session-internal/build.gradle
+++ b/extensions/geode-modules-session-internal/build.gradle
@@ -16,7 +16,7 @@
  */
 
 dependencies {
-  compile project(':extensions/geode-modules')
+  compile project(':extensions:geode-modules')
   compile project(':geode-core')
 }
 
diff --git a/extensions/geode-modules-session/build.gradle b/extensions/geode-modules-session/build.gradle
index 0469070..a1ffa0f 100644
--- a/extensions/geode-modules-session/build.gradle
+++ b/extensions/geode-modules-session/build.gradle
@@ -14,9 +14,8 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
-
 dependencies {
-  compile project(':extensions/geode-modules-session-internal')
+  compile project(':extensions:geode-modules-session-internal')
   compile project(':geode-core')
 
   testCompile 'javax.servlet:javax.servlet-api:' + project.'javax.servlet-api.version'
diff --git a/extensions/geode-modules-tomcat7/build.gradle b/extensions/geode-modules-tomcat7/build.gradle
index 21096f9..5f913d3 100644
--- a/extensions/geode-modules-tomcat7/build.gradle
+++ b/extensions/geode-modules-tomcat7/build.gradle
@@ -16,7 +16,7 @@
  */
 
 dependencies {
-  compile(project(':extensions/geode-modules')) {
+  compile(project(':extensions:geode-modules')) {
     // Remove everything related to Tomcat 6.x
     exclude group: 'org.apache.tomcat'
   }
@@ -39,7 +39,7 @@ dependencies {
 
   testCompile project(path: ':geode-junit')
   testCompile files(project(':geode-core').sourceSets.test.output)
-  testCompile files(project(':extensions/geode-modules').sourceSets.test.output)
+  testCompile files(project(':extensions:geode-modules').sourceSets.test.output)
 
   eclipse.classpath.file {
     whenMerged { classpath ->
diff --git a/extensions/geode-modules-tomcat8/build.gradle b/extensions/geode-modules-tomcat8/build.gradle
index e26123f..111acd8 100644
--- a/extensions/geode-modules-tomcat8/build.gradle
+++ b/extensions/geode-modules-tomcat8/build.gradle
@@ -14,10 +14,8 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
-
-
 dependencies {
-  compile(project(':extensions/geode-modules')) {
+  compile(project(':extensions:geode-modules')) {
     exclude group: 'org.apache.tomcat'
   }
 
@@ -40,7 +38,7 @@ dependencies {
 
   testCompile project(path: ':geode-junit')
   testCompile files(project(':geode-core').sourceSets.test.output)
-  testCompile files(project(':extensions/geode-modules').sourceSets.test.output)
+  testCompile files(project(':extensions:geode-modules').sourceSets.test.output)
 
   eclipse.classpath.file {
     whenMerged { classpath ->
diff --git a/geode-assembly/build.gradle b/geode-assembly/build.gradle
index 02cbf81..9b4ed67 100755
--- a/geode-assembly/build.gradle
+++ b/geode-assembly/build.gradle
@@ -84,16 +84,16 @@ dependencies {
 
   testCompile group: 'org.codehaus.cargo', name: 'cargo-core-uberjar', version: '1.6.3'
 
-  testCompile (project(':extensions/geode-modules')) {
+  testCompile (project(':extensions:geode-modules')) {
     // Remove everything related to Tomcat
     exclude group: 'org.apache.tomcat'
   }
-  testCompile (project(':extensions/geode-modules-session-internal')) {
+  testCompile (project(':extensions:geode-modules-session-internal')) {
     // Remove everything related to Tomcat
     exclude group: 'org.apache.tomcat'
   }
 
-  testCompile project(':extensions/session-testing-war')
+  testCompile project(':extensions:session-testing-war')
 
   testRuntime files("${System.getProperty('java.home')}/../lib/tools.jar")
   testRuntime files("$buildDir/install/${distributions.main.baseName}/lib/geode-dependencies.jar")
@@ -419,9 +419,9 @@ distributions {
         from (project(":geode-pulse").configurations.archives.allArtifacts.files)
       }
 
-      into ('tools/Modules') {
-        from (project(':extensions/geode-modules-assembly').configurations.moduleDistOutputs.files)
-      }
+//      into ('tools/Modules') {
+//        from (project(':extensions:geode-modules-assembly').configurations.moduleDistOutputs.files)
+//      }
     }
   }
 }
@@ -443,7 +443,7 @@ flakyTest dependOnInstalledProduct
 // Make build final task to generate all test and product resources
 build.dependsOn installDist
 
-installDist.dependsOn ':extensions/geode-modules-assembly:dist'
+installDist.dependsOn ':extensions:geode-modules-assembly:dist'
 distributedTest.dependsOn ':extensions/session-testing-war:war'
 distributedTest.dependsOn ':geode-old-versions:build'
 
diff --git a/settings.gradle b/settings.gradle
index 74c1811..160d9a8 100644
--- a/settings.gradle
+++ b/settings.gradle
@@ -31,16 +31,17 @@ include 'geode-old-client-support'
 include 'geode-wan'
 include 'geode-cq'
 include 'geode-connectors'
-include 'extensions/geode-modules'
-include 'extensions/geode-modules-tomcat7'
-include 'extensions/geode-modules-tomcat8'
-include 'extensions/geode-modules-session-internal'
-include 'extensions/geode-modules-session'
-include 'extensions/geode-modules-assembly'
+include 'extensions:geode-modules'
+include 'extensions:geode-modules-tomcat7'
+include 'extensions:geode-modules-tomcat8'
+include 'extensions:geode-modules-session-internal'
+include 'extensions:geode-modules-session'
+include 'extensions:geode-modules-assembly'
+include 'extensions:session-testing-war'
 include 'geode-protobuf'
 include 'geode-experimental-driver'
 include 'geode-protobuf-messages'
-include 'extensions/session-testing-war'
+
 include 'geode-concurrency-test'
 
 

-- 
To stop receiving notification emails like this one, please contact
udo@apache.org.

[geode] 03/03: GEODE-4791: WIP

Posted by ud...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

udo pushed a commit to branch feature/GEODE-4791
in repository https://gitbox.apache.org/repos/asf/geode.git

commit 22905ee4bac13664bf7ea9e01445aa765248e552
Author: Udo Kohlmeyer <uk...@pivotal.io>
AuthorDate: Mon Apr 16 14:47:07 2018 -0700

    GEODE-4791: WIP
---
 geode-assembly/build.gradle | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/geode-assembly/build.gradle b/geode-assembly/build.gradle
index 09665c8..f7872c7 100755
--- a/geode-assembly/build.gradle
+++ b/geode-assembly/build.gradle
@@ -420,7 +420,7 @@ distributions {
       }
 
       into ('tools/Modules') {
-        from (project(':extensions:geode-modules-assembly').configurations.moduleDistOutputs.files)
+        from (project(':extensions:geode-modules-assembly').configurations.archives.allArtifacts.files)
       }
     }
   }

-- 
To stop receiving notification emails like this one, please contact
udo@apache.org.