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

Ensure that the dependency map is rendered in a deterministic order. #14

Merged
merged 1 commit into from Mar 20, 2020
Merged

Ensure that the dependency map is rendered in a deterministic order. #14

merged 1 commit into from Mar 20, 2020

Conversation

lamby
Copy link
Contributor

@lamby lamby commented Mar 20, 2020

Whilst working on the Reproducible Builds effort I noticed that browserify-lite generates unreproducible output. I tracked this down to JSON.stringify not ordering its result by default, resulting in output such as:

-    "./foo": 2,
-    "./bar": 1
+    "./bar": 1,
+    "./foo": 2

This commit sorts the results so that the build is the same each time.

Whilst working on the Reproducible Builds effort [0] I noticed that
browserify-lite generates unreproducible output. I tracked this down to
JSON.stringify not ordering its result default, resulting in output such as:

    -    "./foo": 2,
    -    "./bar": 1
    +    "./bar": 1,
    +    "./foo": 2

This commit sorts the results so that the build is the same each time.

  [0] https://reproducible-builds.org/
@andrewrk andrewrk merged commit f3ba2d6 into andrewrk:master Mar 20, 2020
@andrewrk
Copy link
Owner

andrewrk commented Mar 20, 2020

Please let me know if this has solved the problem in the reproducible builds effort downstream (or if there are more patches needed), and I will tag a release.

@lamby
Copy link
Contributor Author

lamby commented Mar 20, 2020

Hi @andrewrk Making this change in my local copy made, for example, webrtc-adapter reproducible for me in Debian so I would say please go ahead and cut a release. :)

@lamby lamby deleted the reproducible-source-path branch March 20, 2020 22:25
@andrewrk
Copy link
Owner

Done. Cheers

@lamby lamby changed the title Ensure that the dependency map is rendered in a determinstic order. Ensure that the dependency map is rendered in a deterministic order. Mar 31, 2020
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