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

run windows ci on rust 1.78 #13834

Merged
merged 1 commit into from
Jun 14, 2024
Merged

run windows ci on rust 1.78 #13834

merged 1 commit into from
Jun 14, 2024

Conversation

mockersf
Copy link
Member

Objective

  • temporary fix for CI
  • Rust 1.79 seems to have broken bevy on DX12 on some configuration

Solution

  • Keep using Rust 1.78

@mockersf mockersf added the A-Build-System Related to build systems or continuous integration label Jun 13, 2024
@mockersf
Copy link
Member Author

rust-lang/rust#126442

@alice-i-cecile
Copy link
Member

Reported upstream at rust-lang/rust#126442

@alice-i-cecile alice-i-cecile added the S-Waiting-on-Author The author needs to make changes or address concerns before this can be merged label Jun 14, 2024
@alice-i-cecile
Copy link
Member

A fix and backport seems imminent: I'll defer to @mockersf about whether we should merge or close this.

@mockersf
Copy link
Member Author

let's merge this now to be able to continue merging PRs on Bevy while waiting for the patch

@mockersf mockersf added this pull request to the merge queue Jun 14, 2024
Merged via the queue into bevyengine:main with commit 14fdec6 Jun 14, 2024
30 checks passed
@mnmaita
Copy link
Member

mnmaita commented Jun 15, 2024

Which would be the WGPU fix that lets us revert this change @mockersf, do you know this? Asking so I can track it and get it done.

@alice-i-cecile
Copy link
Member

Once there's a wgpu release with gfx-rs/wgpu#5812 in it we can revert this.

@mockersf
Copy link
Member Author

#13863 to merge after that release

github-merge-queue bot pushed a commit that referenced this pull request Jun 27, 2024
zmbush pushed a commit to zmbush/bevy that referenced this pull request Jul 3, 2024
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 S-Waiting-on-Author The author needs to make changes or address concerns before this can be merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants