slachiewicz opened a new pull request, #3789: URL: https://github.com/apache/thrift/pull/3789
Client: java ### Summary This PR removes the unmaintained `thrift-maven-plugin` from `contrib/` and documents standard Maven usage using `exec-maven-plugin` and `build-helper-maven-plugin` in `lib/java/README.md`. ### Context & Rationale - The `thrift-maven-plugin` under `contrib/` has not been published to Maven Central since ~2017 (Thrift 0.10.0 era). - It is not part of the standard build or test process. - The plugin itself does not embed or manage the Thrift compiler; it merely shells out to an externally installed `thrift` executable on the host system. - As noted in discussion on #3347, its functionality is natively and cleanly replaced by standard, widely supported Maven plugins (`exec-maven-plugin` + `build-helper-maven-plugin`). ### Changes 1. **Deleted `contrib/thrift-maven-plugin`**: Removed obsolete plugin source and POM (~1,400 lines). 2. **Updated `build/veralign.sh`**: Removed the `contrib/thrift-maven-plugin/pom.xml` entry from the version alignment script. 3. **Updated `lib/java/README.md`**: Added a dedicated `Using Thrift in Maven Projects` section with a complete, idiomatic Maven configuration snippet. -- 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]
