-
Notifications
You must be signed in to change notification settings - Fork 897
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
Port rustup to Sequoia PGP. #2847
Conversation
Thank you for this - naturally there are many things we need to sort out before this could be merged. I shall edit the top posting to include a checklist. |
Ftr, we're porting to RustCrypto. Due to their own admission, RustCrypto is not ready for production use, but as you used rpgp before, and they also use RustCrypto, you are probably fine with Sequoia backed by RustCrypto. |
That is interesting indeed, it'd reduce the complexity of the CI work for sure. Do you have an ETA on an alpha version with that support? |
ETA on the alpha is now. There are some lxd-related errors that I don't feel responsible for. |
Sadly snap isn't always reliable, I've requeued them. The 32-bit Windows build test is worrying though:
|
Yeah, I don't know what that is about. I can report that we do successfully compile for 32-bit Windows: https://gitlab.com/sequoia-pgp/sequoia/-/jobs/1590997603 |
(Also, sha1collisiondetiction is a crate that we use for all backends, not just for the new RustCrypto one.) |
The only difference I can see is 1.55.0 vs. 1.54.0 -- can you try your build with 1.55.0 ? Maybe there's a compiler bug. |
Seems to build the sha1cd crate fine, but fails for different reasons (new warnings being errors in our ci): https://gitlab.com/sequoia-pgp/sequoia/-/jobs/1592512848 |
48a7b98
to
c6caaa0
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this looks good, but we do want the CI to run. If necessary I'll see if I can trigger things.
c6caaa0
to
b9d34a1
Compare
As of sequoia-openpgp 1.5, we're LGPL2.0+. |
b9d34a1
to
bb6f324
Compare
bb6f324
to
2c8364e
Compare
At minimum, rust-lang/rust#87813 is blocking this because of how sha1collisiondetection breaks the compile. |
2c8364e
to
87c902c
Compare
87c902c
to
da3798a
Compare
Now with the workaround developed by Josh Stone in https://bugzilla.redhat.com/show_bug.cgi?id=1990657. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That the full CI is green heartens me greatly. I need to spend a bit of time with the particular code bits, but I'm on the verge of approval
da3798a
to
2f3dabe
Compare
(Edited from original)
In order to port Rustup to Sequoia-PGP there are a number of steps remaining:
If further steps are identified, the above list can be amended.