Skip to content

CI still runs on Rust 1.62 #5668

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

Closed
joseph-gio opened this issue Aug 13, 2022 · 5 comments
Closed

CI still runs on Rust 1.62 #5668

joseph-gio opened this issue Aug 13, 2022 · 5 comments
Labels
A-Build-System Related to build systems or continuous integration C-Bug An unexpected or incorrect behavior D-Trivial Nice and easy! A great choice to get started with Bevy

Comments

@joseph-gio
Copy link
Member

What you did

In #5659, I made a change that relies on a fix in Rust 1.63.

What went wrong

CI fails. It's probably still running on Rust 1.62.

@joseph-gio joseph-gio added C-Bug An unexpected or incorrect behavior S-Needs-Triage This issue needs to be labelled labels Aug 13, 2022
@TimJentzsch
Copy link
Contributor

Hmmm, I'm a bit confused by the CI action here:

build:
strategy:
matrix:
os: [windows-latest, ubuntu-latest, macos-latest]
runs-on: ${{ matrix.os }}
steps:
- uses: actions/checkout@v3
- uses: actions/cache@v3
with:
path: |
~/.cargo/bin/
~/.cargo/registry/index/
~/.cargo/registry/cache/
~/.cargo/git/db/
target/
key: ${{ runner.os }}-cargo-build-stable-${{ hashFiles('**/Cargo.toml') }}
- name: Install alsa and udev
run: sudo apt-get update; sudo apt-get install --no-install-recommends libasound2-dev libudev-dev
if: runner.os == 'linux'
- name: Build & run tests
# See tools/ci/src/main.rs for the commands this runs
run: cargo run -p ci -- test
env:
CARGO_INCREMENTAL: 0
RUSTFLAGS: "-C debuginfo=0 -D warnings"

Isn't this missing something like

- uses: actions-rs/toolchain@v1
  with:
    toolchain: stable
      components: rustfmt, clippy
      override: true

as the other CI jobs?
Maybe it's taking the old Rust version from the cache.

@mockersf
Copy link
Member

if not changed by the toolchain action, it's taking Rust version from the one installed by default: https://github.com/actions/runner-images/blob/main/images/linux/Ubuntu2004-Readme.md#rust-tools
GitHub will probably update it soon but they didn't push the change on Friday 🤷

@joseph-gio
Copy link
Member Author

it's taking Rust version from the one installed by default: https://github.com/actions/runner-images/blob/main/images/linux/Ubuntu2004-Readme.md#rust-tools

Thanks for the pointer. Seems like the next release is scheduled for August 15.

@TimJentzsch
Copy link
Contributor

Should we wait or use the toolchain action for the build job as well?

@mockersf
Copy link
Member

Should we wait or use the toolchain action for the build job as well?

Yes, it seems we already uses it in most other jobs, it would make sense to use it here also 👍

@mockersf mockersf added D-Trivial Nice and easy! A great choice to get started with Bevy A-Build-System Related to build systems or continuous integration and removed S-Needs-Triage This issue needs to be labelled labels Aug 13, 2022
@bors bors bot closed this as completed in d1e5c50 Aug 13, 2022
maccesch pushed a commit to Synphonyte/bevy that referenced this issue Sep 28, 2022
# Objective

Fixes bevyengine#5668.

The Rust version used in the CI `build` step previously depended on the default Rust version defined by GitHub in the Ubuntu image: <https://github.com/actions/runner-images/blob/main/images/linux/Ubuntu2004-Readme.md#rust-tools>

This currently doesn't allow us to use Rust 1.63 features until this version is updated.

## Solution

We now use the `actions-rs/toolchain@v1` action to always use the latest stable Rust version.
This is already used for other CI jobs that we have.
james7132 pushed a commit to james7132/bevy that referenced this issue Oct 28, 2022
# Objective

Fixes bevyengine#5668.

The Rust version used in the CI `build` step previously depended on the default Rust version defined by GitHub in the Ubuntu image: <https://github.com/actions/runner-images/blob/main/images/linux/Ubuntu2004-Readme.md#rust-tools>

This currently doesn't allow us to use Rust 1.63 features until this version is updated.

## Solution

We now use the `actions-rs/toolchain@v1` action to always use the latest stable Rust version.
This is already used for other CI jobs that we have.
ItsDoot pushed a commit to ItsDoot/bevy that referenced this issue Feb 1, 2023
# Objective

Fixes bevyengine#5668.

The Rust version used in the CI `build` step previously depended on the default Rust version defined by GitHub in the Ubuntu image: <https://github.com/actions/runner-images/blob/main/images/linux/Ubuntu2004-Readme.md#rust-tools>

This currently doesn't allow us to use Rust 1.63 features until this version is updated.

## Solution

We now use the `actions-rs/toolchain@v1` action to always use the latest stable Rust version.
This is already used for other CI jobs that we have.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Build-System Related to build systems or continuous integration C-Bug An unexpected or incorrect behavior D-Trivial Nice and easy! A great choice to get started with Bevy
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants