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

Keep async-channel version same #7648

Closed
wants to merge 1 commit into from

Conversation

lewiszlw
Copy link
Member

Objective

When I update bevy to latest bevy main branch for my game, async-channel version conflicts.

error: failed to select a version for `async-channel`.
    ... required by package `bevy_render v0.9.0 (https://github.com/bevyengine/bevy?rev=f7fbfaf9c72035e98c6b6cec0c7d26ff9f5b1c82#f7fbfaf9)`
    ... which satisfies git dependency `bevy_render` of package `bevy_core_pipeline v0.9.0 (https://github.com/bevyengine/bevy?rev=f7fbfaf9c72035e98c6b6cec0c7d26ff9f5b1c82#f7fbfaf9)`
    ... which satisfies git dependency `bevy_core_pipeline` of package `bevy_gltf v0.9.0 (https://github.com/bevyengine/bevy?rev=f7fbfaf9c72035e98c6b6cec0c7d26ff9f5b1c82#f7fbfaf9)`
    ... which satisfies git dependency `bevy_gltf` of package `bevy_internal v0.9.0 (https://github.com/bevyengine/bevy?rev=f7fbfaf9c72035e98c6b6cec0c7d26ff9f5b1c82#f7fbfaf9)`
    ... which satisfies git dependency `bevy_internal` of package `bevy v0.9.0 (https://github.com/bevyengine/bevy?rev=f7fbfaf9c72035e98c6b6cec0c7d26ff9f5b1c82#f7fbfaf9)`
    ... which satisfies git dependency `bevy` of package `tetris v0.1.0 (/Users/lewis/Github/tetris)`
versions that meet the requirements `^1.8` are: 1.8.0

all possible versions conflict with previously selected packages.

  previously selected package `async-channel v1.7.1`
    ... which satisfies dependency `async-channel = "^1.4"` (locked to 1.7.1) of package `bevy_ecs v0.9.0 (https://github.com/bevyengine/bevy?rev=f7fbfaf9c72035e98c6b6cec0c7d26ff9f5b1c82#f7fbfaf9)`
    ... which satisfies git dependency `bevy_ecs` of package `bevy_animation v0.9.0 (https://github.com/bevyengine/bevy?rev=f7fbfaf9c72035e98c6b6cec0c7d26ff9f5b1c82#f7fbfaf9)`
    ... which satisfies git dependency `bevy_animation` of package `bevy_gltf v0.9.0 (https://github.com/bevyengine/bevy?rev=f7fbfaf9c72035e98c6b6cec0c7d26ff9f5b1c82#f7fbfaf9)`
    ... which satisfies git dependency `bevy_gltf` of package `bevy_internal v0.9.0 (https://github.com/bevyengine/bevy?rev=f7fbfaf9c72035e98c6b6cec0c7d26ff9f5b1c82#f7fbfaf9)`
    ... which satisfies git dependency `bevy_internal` of package `bevy v0.9.0 (https://github.com/bevyengine/bevy?rev=f7fbfaf9c72035e98c6b6cec0c7d26ff9f5b1c82#f7fbfaf9)`
    ... which satisfies git dependency `bevy` of package `tetris v0.1.0 (/Users/lewis/Github/tetris)`

failed to select a version for `async-channel` which could resolve this conflict

Solution

  • Keep async-channel version same

@alice-i-cecile alice-i-cecile added C-Bug An unexpected or incorrect behavior C-Dependencies A change to the crates that Bevy depends on labels Feb 13, 2023
@alice-i-cecile alice-i-cecile added this to the 0.10 milestone Feb 13, 2023
@lewiszlw
Copy link
Member Author

CI dependencies check-bans failed due to error[duplicate]: found 2 duplicate entries for crate 'windows' (which depended by gilrs and wgpu-hal), How can I resolve?

Copy link
Member

@alice-i-cecile alice-i-cecile left a comment

Choose a reason for hiding this comment

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

Weird that it was causing you such issues. Can you reproduce after a cargo clean?

@alice-i-cecile alice-i-cecile removed this from the 0.10 milestone Feb 13, 2023
@mockersf
Copy link
Member

mockersf commented Feb 13, 2023

That change probably wouldn't fix your issue. You had an existing lock file with the 1.7 version, and updating to Bevy main the minimum version needed was 1.8 (from bevy_render) which wasn't compatible with your lock file.

In theory this PR is a breaking change as it would mean people using just bevy_ecs will have to update async-channel, while it would still work with the 1.4

@mockersf mockersf removed the C-Bug An unexpected or incorrect behavior label Feb 13, 2023
@alice-i-cecile
Copy link
Member

In theory this PR is a breaking change as it would mean people using just bevy_ecs will have to update async-channel, while it would still work with the 1.4

Is it a breaking change to update a minor version of a dependency? That doesn't seem quite right to me.

@mockersf
Copy link
Member

If someone specified a fixed version of the dependency, then updating bevy_ecs requiring the newest dependency would break compilation 🤷

@lewiszlw
Copy link
Member Author

Weird that it was causing you such issues. Can you reproduce after a cargo clean?

Yeah, I need cargo clean. Do we still need to bump async-channel crate version, if not, I'll close this pr. @alice-i-cecile

@alice-i-cecile
Copy link
Member

No strong feelings here. I'll close it out for now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Dependencies A change to the crates that Bevy depends on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants