Skip to content

Commit e284817

Browse files
authored
TIKA-4609: Fix Maven verbosity flags (remove line breaks) (#2510)
* TIKA-4609: Fix Maven verbosity flags (remove line breaks) The previous commit had line breaks in the property name which broke the Maven command. This commit fixes it by keeping the full property name on one line without spaces: -Dorg.slf4j.simpleLogger.log.org.apache.maven.cli.transfer.Slf4jMavenTransferListener=warn Changes: - Fix all 5 workflow files to use correct property syntax - Remove accidental spaces/line breaks in property name - Properly suppress Maven transfer listener output * TIKA-4609: Quote Maven property and add Maven caching Two fixes: 1. Quote the -D property to fix Windows parsing issue - Windows PowerShell/CMD was treating the space before -D as argument separator - Wrapping in quotes: "-Dorg.slf4j...=warn" fixes this - Error was: Unknown lifecycle phase ".slf4j.simpleLogger..." 2. Add Maven dependency caching - Add 'cache: maven' to all setup-java@v4 steps - Speeds up builds by caching ~/.m2/repository - Reduces download times for dependencies This maintains the original goal of reducing CI log verbosity while: - Fixing Windows build compatibility - Improving build performance with caching
1 parent 9315d52 commit e284817

File tree

5 files changed

+10
-5
lines changed

5 files changed

+10
-5
lines changed

.github/workflows/main-jdk17-build.yml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,5 +37,6 @@ jobs:
3737
with:
3838
distribution: 'temurin'
3939
java-version: ${{ matrix.java }}
40+
cache: 'maven'
4041
- name: Build with Maven
41-
run: mvn clean test install javadoc:aggregate -Pci
42+
run: mvn clean test install javadoc:aggregate -Pci -B "-Dorg.slf4j.simpleLogger.log.org.apache.maven.cli.transfer.Slf4jMavenTransferListener=warn"

.github/workflows/main-jdk17-windows-build-multi-locale.yml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,5 +41,6 @@ jobs:
4141
with:
4242
distribution: 'temurin'
4343
java-version: ${{ matrix.java }}
44+
cache: 'maven'
4445
- name: Build with Maven
45-
run: mvn clean test install javadoc:aggregate -Pci
46+
run: mvn clean test install javadoc:aggregate -Pci -B "-Dorg.slf4j.simpleLogger.log.org.apache.maven.cli.transfer.Slf4jMavenTransferListener=warn"

.github/workflows/main-jdk17-windows-build.yml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -39,6 +39,7 @@ jobs:
3939
with:
4040
distribution: 'temurin'
4141
java-version: ${{ matrix.java }}
42+
cache: 'maven'
4243
- name: Build with Maven
4344
working-directory: 'tika build dir'
44-
run: mvn clean test install javadoc:aggregate -Pci
45+
run: mvn clean test install javadoc:aggregate -Pci -B "-Dorg.slf4j.simpleLogger.log.org.apache.maven.cli.transfer.Slf4jMavenTransferListener=warn"

.github/workflows/main-jdk21-build.yml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,5 +35,6 @@ jobs:
3535
with:
3636
distribution: 'temurin'
3737
java-version: ${{ matrix.java }}
38+
cache: 'maven'
3839
- name: Build with Maven
39-
run: mvn clean test install javadoc:aggregate -Pci
40+
run: mvn clean test install javadoc:aggregate -Pci -B "-Dorg.slf4j.simpleLogger.log.org.apache.maven.cli.transfer.Slf4jMavenTransferListener=warn"

.github/workflows/main-jdk25-build.yml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,5 +35,6 @@ jobs:
3535
with:
3636
distribution: 'temurin'
3737
java-version: ${{ matrix.java }}
38+
cache: 'maven'
3839
- name: Build with Maven
39-
run: mvn clean test install javadoc:aggregate -Pci
40+
run: mvn clean test install javadoc:aggregate -Pci -B "-Dorg.slf4j.simpleLogger.log.org.apache.maven.cli.transfer.Slf4jMavenTransferListener=warn"

0 commit comments

Comments
 (0)