Skip to content

Allow building on spir-v#67

Merged
Lokathor merged 2 commits intoLokathor:mainfrom
DJMcNab:spirv
Jun 11, 2021
Merged

Allow building on spir-v#67
Lokathor merged 2 commits intoLokathor:mainfrom
DJMcNab:spirv

Conversation

@DJMcNab
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Owner

1.6.1 is out

@DJMcNab
Copy link
Copy Markdown
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