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

Make the build reproducible #319

Conversation

lamby
Copy link
Contributor

@lamby lamby commented Oct 1, 2020

Whilst working on the Reproducible Builds effort we noticed that libsass-python could not be built reproducibly. This is due to the manpage containing content that is generated by iterating over a Python frozenset type, which is non-deterministic.

This was originally filed in Debian as #971527.

@asottile
Copy link
Member

feel free to send a new PR which passes the tests -- otherwise looks fine

@asottile asottile closed this Mar 31, 2021
@lamby
Copy link
Contributor Author

lamby commented Mar 31, 2021

feel free to send a new PR which passes the tests -- otherwise looks fine

Thanks for the review.Three things from that:

  • Can you confirm precisely which tests? (I don't see any failing here)
  • If it "looks fine", can you not merge regardless?
  • Can we not keep this PR open for now?

@asottile
Copy link
Member

ah I assumed since it has sat since October with a failing status that you weren't working on it any more:

All checks have failed
1 failing check

@asottile asottile reopened this Mar 31, 2021
@asottile
Copy link
Member

I will not merge with failing stauses, no

@lamby
Copy link
Contributor Author

lamby commented Mar 31, 2021

Ah, didn't see "failing check" until now - this changes everything and will address shortly, likely tomorrow. :)

@lamby lamby force-pushed the 971527-libsass-python-please-make-the-build-reproducible branch from 522b8e9 to 74889e5 Compare April 1, 2021 06:13
@lamby
Copy link
Contributor Author

lamby commented Apr 1, 2021

Updated 👍 - thanks @asottile

Whilst working on the Reproducible Builds effort [0] we noticed that
libsass-python could not be built reproducibly.

This is due to the manpage containing content that is generated by
iterating over a Python frozenset type, which is non-deterministic.

This was originally filed in Debian as #971527 [1].

 [0] https://reproducible-builds.org/
 [1] https://bugs.debian.org/971527
@lamby lamby force-pushed the 971527-libsass-python-please-make-the-build-reproducible branch from 7ac5903 to 905d3dd Compare April 1, 2021 06:16
Copy link
Member

@asottile asottile left a comment

Choose a reason for hiding this comment

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

@asottile asottile merged commit ebd4233 into sass:master Apr 1, 2021
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

3 participants