Add bevy_pbr as a dependency to bevy_gizmos #11035
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ItsDoot
added
C-Dependencies
matiqo15
approved these changes
Dec 20, 2023
|
Like @irate-devil said in the comments of the original issue:
So I'm not sure this is the right solution. |
|
bevy_sprite doesn't require bevy_gizmos does it? |
|
it should be possible to draw gizmos without having to bring |
github-merge-queue bot
pushed a commit
that referenced
this pull request
Jan 14, 2024
…11296) # Objective - `bevy_gizmos` cannot work if both `bevy_sprite` and `bevy_pbr` are disabled. - It silently fails to render, making it difficult to debug. - Fixes #10984 ## Solution - Log an error message when `GizmoPlugin` is registered. ## Alternatives I chose to log an error message, since it seemed the least intrusive of potential solutions. Some alternatives include: - Choosing one dependency as the default, neglecting the other. (#11035) - Raising a compile error when neither dependency is enabled. ([See my original comment](#10984 (comment))) - Raising a compile warning using a macro hack. ([Pre-RFC - Add compile_warning! macro](https://internals.rust-lang.org/t/pre-rfc-add-compile-warning-macro/9370/7?u=bd103)) - Logging a warning instead of an error. - _This might be the better option. Let me know if I should change it._ --- ## Changelog - `bevy_gizmos` will now log an error if neither `bevy_pbr` nor `bevy_sprite` are enabled.
|
fixed in #11296 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Objective
bevy_gizmosfeature withdefault-features=falserenders nothing (requires extrabevy_pbrfeature) #10984Solution