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

Bump MSRV to 1.61.0 #3809

Merged
merged 1 commit into from
Dec 4, 2022
Merged

Bump MSRV to 1.61.0 #3809

merged 1 commit into from
Dec 4, 2022

Conversation

link2xt
Copy link
Collaborator

@link2xt link2xt commented Dec 4, 2022

This is required by image crate. env_logger and axum require 1.60.0.

Also update current Rust version to 1.65.0.

This is required by `image` crate.

Also update current Rust version to 1.65.0.
@@ -1 +1 @@
1.61.0
1.65.0
Copy link
Member

Choose a reason for hiding this comment

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

if we are testing the msrv why not delete this file and let us always develop against the latest version? The ci.yml could then also use stable instead of having to code this version in.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

This file is currently used by deltachat-android build tools and scripts, I don't want to break them right now: https://github.com/deltachat/deltachat-android/blob/master/scripts/install-toolchains.sh and https://github.com/deltachat/deltachat-android/blob/master/scripts/ndk-make.sh

It ensures that we always know which version of Rust is currently used for android builds at least.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I personally just use whatever rust comes from the distro repositories, without rustup.

Copy link
Member

Choose a reason for hiding this comment

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

huh, interesting. which distro do you use that it gives you a good enough rust setup?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Arch Linux has rust 1.65 in the repository, various cargo plugins and also a rustup package if you really need it.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

But also Debian sid has a recent enough version usually in sid, it's 1.62 currently, still good enough to compile deltachat-core-rust after this change.

@link2xt link2xt merged commit 4df5886 into master Dec 4, 2022
@link2xt link2xt deleted the link2xt/msrv-1.61 branch December 4, 2022 13:36
@link2xt
Copy link
Collaborator Author

link2xt commented Dec 4, 2022

@r10s you will need to run scripts/install_toolchains.sh when you update the core :)

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.

2 participants