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

Allow building on spir-v #67

Merged
merged 2 commits into from
Jun 11, 2021
Merged

Allow building on spir-v #67

merged 2 commits into from
Jun 11, 2021

Conversation

DJMcNab
Copy link
Contributor

@DJMcNab DJMcNab commented Jun 11, 2021

This allows this crate to be used on the gpu side for rust-gpu

This removes a roadblock from using crevice on the gpu side, and prevents confusing errors.

Complementary PR to EmbarkStudios/rust-gpu#668

DJMcNab added a commit to DJMcNab/rust-gpu that referenced this pull request Jun 11, 2021
This works simply by naming the binary crate
anything other than the name of the lib, which is example-runner-wgpu
As far as I know, the warning started in
EmbarkStudios#215

Since there is only one binary crate in the package, the command
(`cargo run -p example-runner-wgpu --release`)
maintains the same behaviour

The cargo issue is rust-lang/cargo#6313

This warning caused problems for me in testing
Lokathor/bytemuck#67
since I didn't notice the warning that my patch was not applied
@Lokathor Lokathor merged commit 043c18b into Lokathor:main Jun 11, 2021
@Lokathor
Copy link
Owner

I'll try to do a patch release later today.

@DJMcNab DJMcNab deleted the spirv branch June 11, 2021 19:45
@Lokathor
Copy link
Owner

1.6.1 is out

@DJMcNab
Copy link
Contributor Author

DJMcNab commented Jun 12, 2021

Thanks for the swift release.

khyperia pushed a commit to EmbarkStudios/rust-gpu that referenced this pull request Jun 14, 2021
This works simply by naming the binary crate
anything other than the name of the lib, which is example-runner-wgpu
As far as I know, the warning started in
#215

Since there is only one binary crate in the package, the command
(`cargo run -p example-runner-wgpu --release`)
maintains the same behaviour

The cargo issue is rust-lang/cargo#6313

This warning caused problems for me in testing
Lokathor/bytemuck#67
since I didn't notice the warning that my patch was not applied
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants