Skip to content

enable bevy_sprite_render when enabling bevy_sprite for bevy_gizmos#21011

Closed
janis-bhm wants to merge 1 commit into
bevyengine:release-0.17.0from
janis-bhm:bevy-gizmos-features
Closed

enable bevy_sprite_render when enabling bevy_sprite for bevy_gizmos#21011
janis-bhm wants to merge 1 commit into
bevyengine:release-0.17.0from
janis-bhm:bevy-gizmos-features

Conversation

@janis-bhm
Copy link
Copy Markdown
Contributor

Objective

fixes #21008

Solution

bevy_gizmos enables the pipeline_2d mod and plugin when bevy_sprite and bevy_render features are enabled, but pipeline_2d also imports bevy_sprite_render. Since bevy_sprite_render is only used together with bevy_sprite and bevy_sprite isn't otherwise used, I think it makes sense to automatically enable bevy_sprite_render together with bevy_sprite

Alternative Solutions

bevy_sprite_render could be enabled by bevy_render, which already enables bevy_core_pipeline

@janis-bhm janis-bhm added C-Bug An unexpected or incorrect behavior A-Build-System Related to build systems or continuous integration X-Contentious There are nontrivial implications that should be thought through S-Needs-Review Needs reviewer attention (from anyone!) to move forward labels Sep 13, 2025
@janis-bhm janis-bhm closed this Sep 13, 2025
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 S-Needs-Review Needs reviewer attention (from anyone!) to move forward X-Contentious There are nontrivial implications that should be thought through

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant