Skip to content

#205 Fix non-determinism in build-data.properties file #206

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 1 commit into from
Oct 28, 2024

Conversation

jpgxs
Copy link
Contributor

@jpgxs jpgxs commented Oct 28, 2024

Invoke singlejar with a relative path to the output JAR, preventing the build-data.properties file from containing the absolute path.

Fixes: #205

Copy link

Thanks for the contribution! Before we can merge this, we need @jgriffiths-od to sign the Salesforce Inc. Contributor License Agreement.

@jpgxs jpgxs force-pushed the 205_fix_build_data_determinism branch from 9aad631 to abeb611 Compare October 28, 2024 11:36
Copy link

Thanks for the contribution! Before we can merge this, we need @jpgxs to sign the Salesforce Inc. Contributor License Agreement.

Invoke singlejar with a relative path to the output JAR, preventing the
build-data.properties file from containing the absolute path.
@jpgxs jpgxs force-pushed the 205_fix_build_data_determinism branch from abeb611 to 971b7c9 Compare October 28, 2024 11:41
Copy link
Contributor

@plaird plaird left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the fix!

@plaird
Copy link
Contributor

plaird commented Oct 28, 2024

I will test within Salesforce (we have a large number of springboot use cases), and cut an official release soon.

@plaird plaird merged commit bc879f7 into salesforce:main Oct 28, 2024
1 check passed
@jpgxs jpgxs deleted the 205_fix_build_data_determinism branch October 28, 2024 15:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Output JAR is not hermetic
2 participants