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

Update shaderc to 0.6 in order to significantly reduce compile-times … #1226

Merged
merged 4 commits into from
Aug 12, 2019

Conversation

Abendstolz
Copy link
Contributor

@Abendstolz Abendstolz commented Jul 30, 2019

…on (Arch/Void)Linux again by linking towards shared libraries

  • Added an entry to CHANGELOG_VULKANO.md or CHANGELOG_VK_SYS.md if knowledge of this change could be valuable to users

@mitchmindtree fyi

…on (Arch/Void)Linux again by linking towards shared libraries
@mitchmindtree
Copy link
Contributor

@Abendstolz nice! I think using pre-installed libraries was already active in 0.5, but at least this gets it working again on Arch and Void 👍

Could you also add a shaderc-build-from-source feature to vulkano-shaders which enables the shaderc/build-from-source feature? I just realised this feature is currently inaccessible via vulkano-shaders and might save us temporarily if shaderc's linking to pre-built libraries breaks again in the future.

…build from source if linking to system libraries fails
@Abendstolz
Copy link
Contributor Author

@mitchmindtree sure thing. Done in 5db99c8

@mitchmindtree
Copy link
Contributor

mitchmindtree commented Aug 3, 2019

Thanks a lot @Abendstolz - it looks like travis is just complaining about some trailing whitespace now, but after that's fixed I'd be happy to merge!

@rukai would you mind if we published 0.14 after this as another emergency patch to make it possible for Arch/Void users to build vulkano-shaders? Maybe we can also include #1227 and #1225, though I'm not well versed enough to understand if #1225 is correct/safe.

Edit: Oh also now that I've merged #1225 we'll need to fix the conflict in the CHANGELOG.

@rukai
Copy link
Member

rukai commented Aug 3, 2019

Sure, feel free to publish 0.14 if need be, the website would additionally need to be updated.

I cannot review PRs as i am already using my free time to work on gfx Hal/wgpu stack. Sorry. It might be a simple change but it's hard to tell without investing a lot of time in the first place.

@AustinJ235
Copy link
Member

@rukai I'd like to see this go through and a version published, my projects are still on vulkano 11.1 due to the linking errors with shaderc. That issue is solved by this pull request.

@m4b
Copy link

m4b commented Aug 12, 2019

can this be emergency pushed? basically all vulkano projects are broken now on archlinux et. al

@rukai
Copy link
Member

rukai commented Aug 12, 2019

You can workaround it by adding the build-from-source feature to shaderc. As I have done here https://github.com/rukai/PF_Sandbox/blob/master/pf_sandbox/Cargo.toml

Can @Abendstolz fix the CI errors? Then I can proceed with an emergency release

@Abendstolz
Copy link
Contributor Author

@rukai or @mitchmindtree go for it ;)

@rukai rukai merged commit 0a89443 into vulkano-rs:master Aug 12, 2019
mitchmindtree added a commit to mitchmindtree/vulkano that referenced this pull request Aug 17, 2019
This is kind of a delayed emergency release to address an issue where
some Linux distributions (Arch, Void and perhaps others) are unable to
build any projects depending on vulkano-shaders due to
google/shaderc-rs#58. This is resolved in shaderc 0.6 and in turn will
be resolved in vulkano 0.14 thanks to vulkano-rs#1226.

See the CHANGELOG-VULKANO.md for more details on the release.
mitchmindtree added a commit to mitchmindtree/vulkano that referenced this pull request Aug 17, 2019
This is kind of a delayed emergency release to address an issue where
some Linux distributions (Arch, Void and perhaps others) are unable to
build any projects depending on vulkano-shaders due to
google/shaderc-rs#58. This is resolved in shaderc 0.6 and in turn will
be resolved in vulkano 0.14 thanks to vulkano-rs#1226.

See the CHANGELOG-VULKANO.md for more details on the release.
mitchmindtree added a commit that referenced this pull request Aug 17, 2019
This is kind of a delayed emergency release to address an issue where
some Linux distributions (Arch, Void and perhaps others) are unable to
build any projects depending on vulkano-shaders due to
google/shaderc-rs#58. This is resolved in shaderc 0.6 and in turn will
be resolved in vulkano 0.14 thanks to #1226.

See the CHANGELOG-VULKANO.md for more details on the release.
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.

5 participants