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

Update Dockerfile to rust 1.84.0 #86

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

GlenDC
Copy link

@GlenDC GlenDC commented Jan 10, 2025

Checklist

  • I have read the Contributor Guide
  • I have read and agree to the Code of Conduct
  • I have added a description of my changes and why I'd like them included in the section below

Description of Changes

There is currently no way to override this rust version it seems as it doesn't seem to respect RUST_TOOLCHAIN env variable either...

Problem is that I'm using already package resolver v3 and that is not possible unless it's running 1.84 or higher.

Nightly would be an option as well of course

Related Issues

N/A

### Checklist

* [x] I have read the [Contributor Guide](../../CONTRIBUTING.md)
* [x] I have read and agree to the [Code of Conduct](../../CODE_OF_CONDUCT.md)
* [x] I have added a description of my changes and why I'd like them included in the section below

### Description of Changes

There is currently no way to override this rust version it seems as it doesn't seem to respect RUST_TOOLCHAIN env variable either... 

Problem is that I'm using already package resolver v3 and that is not possible unless it's running 1.84 or higher.

Nightly would be an option as well of course

### Related Issues

N/A
@GlenDC GlenDC requested a review from Jake-Shadle as a code owner January 10, 2025 17:50
@GlenDC
Copy link
Author

GlenDC commented Jan 10, 2025

There is currently no way to override this rust version it seems as it doesn't seem to respect RUST_TOOLCHAIN env variable either...

rust-toolchain.toml file does allow me to do this.. I wonder why RUST_TOOLCHAIN is not respected.

either way this PR is not urgent, take your time

@Jake-Shadle
Copy link
Member

@GlenDC
Copy link
Author

GlenDC commented Jan 11, 2025

https://github.com/EmbarkStudios/cargo-deny-action/blob/main/README.md#L22

I assume you mean rust-version? Thanks, totally missed it. Was looking for the usual suspects like toolchain, which is more commonly used in rust actions. Gonna try it out! Either way I guess it doesn't hurt to keep the dockerfile up to date, unless there are good reasons not to of course, I'm sure you know best.

Thanks a ton for taking the time! Have a nice day.

GlenDC added a commit to plabayo/rama that referenced this pull request Jan 11, 2025
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