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

Support reproducible builds #24

Merged
merged 1 commit into from
Jun 10, 2021
Merged

Conversation

adangel
Copy link
Contributor

@adangel adangel commented Nov 8, 2020

META-INF/MANIFEST.MF contains values, that make the build harder
to reproduce than necessary.

Uses gitDescribe with tags, so that SCM-Git-Commit-ID-Description
contains the useful information, from which tag this build has
been built.

Removes SCM-Git-Branch, since this is the current HEAD branch.
When initially built, this is master, but when rebuilding,
master might have already moved along. The branch info
is actually not needed as long as SCM-Git-Commit-ID* properties
are provided.

Removes Implementation-Build-Java-Vendor and
Implementation-Build-Java-Version as these are too specific
(e.g. AdoptOpenJDK vs. OpenJDK vs. Oracle and 14.0.1 vs. 14.0.2).
Instead the property Implementation-Build-Java-Spec-Version is
used.

See also jvm-repo-rebuild/reproducible-central#45

META-INF/MANIFEST.MF contains values, that make the build harder
to reproduce than necessary.

Uses gitDescribe with tags, so that SCM-Git-Commit-ID-Description
contains the useful information, from which tag this build has
been built.

Removes SCM-Git-Branch, since this is the current HEAD branch.
When initially built, this is master, but when rebuilding,
master might have already moved along. The branch info
is actually not needed as long as SCM-Git-Commit-ID* properties
are provided.

Removes Implementation-Build-Java-Vendor and
Implementation-Build-Java-Version as these are too specific
(e.g. AdoptOpenJDK vs. OpenJDK vs. Oracle and 14.0.1 vs. 14.0.2).
Instead the property Implementation-Build-Java-Spec-Version is
used.
@jrivard
Copy link
Contributor

jrivard commented Jun 10, 2021

sorry for the long delay, thanks for the PR!

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.

None yet

2 participants