Skip to content
This repository has been archived by the owner on Jan 5, 2024. It is now read-only.

Uses new reproducible wheels from our builds #21

Merged
merged 1 commit into from
Jan 26, 2021

Conversation

kushaldas
Copy link
Contributor

@kushaldas kushaldas commented Jan 6, 2021

The new wheels are coming in the following PR:

freedomofpress/securedrop-builder#213

These wheels are reproducible, you can build them following our
Makefile.

How to test?

They sha256sums should match with the above mentioned PR.

https://github.com/freedomofpress/securedrop-debian-packaging/pull/213/files#diff-7899317ed7333cf6bdab16b9bf5773f780ecb55b11114fc715e9240b16fd76ff

There is a temporary commit, which we can remove before merge.

Copy link
Contributor

@conorsch conorsch left a comment

Choose a reason for hiding this comment

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

Manually verified the new checksum matches what's shown in freedomofpress/securedrop-builder#213. Approving, but not merging, since there's still a temporary commit that should be removed.

@conorsch conorsch self-assigned this Jan 25, 2021
The new wheels are coming in the following PR:

freedomofpress/securedrop-builder#213

These wheels are reproducible, you can build them following our
Makefile.
@conorsch conorsch force-pushed the reproducible_wheels_usage branch from e540e0c to 0329232 Compare January 26, 2021 00:25
@conorsch
Copy link
Contributor

For reference, the historical CI build showing all tests passing can be found here: https://app.circleci.com/pipelines/github/freedomofpress/securedrop-log/60/workflows/a1a03582-be5f-46dd-a131-d07aca520156/jobs/77 I've removed the temporary feature branch pin, and will proceed with merge, verifying all tests are passing over in freedomofpress/securedrop-builder#213

@conorsch conorsch removed the Blocked label Jan 26, 2021
@conorsch conorsch merged commit b360ae2 into main Jan 26, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants