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

Fix bevy_render's image dependency version #14505

Merged

Conversation

SkiFire13
Copy link
Contributor

Objective

  • bevy_render depends on image 0.25 but uses image::ImageReader which was added only in image 0.25.2
  • users that have image 0.25 in their Cargo.lock and update to the latest bevy_render may thus get a compilation due to this (at least I did)

Solution

  • Properly set the correct minimum version of image that bevy_render depends on.

@alice-i-cecile alice-i-cecile added A-Rendering Drawing game state to the screen C-Dependencies A change to the crates that Bevy depends on S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it labels Jul 28, 2024
@alice-i-cecile alice-i-cecile added this pull request to the merge queue Jul 28, 2024
@alice-i-cecile
Copy link
Member

Good explanation, thank you :)

Merged via the queue into bevyengine:main with commit 59a33e6 Jul 28, 2024
34 checks passed
@SkiFire13 SkiFire13 deleted the fix-image-dependency-version branch July 28, 2024 16:14
mockersf pushed a commit that referenced this pull request Aug 2, 2024
# Objective

- `bevy_render` depends on `image 0.25` but uses `image::ImageReader`
which was added only in `image 0.25.2`
- users that have `image 0.25` in their `Cargo.lock` and update to the
latest `bevy_render` may thus get a compilation due to this (at least I
did)

## Solution

- Properly set the correct minimum version of `image` that `bevy_render`
depends on.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Rendering Drawing game state to the screen C-Dependencies A change to the crates that Bevy depends on S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants