Skip to content
This repository was archived by the owner on Mar 6, 2024. It is now read-only.

Commit ddac87b

Browse files
committed
build: upgrade to java 11 and groovy 3
Update the upgrade-framework to compile using Java 11 and Groovy 3. Had to switch to using gobuild's MakeHelper instead of MavenHelper because MavenHelper only looks at versions of maven available in toolchain, which is not being updated anymore. Had to make a variety of javadoc changes to appease the newer version maven-javadoc-plugin which seems to be more strict. I just picked the path of least resistance in modifying them. Also updated the project's version to 2.0.0 and replaced travis with github actions. Signed-off-by: Ryan Lewis <[email protected]>
1 parent d6c8fa3 commit ddac87b

File tree

17 files changed

+72
-47
lines changed

17 files changed

+72
-47
lines changed

Diff for: .github/workflows/maven.yml

+27
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,27 @@
1+
# This workflow will build a Java project with Maven, and cache/restore any dependencies to improve the workflow execution time
2+
# For more information see: https://help.github.com/actions/language-and-framework-guides/building-and-testing-java-with-maven
3+
4+
name: Java CI with Maven
5+
6+
on:
7+
push:
8+
branches: [ "master" ]
9+
pull_request:
10+
branches: [ "master" ]
11+
12+
jobs:
13+
build:
14+
15+
runs-on: ubuntu-latest
16+
17+
steps:
18+
- uses: actions/checkout@v3
19+
- name: Set up JDK 11
20+
uses: actions/setup-java@v3
21+
with:
22+
java-version: '11'
23+
distribution: 'temurin'
24+
cache: maven
25+
- name: Build with Maven
26+
run: mvn -B install --file pom.xml
27+

Diff for: .travis.yml

-8
This file was deleted.

Diff for: pom.xml

+15-13
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
<modelVersion>4.0.0</modelVersion>
77
<groupId>com.vmware.vcloud</groupId>
88
<artifactId>upgrade-framework-parent</artifactId>
9-
<version>1.0.1</version>
9+
<version>2.0.0</version>
1010

1111
<name>upgrade-framework-parent - Upgrade Framework</name>
1212
<url>https://github.com/vmware/upgrade-framework/</url>
@@ -30,9 +30,9 @@
3030
<commons.lang3.version>3.4</commons.lang3.version>
3131
<commons.math.version>1.2</commons.math.version>
3232
<easymock.version>3.1</easymock.version>
33-
<groovy.eclipse.batch.version>2.4.3-01</groovy.eclipse.batch.version>
34-
<groovy.eclipse.compiler.version>2.9.2-01</groovy.eclipse.compiler.version>
35-
<groovy.version>2.4.8</groovy.version>
33+
<groovy.eclipse.batch.version>3.0.5-01</groovy.eclipse.batch.version>
34+
<groovy.eclipse.compiler.version>3.6.0-03</groovy.eclipse.compiler.version>
35+
<groovy.version>3.0.7</groovy.version>
3636
<jackson.version>2.9.10</jackson.version>
3737
<jackson.databind.version>2.9.10.1</jackson.databind.version>
3838
<log4j.version>1.2.17</log4j.version>
@@ -70,6 +70,7 @@
7070
<groupId>org.codehaus.groovy</groupId>
7171
<artifactId>groovy-all</artifactId>
7272
<version>${groovy.version}</version>
73+
<type>pom</type>
7374
</dependency>
7475

7576
<dependency>
@@ -170,14 +171,15 @@
170171
<plugins>
171172
<plugin>
172173
<artifactId>maven-assembly-plugin</artifactId>
173-
<version>2.4</version>
174+
<version>3.3.0</version>
174175
</plugin>
175176

176177
<!-- This pulls in the MANIFEST.MF file that is maintained by the developers.
177178
Without this stanza, the JAR would generate a manifest. -->
178179
<plugin>
179180
<groupId>org.apache.maven.plugins</groupId>
180181
<artifactId>maven-jar-plugin</artifactId>
182+
<version>2.3.2</version>
181183
<configuration>
182184
<archive>
183185
<manifestFile>${project.basedir}/META-INF/MANIFEST.MF</manifestFile>
@@ -199,7 +201,7 @@
199201
<plugin>
200202
<groupId>org.apache.felix</groupId>
201203
<artifactId>maven-bundle-plugin</artifactId>
202-
<version>2.3.7</version>
204+
<version>4.0.0</version>
203205
<executions>
204206
<execution>
205207
<id>bundle-manifest</id>
@@ -226,28 +228,27 @@
226228
<plugins>
227229
<plugin>
228230
<artifactId>maven-clean-plugin</artifactId>
229-
<version>2.5</version>
231+
<version>3.1.0</version>
230232
</plugin>
231233

