gfx-rs / wgpu-native

Native WebGPU implementation based on wgpu-core
Apache License 2.0
875 stars 105 forks source link

add gnu Windows/iOS/Android builds and improve continuous delivery workflow #405

Closed ygdrasil-io closed 2 months ago

ygdrasil-io commented 3 months ago
ygdrasil-io commented 2 months ago

For testing purposes, I created a fake release. If necessary, you can find the new builds here: https://github.com/ygdrasil-io/wgpu-native/releases/tag/v0.0.0.1

ygdrasil-io commented 2 months ago

https://github.com/ygdrasil-io/wgpu-native/releases/tag/v0.0.0.2 (There was a typo on Macos release)

The android version is working well : image

almarklein commented 2 months ago

This is great! I reviewed your changes, and it looks fine to me. Can you please also update the description at the top? It says "Each target (os + architecture) is build in a seperate job." but this is no longer the case.

Since this change is potentially impactful, I prefer to first merge #402 and do a release, then merge this pr and do another release.

almarklein commented 2 months ago

We can combine this with #410

almarklein commented 2 months ago

Note: #411 will probably introduce merge conflicts.

ygdrasil-io commented 2 months ago

@almarklein If you have the permissions, could you please retry the failing job? The cloning of the Git repository https://github.com/glfw/glfw.git seems to have failed for some reason.

almarklein commented 2 months ago

Awesome work, thanks! 🚀

almarklein commented 2 months ago

Out of curiosity, do you know why the linux-aarch64 build takes abour half an hour, while all other builds take less than 4 minutes (most less than 1 min)? Would it be due to emulating arm64 on x86?

ygdrasil-io commented 2 months ago

I think you are right, on my Mac Studio I see the same behaviour when I'm running a x86 VM, while arm VM are almost as quick of the host OS.

May be we could try cross compilation over virtualisation ?

almarklein commented 2 months ago

It's not the end of the world, I was mostly curious because that one build stands out.