mattcasters opened a new issue, #8426:
URL: https://github.com/apache/hop/issues/8426

   ### Apache Hop version?
   
   2.20.0-SNAPSHOT (after #8407 and #8413)
   
   ### Java version?
   
   OpenJDK 21
   
   ### Operating system
   
   Linux
   
   ### What happened?
   
   #8406 / #8407 made `compilerId=eclipse` work on Debian/Ubuntu distro Maven 
by shipping a 1 KB Plexus `components.xml` jar. #8413 correctly removed that 
jar from git (ASF source releases cannot contain binaries) and left a Groovy 
`validate` script that writes it at build time via `systemPath`.
   
   That is not a complete replacement:
   
   - `systemPath` is deprecated, warns on every module when the file is missing 
at session start, and fights Maven 4 (#8425).
   - The Groovy execution is inherited, so it starts on every module. That tax 
is large enough that the ECJ switch from #8325 did **not** show a speedup on 
Ubuntu 24.04 (the 7 min → 1 min figure was measured on Apache/SDKMAN Maven, not 
`apt` Maven).
   - Distro Maven still cannot ASM-index `plexus-compiler-eclipse` 2.17.0. 
Ubuntu 24.04 is Maven 3.8.7 + Sisu 0.3.4; Ubuntu 26.04 is Maven 3.9.12 + Sisu 
0.3.5. Requiring Maven ≥ 3.9 with enforcer does not fix this — the 
incompatibility is Sisu's ASM, not the Maven version number.
   
   `apt install maven && mvn clean install` (and the same on WSL2) must keep 
working. `./mvnw` is not an acceptable only path.
   
   Follow-up: register `compilerId=eclipse` from the XML descriptor without a 
committed jar and without `systemPath`, and stop paying Groovy per module for 
the Lombok ECJ agent.
   
   Related: #8406, #8407, #8413, #8325, #8425.
   
   ### Issue Priority
   
   Priority: 2
   
   ### Issue Component
   
   Component: Infrastructure


-- 
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]

Reply via email to