Skip to content

Conversation

bmwiedemann
Copy link
Contributor

@bmwiedemann bmwiedemann commented Sep 10, 2019

Sort input file list
so that _libiminuit.so builds in a reproducible way
in spite of indeterministic filesystem readdir order
and http://bugs.python.org/issue30461

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

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

so that _libiminuit.so builds in a reproducible way
in spite of indeterministic filesystem readdir order
and http://bugs.python.org/issue30461

See https://reproducible-builds.org/ for why this is good.
@codecov
Copy link

codecov bot commented Sep 10, 2019

Codecov Report

Merging #355 into develop will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff            @@
##           develop     #355   +/-   ##
========================================
  Coverage    65.71%   65.71%           
========================================
  Files          137      137           
  Lines         4431     4431           
  Branches       651      651           
========================================
  Hits          2912     2912           
  Misses        1281     1281           
  Partials       238      238

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 18ea7b2...f6f8785. Read the comment docs.

@HDembinski HDembinski merged commit d0ed6a3 into scikit-hep:develop Sep 11, 2019
@HDembinski
Copy link
Member

If I understand correctly, this change may help to make all binaries from the same build environment identical, which is certainly a good thing. Thank you for the PR!

@bmwiedemann
Copy link
Contributor Author

Yes, that is the goal.
Thanks for merging.

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