-
Notifications
You must be signed in to change notification settings - Fork 187
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
Add no_std
support
#664
Add no_std
support
#664
Conversation
7da239b
to
6212734
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This seems to be in early stages, I'll mark this as WIP until you deem it ready to review; here are a few basal points I started writing on while realizing that.
@i509VCB is this still something you want to pursue? |
Yes I plan to work on this, just have been doing other things the last month |
ac70e41
to
3e161b7
Compare
other than the issue with |
1aa975e
to
5b3c4e4
Compare
Note that Gecko is currently at rustc 1.63 let's please wait until Gecko updates to 1.64 before raising the msrv. Gecko tends to lag behind the latest stable a bit but usually not too much, so it shouldn't be long. |
@nical Ash won't blindly bump its MSRV, at least not in regular patch releases. |
Sorry, I got confused, I though I was on the wgpu repo. That said, bumping the msrv will prevent wgpu from updating ash until gecko catches up. |
Again, non-breaking releases with bugfixes and improvements won't include an MSRV bump (that makes them... breaking...). That said, I'd like to be requested new releases explicitly, otherwise they might sit for a while. |
The MSRV bump here only applies if the std feature is disabled. So ash effectively has an std MSRV and an alloc MSRV |
@nical fwiw #695 was published in https://github.com/ash-rs/ash/releases/tag/0.37.2. |
Thanks a lot! |
@nical it looks like @i509VCB is it time to revive this on 1.64, and drop all the workarounds for older MSRV? |
Yes, thanks a lot for being mindful of wgpu's constraints! |
Yeah I can reapply the changes into 1.64, just not going to be today. |
Also includes an explicit change to the Cargo.toml to raise the MSRV to 1.69, which ash already requires.
@MarijnS95 I have rebuilt the pull request. Ash appears to have raised it's MSRV to 1.69 already, so I went with that. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Any clue when unnecessary_qualifications
broke?
3bcfcfe
to
3fb62c0
Compare
no_std support is enabled by disabling the default
std
feature.