mattcasters commented on PR #8430: URL: https://github.com/apache/hop/pull/8430#issuecomment-5711510897
I tried this on Ubuntu 24.04 distro Maven, which is the setup that has been breaking: ``` Apache Maven 3.8.7 Maven home: /usr/share/maven Java version: 21.0.12, vendor: Ubuntu ``` (`libsisu-inject-java` 0.3.4) **What works:** `compilerId=eclipse` loads. `mvn -pl core clean compile` gets to `Compiling with eclipse [debug release 21]`. Publishing `ecj-plexus-hint` as a real Central artifact is the right replacement for the in-tree generated jar / `systemPath` hack. **What does not work:** the Lombok agent never attaches. hop-core then fails with a wall of `The method getX() is undefined` (missing Lombok getters). There is no `Lombok ECJ agent attached` line in the log. `./mvnw -pl core clean compile` (Apache Maven 3.9.12, Sisu 0.9) succeeds on the same tree. GitHub Actions is green for the same reason. `ecj-lombok-extension` 1.0.0 is compiled as **Java 21** (class major 65). Distro Sisu 0.3.x ASM stops at Java 14, so it never indexes `SetupEcjLombokAgent` (`@Named`). Same class of bug as `plexus-compiler-eclipse` 2.17.0, just on the extension this time. gitflow-incremental-builder still loads here because that one is Java 11 bytecode. Please ship the extension as `--release 11` (the `AbstractMavenLifecycleParticipant` / `ExecutionListener` APIs exist on Maven 3.8.7) and/or add a Plexus `components.xml` for the participant so Ubuntu does not have to ASM-scan a Java 21 class. Happy to re-run `/usr/bin/mvn -pl core clean compile` once that is out. The personal `com.github.rmannibucau.maven` groupId is fine as a build-only Central dependency (Apache-2.0). Longer term it would be healthier under something Hop/Apache owns, but that is not what is blocking Ubuntu. -- 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]
