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 input file list #3302

Merged
merged 1 commit into from
Jan 26, 2021
Merged

Sort input file list #3302

merged 1 commit into from
Jan 26, 2021

Conversation

bmwiedemann
Copy link
Contributor

Sort input file list
so that torchvision/_C.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 torchvision/_C.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.
Copy link
Contributor

@datumbox datumbox left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks!

@datumbox datumbox merged commit d732562 into pytorch:master Jan 26, 2021
@bmwiedemann bmwiedemann deleted the sort branch January 31, 2021 18:37
facebook-github-bot pushed a commit that referenced this pull request Feb 1, 2021
Reviewed By: datumbox

Differential Revision: D26156387

fbshipit-source-id: 443b0a34d6186b3a372d900c3ac78b3473abef64
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants