Skip to content

Update Dockerfile #13

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

Merged
merged 1 commit into from
May 18, 2023
Merged

Update Dockerfile #13

merged 1 commit into from
May 18, 2023

Conversation

AnonymousAmalgrams
Copy link
Contributor

Suggested fix for following error, new to docker but my impression is that this should work
#0 2.063 error: package time-core v0.1.1 cannot be built because it requires rustc 1.65.0 or newer, while the currently active rustc version is 1.64.0

Taken from their official image page on docker hub,
https://hub.docker.com/_/rust
https://github.com/rust-lang/docker-rust/tree/35579d26bda862c00d127d63cee4ab9cd5d114c2/1.69.0

Fix 
#0 2.063 error: package `time-core v0.1.1` cannot be built because it requires rustc 1.65.0 or newer, while the currently active rustc version is 1.64.0
Copy link
Member

juntao commented May 18, 2023

Hello, I am a code review bot on flows.network. Here are my reviews of code commits in this PR.


Commit 6a2d89383564b665537767cefe93b34cc832d564

The patch updates the Dockerfile's base image from rust:1.64 to rust:1.69. The reason for this change is because package time-core v0.1.1 cannot be built using rustc 1.64.0, which is the version used by rust:1.64. This change will resolve this issue.

One potential problem could be that any code that relies on features of rustc 1.64.0 that have been removed or changed in rustc 1.69 may fail to build or work properly. However, without more information about the project, it is impossible to evaluate the impact of this change.

Copy link
Member

@juntao juntao left a comment

Choose a reason for hiding this comment

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

Thank you for catching this!

@juntao juntao merged commit 7d8ab10 into second-state:main May 18, 2023
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