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

Move rust-toolchain file from the core repository #1781

Merged
merged 2 commits into from
Jan 3, 2023

Conversation

link2xt
Copy link
Contributor

@link2xt link2xt commented Jan 3, 2023

This way updating the core does not automatically update toolchain version and any toolchain can be used as long as it can build the core.

Similar change for Android: deltachat/deltachat-android#2440

This way updating the core does not automatically update
toolchain version and any toolchain can be used as long
as it can build the core.
@link2xt
Copy link
Contributor Author

link2xt commented Jan 3, 2023

Once this is merged, we can remove rust-toolchain from the core repository and stop forcing all developers to use the same toolchain for core building. Core developers should be able to use nightly or beta if they want, Android can be upgraded to the latest stable release and iOS can stay with 1.64.0 if there are currently problems with building on a new version (see #1775, #1771).

`rust-toolchain` is accessed from `DcCore` subdirectory,
therefore, just `../rust-toolchain` does the job.
@r10s r10s force-pushed the link2xt/pin-rust-toolchain branch from 5b9eb0a to fd51090 Compare January 3, 2023 12:39
Copy link
Member

@r10s r10s left a comment

Choose a reason for hiding this comment

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

thanks for taking care!

i adapted the path, force-pushed, and tested if it builds (it does :)

@r10s r10s merged commit 01c1d29 into master Jan 3, 2023
@r10s r10s deleted the link2xt/pin-rust-toolchain branch January 3, 2023 12:41
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