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

Different reproducible changes #298

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

fridrich
Copy link
Contributor

This might be a bit controversial, but if the SOURCE_DATE_EPOCH is not set, everything is like before. Now, with this patch, the uuids are still unique in the package and also around the whole system, but they are the same if the zero-change rebuild is done. We run this since early August and everything works just fine. All xmvn tests and -Prun-its pass.

I rerun the whole build with this particular version of patch, just to be sure and the stuff is reproducible but still uuids are unique if SOURCE_DATE_EPOCH is set.

@codecov
Copy link

codecov bot commented Sep 21, 2023

Codecov Report

Merging #298 (6bc62e0) into master (9f5fce1) will not change coverage.
Report is 10 commits behind head on master.
The diff coverage is 0.00%.

@@          Coverage Diff           @@
##           master    #298   +/-   ##
======================================
  Coverage    0.00%   0.00%           
======================================
  Files          71      71           
  Lines        2091    2101   +10     
  Branches      305     308    +3     
======================================
- Misses       2091    2101   +10     
Files Changed Coverage Δ
.../java/org/fedoraproject/xmvn/mojo/JavadocMojo.java 0.00% <0.00%> (ø)
.../fedoraproject/xmvn/tools/install/JavaPackage.java 0.00% <0.00%> (ø)
...n/tools/install/impl/DefaultArtifactInstaller.java 0.00% <0.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@fridrich
Copy link
Contributor Author

Ping?

@fridrich fridrich changed the title Make metadata UUIDs reproducible if SOURCE_DATE_EPOCH is set Different reproducible changes Jul 18, 2024
@fridrich
Copy link
Contributor Author

Pingo ;)

@fridrich
Copy link
Contributor Author

Would it be possible to review and take at least some of the patches. The reproducible manifest or the -notimestamp for javadoc should be reasonably non-controversial. The metadata stuff might look scary, but we run with it for almost 2 years already here in SUSE and it is just fine.

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.

1 participant