Skip to content

Commit c0bf3e8

Browse files
committed
Revert "java matrix build"
This reverts commit d750cad.
1 parent d750cad commit c0bf3e8

3 files changed

Lines changed: 45 additions & 24 deletions

File tree

.github/workflows/maven-tests.yml

Lines changed: 26 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -17,24 +17,15 @@ env:
1717
permissions:
1818
contents: read
1919

20-
concurrency:
21-
group: maven-tests-${{ github.ref }}
22-
cancel-in-progress: true
23-
2420
jobs:
25-
build:
26-
name: Build and Test (JDK ${{ matrix.java }})
21+
test:
22+
name: Test (JDK ${{ matrix.java }})
2723
runs-on: ubuntu-latest
2824
strategy:
2925
fail-fast: false
3026
matrix:
3127
java:
3228
- '8'
33-
- '17'
34-
- '18'
35-
- '19'
36-
- '20'
37-
- '21'
3829

3930
steps:
4031
- name: Checkout
@@ -47,8 +38,8 @@ jobs:
4738
java-version: ${{ matrix.java }}
4839
cache: maven
4940

50-
- name: Build and run tests
51-
run: mvn -B -ntp verify
41+
- name: Run tests
42+
run: mvn -B -ntp test
5243

5344
- name: Upload surefire reports
5445
if: always()
@@ -58,12 +49,27 @@ jobs:
5849
path: target/surefire-reports
5950
if-no-files-found: warn
6051

61-
- name: Upload packaged jars
62-
if: success() && (matrix.java == '8' || matrix.java == '21')
63-
uses: actions/upload-artifact@v4
52+
package:
53+
name: Package (JDK ${{ matrix.java }})
54+
runs-on: ubuntu-latest
55+
needs: test
56+
strategy:
57+
fail-fast: false
58+
matrix:
59+
java:
60+
- '8'
61+
62+
steps:
63+
- name: Checkout
64+
uses: actions/checkout@v4
65+
66+
- name: Set up JDK
67+
uses: actions/setup-java@v4
6468
with:
65-
name: java-code-tracer-jars-jdk-${{ matrix.java }}
66-
path: |
67-
target/*.jar
68-
if-no-files-found: error
69+
distribution: temurin
70+
java-version: ${{ matrix.java }}
71+
cache: maven
72+
73+
- name: Build package
74+
run: mvn -B -ntp package -DskipTests
6975

README.md

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -103,8 +103,6 @@ This is a deliberate choice — the primary target is legacy and monolithic syst
103103

104104
It runs fine on newer JVMs (11, 17, 21, …) without any changes.
105105

106-
The GitHub Actions build validates the project on JDK 8 and on JDK 17 through 21.
107-
108106
## Build
109107

110108
```bash

pom.xml

Lines changed: 19 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,8 @@
1212
<properties>
1313
<project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
1414
<java.version>8</java.version>
15-
<maven.compiler.release>${java.version}</maven.compiler.release>
15+
<maven.compiler.source>${java.version}</maven.compiler.source>
16+
<maven.compiler.target>${java.version}</maven.compiler.target>
1617
<maven.surefire.plugin.version>3.2.5</maven.surefire.plugin.version>
1718
<maven.compiler.plugin.version>3.13.0</maven.compiler.plugin.version>
1819
<maven.assembly.plugin.version>3.7.1</maven.assembly.plugin.version>
@@ -31,7 +32,22 @@
3132
</license>
3233
</licenses>
3334

35+
<repositories>
36+
<repository>
37+
<id>Sonatype-public</id>
38+
<name>SnakeYAML repository</name>
39+
<url>http://oss.sonatype.org/content/groups/public/</url>
40+
</repository>
41+
</repositories>
42+
3443
<dependencies>
44+
<dependency>
45+
<groupId>com.sun</groupId>
46+
<artifactId>tools</artifactId>
47+
<version>1.6.0</version>
48+
<scope>system</scope>
49+
<systemPath>${java.home}/../lib/tools.jar</systemPath>
50+
</dependency>
3551
<dependency>
3652
<groupId>org.yaml</groupId>
3753
<artifactId>snakeyaml</artifactId>
@@ -81,7 +97,8 @@
8197
<artifactId>maven-compiler-plugin</artifactId>
8298
<version>${maven.compiler.plugin.version}</version>
8399
<configuration>
84-
<release>${maven.compiler.release}</release>
100+
<source>${maven.compiler.source}</source>
101+
<target>${maven.compiler.target}</target>
85102
</configuration>
86103
</plugin>
87104

0 commit comments

Comments
 (0)