Skip to content

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
@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