You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tika.apache.org by "nddipiazza (via GitHub)" <gi...@apache.org> on 2025/12/31 15:50:33 UTC
[PR] TIKA-4609: Fix Maven verbosity flags (remove line breaks) [tika]
nddipiazza opened a new pull request, #2510:
URL: https://github.com/apache/tika/pull/2510
## Summary
Fixes https://issues.apache.org/jira/browse/TIKA-4609
The original commit 218508d48 accidentally introduced line breaks in the Maven property name, breaking Windows builds.
## Problem
The property name was split across lines with a space:
```
-Dorg.slf4j.simpleLogger.log.org.apache.maven.cli.transfe r.Slf4jMavenTransferListener=warn
```
This caused Maven to not recognize the property correctly.
## Solution
Keep 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 (.github/workflows/*.yml)
- Properly suppress Maven transfer listener output
- Maintains the original goal of reducing CI log verbosity
## Testing
This should make CI logs cleaner by:
- Using `-B` (batch mode) to suppress interactive progress
- Suppressing Maven download/upload progress messages
- Making error messages more visible
Without breaking Windows or any other builds.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@tika.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] TIKA-4609: Fix Maven verbosity flags (remove line breaks) [tika]
Posted by "nddipiazza (via GitHub)" <gi...@apache.org>.
nddipiazza commented on PR #2510:
URL: https://github.com/apache/tika/pull/2510#issuecomment-3702468976
## Transient Maven Central Issue
The current failure is **not related to our changes**. It's a network/repository connectivity issue:
```
Error: Extension eu.maveniverse.maven.nisse:extension:0.7.0 or one of its dependencies could not be resolved
Failed to read artifact descriptor for eu.maveniverse.maven.nisse:extension:jar:0.7.0
```
This is a transient Maven repository issue - the Nisse extension can't be downloaded from Maven Central. This happens occasionally in CI environments.
**Evidence this is unrelated to our changes:**
- Our changes only affect Maven output flags and caching
- The error occurs during extension resolution, before our flags are even used
- This is a known transient issue with Maven extensions
**Options:**
1. Wait and re-run the failed jobs (usually resolves itself)
2. Check if main branch has the same issue
3. Temporarily disable the Nisse extension if this persists
I'll monitor the in-progress runs to see if they succeed.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@tika.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] TIKA-4609: Fix Maven verbosity flags (remove line breaks) [tika]
Posted by "nddipiazza (via GitHub)" <gi...@apache.org>.
nddipiazza commented on PR #2510:
URL: https://github.com/apache/tika/pull/2510#issuecomment-3702441564
Updated the PR to fix the Windows parsing issue and add Maven caching:
## Changes in Latest Commit
### 1. Fixed Windows Command Parsing
**Problem:** Windows PowerShell/CMD was treating the space before `-D` as an argument separator, causing this error:
```
[ERROR] Unknown lifecycle phase ".slf4j.simpleLogger.log.org.apache.maven.cli.transfer.Slf4jMavenTransferListener=warn"
```
**Solution:** Quote the property definition:
```bash
# Before (broken on Windows)
-Dorg.slf4j.simpleLogger.log.org.apache.maven.cli.transfer.Slf4jMavenTransferListener=warn
# After (works on all platforms)
"-Dorg.slf4j.simpleLogger.log.org.apache.maven.cli.transfer.Slf4jMavenTransferListener=warn"
```
### 2. Added Maven Dependency Caching
Added `cache: 'maven'` to all `setup-java@v4` steps to:
- Cache `~/.m2/repository` between builds
- Significantly speed up builds by avoiding repeated downloads
- Reduce bandwidth usage
## What This Achieves
✅ Reduced CI log verbosity (Maven transfer messages suppressed)
✅ Works on Windows, Linux, and all platforms
✅ Faster builds with dependency caching
✅ Error messages are now easily visible in logs
Previous failure: https://github.com/apache/tika/actions/runs/20618695661
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@tika.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] TIKA-4609: Fix Maven verbosity flags (remove line breaks) [tika]
Posted by "nddipiazza (via GitHub)" <gi...@apache.org>.
nddipiazza commented on PR #2510:
URL: https://github.com/apache/tika/pull/2510#issuecomment-3702450627
## About the Maven Caching Change
Added `cache: 'maven'` to all `setup-java@v4` steps.
**What it does:**
- Caches `~/.m2/repository` between CI runs
- Uses `hashFiles('**/pom.xml')` to invalidate cache when dependencies change
- Built-in feature of setup-java@v4 action
**Benefits:**
- Saves ~30-60 seconds per build by avoiding repeated dependency downloads
- Reduces bandwidth usage
- No code changes required - just one line per workflow
**How it works:**
```yaml
- name: Set up JDK ${{ matrix.java }}
uses: actions/setup-java@v4
with:
distribution: 'temurin'
java-version: ${{ matrix.java }}
cache: 'maven' # ← This enables caching
```
This is equivalent to manually using `actions/cache@v4` but more concise. See [setup-java caching docs](https://github.com/actions/setup-java#caching-packages-dependencies).
The cache will be automatically invalidated if any pom.xml files change.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@tika.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] TIKA-4609: Fix Maven verbosity flags (remove line breaks) [tika]
Posted by "nddipiazza (via GitHub)" <gi...@apache.org>.
nddipiazza merged PR #2510:
URL: https://github.com/apache/tika/pull/2510
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@tika.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org