-
-
Notifications
You must be signed in to change notification settings - Fork 86
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
Update to rPGP 0.10 #4236
Update to rPGP 0.10 #4236
Conversation
This adds more duplicate dependencies, mainly because rPGP updated some crypto dependencies while iroh still depends on old versions. |
The test This commit is a squashed PR rpgp/rpgp#215. EDIT: I made a fix rpgp/rpgp#231 |
aa6b370
to
bd92140
Compare
c6a0504
to
d5ac5c0
Compare
{ name = "convert_case", version = "0.4.0" }, | ||
{ name = "darling", version = "<0.14" }, | ||
{ name = "convert_case", version = "0.4.0" }, | ||
{ name = "curve25519-dalek", version = "3.2.0" }, |
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.
what still pulls this in? this shouldn't be in the dep tree anymore
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.
cargo tree -i curve25519-dalek@3.2.0 output:
curve25519-dalek v3.2.0
├── ed25519-dalek v1.0.1
│ ├── iroh v0.4.0
│ │ └── deltachat v1.112.3 (/home/user/src/deltachat/deltachat-core-rust)
│ └── ssh-key v0.5.1
│ └── iroh v0.4.0 (*)
└── x25519-dalek v2.0.0-pre.1
└── pgp v0.10.1
└── deltachat v1.112.3 (/home/user/src/deltachat/deltachat-core-rust)
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.
oh lol, okay iroh will upgrade, but we still need some deps to be released first, so I guess we have to accept the duplicate deps for the moment.
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.
looks good, just some questions regarding deps
d5ac5c0
to
4f63325
Compare
4f63325
to
3e65b6f
Compare
This increases MSRV to 1.65.0, so blocked by deltachat/deltachat-ios#1771
And we need to merge deltachat/deltachat-android#2494 first, but this can be done right after releasing Delta Chat Android 1.36.