Skip to content

Conversation

@zicklag
Copy link
Member

@zicklag zicklag commented Aug 9, 2021

Objective

  • Allow you to compile Bevy with the bevy_sprite2 feature, but without the bevy_pbr2 feature.
    • This currently fails because the bevy_sprite2 crate does not require the derive feature of the bytemuck crate in its Cargo.toml, even though it is required to compile.

Solution

  • Add the derive feature of bytemuck to the bevy_sprite2 crate

When building Bevy with the bevy_sprite2 feature, but without the
bevy_pbr2 feature, the bytemuck crate was missing the "derive" feature.
@zicklag zicklag changed the base branch from main to pipelined-rendering August 9, 2021 16:05
@github-actions github-actions bot added the S-Needs-Triage This issue needs to be labelled label Aug 9, 2021
@cart
Copy link
Member

cart commented Aug 10, 2021

bors r+

bors bot pushed a commit that referenced this pull request Aug 10, 2021
# Objective

- Allow you to compile Bevy with the `bevy_sprite2` feature, but without the `bevy_pbr2` feature.
  - This currently fails because the `bevy_sprite2` crate does not require the `derive` feature of the `bytemuck` crate in its `Cargo.toml`, even though it is required to compile.

## Solution

- Add the `derive` feature of `bytemuck` to the `bevy_sprite2` crate
@bors
Copy link
Contributor

bors bot commented Aug 10, 2021

@bors bors bot changed the title Add missing bytemuck feature [Merged by Bors] - Add missing bytemuck feature Aug 10, 2021
@bors bors bot closed this Aug 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-Needs-Triage This issue needs to be labelled

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants