Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cherry pick changes to drop Java pom.xml files unused for release. #17728

Merged
merged 3 commits into from
Aug 5, 2024

Conversation

zhangskz
Copy link
Member

@zhangskz zhangskz commented Aug 5, 2024

This drops support for building Protobuf Java from source with Maven per https://protobuf.dev/news/2024-06-26/

Cherry-pick of commits 7c93804, 8714892 and f5ffaf9

This allows us to test our maven setup without relying on static `pom.xml` files and clears the way to remove them in a subsequent CL.

PiperOrigin-RevId: 658057911
This will cause the script to fail if any command fails, which will make it easier to debug failures.

PiperOrigin-RevId: 658878371
We mostly use generated pom files in our release currently, so we can delete all the files that aren't used and the tools to update them.

Note, java/bom/pom.xml java/pom.xml and java/protoc/pom.xml are all still used at release and java/kotlin/pom.xml is used for documentation so all of those need to stay for now.

PiperOrigin-RevId: 659664012
@zhangskz zhangskz requested review from a team as code owners August 5, 2024 20:57
@zhangskz zhangskz requested review from acozzette and googleberg and removed request for a team and acozzette August 5, 2024 20:57
@zhangskz zhangskz merged commit 83c0b48 into 28.x Aug 5, 2024
176 checks passed
@zhangskz zhangskz deleted the cp-pom branch August 5, 2024 22:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants