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

Sort beipack input file list #21460

Closed
wants to merge 1 commit into from

Conversation

bmwiedemann
Copy link
Contributor

Sort input file list, so that cockpit-bridge.beipack.xz builds in a reproducible way in spite of non-deterministic filesystem readdir order.

See https://reproducible-builds.org/ for why this is good.

This patch was done while working on reproducible builds for openSUSE.

so that cockpit-bridge.beipack.xz builds in a reproducible way
in spite of non-deterministic filesystem readdir order.

See https://reproducible-builds.org/ for why this is good.
This patch was done while working on reproducible builds for openSUSE.
@martinpitt
Copy link
Member

Did you actually observe some non-reproducibility with that? Do you have a pointer?

We are well aware of this, Debian also tests this (and it failed a long time ago), and then we fixed it with commit 43cfb22. In other words, the build system already sorts the result of find_sources(), so this should be unnecessary.

@martinpitt martinpitt added the question Further information is requested label Dec 20, 2024
@martinpitt martinpitt marked this pull request as draft December 20, 2024 11:20
@bmwiedemann
Copy link
Contributor Author

I was testing with https://build.opensuse.org/package/show/openSUSE:Factory/cockpit that has cockpit-322 without commit 43cfb22. Adding it there, indeed fixes the issue.

=> closing

@bmwiedemann bmwiedemann deleted the readdir branch December 20, 2024 12:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants