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

Suggest sorting of extension sources to ensure a reproducible build #53

Merged
merged 1 commit into from Apr 26, 2020

Conversation

lamby
Copy link
Contributor

@lamby lamby commented Feb 8, 2020

This PR suggests the sorting of extension sources to ensure a reproducible build regardless of the layout on the file system. For example, in my PR to pikepdf:

-        glob('src/qpdf/*.cpp'),
+       sorted(glob('src/qpdf/*.cpp')),

… otherwise the sources were iterated and linked in a nondeterminstic order and thus the build was not bit-for-bit identical on subsequent build runs.

tasmo pushed a commit to prototypefund/reproducible-website that referenced this pull request Apr 24, 2020
@wjakob
Copy link
Member

wjakob commented Apr 26, 2020

LGTM, thanks!

@wjakob wjakob merged commit 59c2a6d into pybind:master Apr 26, 2020
@lamby lamby deleted the suggest-sorting-sources branch April 26, 2020 15:48
@wjakob
Copy link
Member

wjakob commented Apr 26, 2020

I amended the commit to add an explanation for why the sort() command is needed.

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.

None yet

2 participants