232234
<plugin>
233235
<artifactId>maven-compiler-plugin</artifactId>
234-
<version>3.1</version>
236+
<version>3.8.1</version>
235237
<configuration>
236238
<encoding>utf8</encoding>
237-
<source>1.8</source>
238-
<target>1.8</target>
239+
<release>11</release>
239240
</configuration>
240241
</plugin>
241242

242243
<plugin>
243244
<artifactId>maven-dependency-plugin</artifactId>
244-
<version>2.5.1</version>
245+
<version>2.8</version>
245246
</plugin>
246247

247248
<plugin>
248249
<groupId>org.apache.maven.plugins</groupId>
249250
<artifactId>maven-surefire-plugin</artifactId>
250-
<version>2.12.1</version>
251+
<version>3.0.0-M5</version>
251252
<configuration>
252253
<groups>Unit, Minimum</groups>
253254
<suiteXmlFiles>
@@ -259,7 +260,7 @@
259260
<plugin>
260261
<groupId>org.apache.maven.plugins</groupId>
261262
<artifactId>maven-javadoc-plugin</artifactId>
262-
<version>2.8</version>
263+
<version>3.0.1</version>
263264
<configuration>
264265
<show>public</show>
265266
</configuration>
@@ -289,6 +290,7 @@
289290

290291
<plugin>
291292
<artifactId>maven-source-plugin</artifactId>
293+
<version>2.2</version>
292294
<executions>
293295
<execution>
294296
<phase>package</phase>

Diff for: upgrade-framework-core/instructions.bnd

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
11
Bundle-SymbolicName: com.vmware.upgrade
2-
Bundle-RequiredExecutionEnvironment: JavaSE-1.6
2+
Bundle-RequiredExecutionEnvironment: JavaSE-11
33
Export-Package: *
44
Import-Package: *

Diff for: upgrade-framework-core/pom.xml

+1-1
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@
1414
<parent>
1515
<groupId>com.vmware.vcloud</groupId>
1616
<artifactId>upgrade-framework-parent</artifactId>
17-
<version>1.0.1</version>
17+
<version>2.0.0</version>
1818
<relativePath>../pom.xml</relativePath>
1919
</parent>
2020

Diff for: upgrade-framework-distribution/pom.xml

+1-1
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@
1414
<parent>
1515
<groupId>com.vmware.vcloud</groupId>
1616
<artifactId>upgrade-framework-parent</artifactId>
17-
<version>1.0.1</version>
17+
<version>2.0.0</version>
1818
</parent>
1919

2020
<dependencies>

Diff for: upgrade-framework-dsl/instructions.bnd

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
11
Bundle-SymbolicName: com.vmware.upgrade.dsl
2-
Bundle-RequiredExecutionEnvironment: JavaSE-1.6
2+
Bundle-RequiredExecutionEnvironment: JavaSE-11
33
Export-Package: *
44
Import-Package: org.apache.commons.lang, *

Diff for: upgrade-framework-dsl/pom.xml

+3-1
Original file line numberDiff line numberDiff line change
@@ -14,14 +14,16 @@
1414
<parent>
1515
<groupId>com.vmware.vcloud</groupId>
1616
<artifactId>upgrade-framework-parent</artifactId>
17-
<version>1.0.1</version>
17+
<version>2.0.0</version>
1818
<relativePath>../pom.xml</relativePath>
1919
</parent>
2020

2121
<dependencies>
2222
<dependency>
2323
<groupId>org.codehaus.groovy</groupId>
2424
<artifactId>groovy-all</artifactId>
25+
<version>${groovy.version}</version>
26+
<type>pom</type>
2527
</dependency>
2628

2729
<dependency>

Diff for: upgrade-framework-dsl/src/main/java/com/vmware/upgrade/dsl/model/package-info.java

+4-4
Original file line numberDiff line numberDiff line change
@@ -24,10 +24,10 @@
2424
* The core model objects which compose the domain specific language for defining upgrade
2525
* framework constructs.
2626
* <p>
27-
* A {@link com.vmware.upgrade.dsl.model.ManifestModel} consists of
28-
* {@link com.vmware.upgrade.dsl.model.UpgradeTaskModel}s which reference
29-
* {@link com.vmware.upgrade.dsl.model.UpgradeDefinitionModel}s within
30-
* {@link com.vmware.upgrade.dsl.model.NamespaceModel}s
27+
* A com.vmware.upgrade.dsl.model.ManifestModel consists of
28+
* com.vmware.upgrade.dsl.model.UpgradeTaskModels which reference
29+
* com.vmware.upgrade.dsl.model.UpgradeDefinitionModels within
30+
* com.vmware.upgrade.dsl.model.NamespaceModels
3131
* <p>
3232
* (Package contains Groovy classes.)
3333
*

Diff for: upgrade-framework-dsl/src/main/java/com/vmware/upgrade/dsl/package-info.java

+6-6
Original file line numberDiff line numberDiff line change
@@ -31,13 +31,13 @@
3131
* the syntax classes for constructing this model are in {@link com.vmware.upgrade.dsl.syntax}.
3232
* <p>
3333
* This package contains the objects a consumer of the language will need to directly interface
34-
* with. {@link com.vmware.upgrade.dsl.Loader} is the primary entry point for consuming a script.
34+
* with. com.vmware.upgrade.dsl.Loader is the primary entry point for consuming a script.
3535
* Implementations of {@link com.vmware.upgrade.dsl.Processor} and
3636
* {@link com.vmware.upgrade.dsl.TaskResolver} are used to extend the language and control the
3737
* {@link com.vmware.upgrade.Task}s which are produced.
3838
* <h3>A single-file example</h3>
3939
* This example demonstrates defining an upgrade and sequencing it in a manifest in a single file.
40-
* <h4><tt>src/main/resources/upgrade/upgrade.groovy</tt></h4>
40+
* <h4><code>src/main/resources/upgrade/upgrade.groovy</code></h4>
4141
* <pre><code>
4242
* foo = upgrade {
4343
* name "Upgrade to do 'foo'"
@@ -74,7 +74,7 @@
7474
* <h3>A multi-file example</h3>
7575
* This example demonstrates defining an upgrade within a namespace in a file included from a
7676
* separate file which defines a manifest which sequences the upgrade.
77-
* <h4><tt>src/main/resources/upgrade/baz.groovy</tt></h4>
77+
* <h4><code>src/main/resources/upgrade/baz.groovy</code></h4>
7878
* <pre><code>
7979
* bar = namespace {
8080
* foo = upgrade {
@@ -84,7 +84,7 @@
8484
* }
8585
* }
8686
* </code></pre>
87-
* <h4><tt>src/main/resources/upgrade/manifest.groovy</tt></h4>
87+
* <h4><code>src/main/resources/upgrade/manifest.groovy</code></h4>
8888
* <pre><code>
8989
* import "baz.groovy"
9090
*
@@ -98,7 +98,7 @@
9898
* </code></pre>
9999
* <h3>A manifest file for a complex graph</h3>
100100
* This example demonstrates the ability defining a set of upgrades to form a non-trivial graph.
101-
* <h4><tt>src/main/resources/upgrade/manifest.groovy</tt></h4>
101+
* <h4><code>src/main/resources/upgrade/manifest.groovy</code></h4>
102102
* <pre><code>
103103
* import "upgrades.groovy"
104104
*
@@ -123,7 +123,7 @@
123123
* <h3>An upgrade definition with complex orchestration</h3>
124124
* This example demonstrates the ability to define an upgrade that explicitly controls the way in
125125
* which a set of {@link com.vmware.upgrade.Task}s are executed.
126-
* <h4><tt>src/main/resources/upgrade/upgrade.groovy</tt></h4>
126+
* <h4><code>src/main/resources/upgrade/upgrade.groovy</code></h4>
127127
* <pre><code>
128128
* foo = upgrade {
129129
* name "Run a complex set of tasks"

Diff for: upgrade-framework-dsl/src/main/java/com/vmware/upgrade/dsl/syntax/package-info.java

+1-1
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@
2323
/**
2424
* The core syntax objects which define the domain specific language for the upgrade framework.
2525
* <p>
26-
* {@link com.vmware.upgrade.dsl.syntax.ScriptSyntax} serves as the primary entry point for
26+
* com.vmware.upgrade.dsl.syntax.ScriptSyntax serves as the primary entry point for
2727
* processing scripts.
2828
* <p>
2929
* (Package contains Groovy classes.)

Diff for: upgrade-framework-sql-dsl/instructions.bnd

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
Bundle-SymbolicName: com.vmware.upgrade.sql.dsl
2-
Bundle-RequiredExecutionEnvironment: JavaSE-1.6
2+
Bundle-RequiredExecutionEnvironment: JavaSE-11
33
Export-Package: *
44
Import-Package: com.vmware.upgrade.dsl.syntax,\
55
com.vmware.upgrade.sql.task,\

Diff for: upgrade-framework-sql-dsl/pom.xml

+3-1
Original file line numberDiff line numberDiff line change
@@ -14,14 +14,16 @@
1414
<parent>
1515
<groupId>com.vmware.vcloud</groupId>
1616
<artifactId>upgrade-framework-parent</artifactId>
17-
<version>1.0.1</version>
17+
<version>2.0.0</version>
1818
<relativePath>../pom.xml</relativePath>
1919
</parent>
2020

2121
<dependencies>
2222
<dependency>
2323
<groupId>org.codehaus.groovy</groupId>
2424
<artifactId>groovy-all</artifactId>
25+
<version>${groovy.version}</version>
26+
<type>pom</type>
2527
</dependency>
2628

2729
<dependency>

Diff for: upgrade-framework-sql-dsl/src/main/java/com/vmware/upgrade/dsl/sql/package-info.java

+5-6
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@
2828
* <h3>A simple example</h3>
2929
* This example demonstrates defining an upgrade which defines a single SQL statement to be
3030
* executed for all database types.
31-
* <h4><tt>src/main/resources/upgrade/upgrade.groovy</tt></h4>
31+
* <h4><code>src/main/resources/upgrade/upgrade.groovy</code></h4>
3232
* <pre><code>
3333
* foo = upgrade {
3434
* name "Upgrade to do 'foo'"
@@ -69,7 +69,7 @@
6969
* The first SQL statement explicitly specifies the SQL to use for each database type while the
7070
* second demonstrates defining a default value used for all database types for which no SQL is
7171
* explicitly defined.
72-
* <h4><tt>src/main/resources/upgrade/upgrade.groovy</tt></h4>
72+
* <h4><code>src/main/resources/upgrade/upgrade.groovy</code></h4>
7373
* <pre><code>
7474
* foo = upgrade {
7575
* name "Upgrade to do 'foo'"
@@ -86,10 +86,10 @@
8686
* from "" to "1.0.0" call foo
8787
* }
8888
* </code></pre>
89-
* <h3>Leveraging {@link com.vmware.upgrade.dsl.sql.util.AgnosticSqlProcessor}</h3>
89+
* <h3>Leveraging com.vmware.upgrade.dsl.sql.util.AgnosticSqlProcessor</h3>
9090
* This example demonstrates defining an upgrade which using keywords processed by
91-
* {@link com.vmware.upgrade.dsl.sql.util.AgnosticSqlProcessor}.
92-
* <h4><tt>src/main/resources/upgrade/upgrade.groovy</tt></h4>
91+
* com.vmware.upgrade.dsl.sql.util.AgnosticSqlProcessor.
92+
* <h4><code>src/main/resources/upgrade/upgrade.groovy</code></h4>
9393
* <pre><code>
9494
* foo = upgrade {
9595
* name "Upgrade to do 'foo'"
@@ -141,4 +141,3 @@
141141
* @since 1.0
142142
*/
143143
package com.vmware.upgrade.dsl.sql;
144-

Diff for: upgrade-framework-sql-dsl/src/main/java/com/vmware/upgrade/dsl/sql/util/NullAware.java

+1
Original file line numberDiff line numberDiff line change
@@ -34,6 +34,7 @@ public interface NullAware extends HasClosureMap {
3434
* Set the allowing of null values.
3535
*
3636
* @param arg the {@link Object} used to determine allowing of null values
37+
* @return Object
3738
*/
3839
public Object makeNullable(Object arg);
3940

Diff for: upgrade-framework-sql/instructions.bnd

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
11
Bundle-SymbolicName: com.vmware.upgrade.sql
2-
Bundle-RequiredExecutionEnvironment: JavaSE-1.6
2+
Bundle-RequiredExecutionEnvironment: JavaSE-11
33
Import-Package: *
44
Export-Package: *

Diff for: upgrade-framework-sql/pom.xml

+1-1
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@
1414
<parent>
1515
<groupId>com.vmware.vcloud</groupId>
1616
<artifactId>upgrade-framework-parent</artifactId>
17-
<version>1.0.1</version>
17+
<version>2.0.0</version>
1818
<relativePath>../pom.xml</relativePath>
1919
</parent>
2020

0 commit comments

Comments
 (0)