Skip to content

Reproducible builds #159

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 2 commits into from
Oct 2, 2024
Merged

Reproducible builds #159

merged 2 commits into from
Oct 2, 2024

Conversation

shroffk
Copy link
Collaborator

@shroffk shroffk commented Oct 1, 2024

Based on similar work as:

ControlSystemStudio/phoebus#3153

Copy link

sonarqubecloud bot commented Oct 1, 2024

@shroffk shroffk requested a review from jacomago October 1, 2024 17:25
@shroffk
Copy link
Collaborator Author

shroffk commented Oct 1, 2024

@minijackson hope this addresses your use case

Copy link

github-actions bot commented Oct 1, 2024

Overall Project 0.67%

There is no coverage information present for the Files changed

@shroffk shroffk merged commit 1e3c29f into master Oct 2, 2024
6 checks passed
@shroffk shroffk deleted the reproducible_builds branch October 2, 2024 13:59
@minijackson
Copy link

minijackson commented Oct 3, 2024

@shroffk It almost works out of the box for me. The issue for me is that since I download GitHub tarballs, the .git folder is not present, and the git-commit-id Maven plugin fails with:

[ERROR] Failed to execute goal io.github.git-commit-id:git-commit-id-maven-plugin:9.0.1:revision (get-the-git-infos) on project ChannelFinder:
    .git directory is not found! Please specify a valid [dotGitDirectory] in your project -> [Help 1]

It can easily be fixed on my end, but other users using the GitHub tarball might encounter this error.

@jacomago
Copy link
Contributor

jacomago commented Oct 3, 2024

@minijackson good point. We should add a default incase the build folder is not a git repository.

@shroffk
Copy link
Collaborator Author

shroffk commented Oct 3, 2024

@jacomago @minijackson
https://github.com/git-commit-id/git-commit-id-maven-plugin/blob/8d15df585877c1f13dab2369dd20be91cc45e442/src/main/java/pl/project13/maven/git/GitCommitIdMojo.java#L568

maybe we should set this to false failOnNoGitDirectory

not sure what the default should be for the timestamp ( maybe we should just skip it and just reply on the git.properties included in the jar )

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.

3 participants