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
kinnison
left a comment
There was a problem hiding this comment.
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. |
kinnison
left a comment
There was a problem hiding this comment.
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.