Skip to content

Extend Gradle integration test to check checksums and metadata marker #23

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

Merged
merged 4 commits into from
Mar 1, 2025

Conversation

jjohannes
Copy link
Contributor

Validate that the checksums for the Jar in the .module file corresponds to the actual checksum of the Jar.

This is not correct right now for some modules, because the Jar is modified by other plugins (e.g. moditec) after the Gradle metadata plugin computed size and checksums.

Checks what is reported here: FasterXML/jackson-databind#4985

Signed-off-by: Jendrik Johannes <jendrik.johannes@gmail.com>
@jjohannes jjohannes force-pushed the validate-checksums-in-gmm branch from 55bcf5c to ffcf2c5 Compare February 26, 2025 15:12
@jjohannes jjohannes changed the title Extend Gradle integration test to check checksums Extend Gradle integration test to check checksums and metadata marker Feb 26, 2025
@cowtowncoder
Copy link
Member

I guess we'll keep this as WIP until modules are fixed, to avoid breaking the build?

@jjohannes
Copy link
Contributor Author

jjohannes commented Feb 28, 2025

I switched this PR to test against 2.19.0-SNAPSHOT. It should turn green once all PRs here have been merged and new Snapshots are published.

Once 2.19.0 is releases, the test should run against the release again.

@cowtowncoder cowtowncoder merged commit 117c889 into FasterXML:2.19 Mar 1, 2025
3 checks passed
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