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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
