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

build from release commit #358

Merged
merged 1 commit into from
Oct 31, 2024
Merged

build from release commit #358

merged 1 commit into from
Oct 31, 2024

Conversation

hboutemy
Copy link
Contributor

@hboutemy hboutemy commented Oct 31, 2024

currently, Git commit reference in release binaries output is one commit behind the Git tag, given the order of operation in the workflow

Git commit has to be done before building: perhaps it won't be pushed if build fails, but at least build references the right commit hash that will get tagged

issue found while trying to rebuild for Reproducible Builds: https://github.com/jvm-repo-rebuild/reproducible-central/blob/master/content/org/infinispan/protostream/README.md
and more precisely analysis differences https://github.com/jvm-repo-rebuild/reproducible-central/blob/master/content/org/infinispan/protostream/protostream-aggregator-5.0.7.Final.diffoscope

@hboutemy hboutemy requested a review from a team as a code owner October 31, 2024 06:44
@ryanemerson ryanemerson merged commit 1805508 into infinispan:main Oct 31, 2024
2 of 3 checks passed
@ryanemerson
Copy link
Contributor

Thanks @hboutemy

@hboutemy hboutemy deleted the patch-1 branch October 31, 2024 15:11
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