revise lua script to use zig tool chain for stride#1
revise lua script to use zig tool chain for stride#1jazzay wants to merge 3 commits intostride3d:masterfrom
Conversation
manio143
left a comment
There was a problem hiding this comment.
Overall I think it looks good. Left some small comments.
Not sure how this Lua is part of the build to be honest, so an example on how to invoke it would be nice.
This is old build tooling that predates the formation of the OSS Xenko. You can see where it is used to build NativePath inside deps/NativePath/build.bat. I have also added a new build.bat for the Celt dependency that uses this as well, since we need an update to that dep to support the new platform targets. I will be putting up a separate PR in the main Stride repo that illustrates changes related to this soon. Sadly this is in a separate repo. |
Actually it was Celt dep that was using this already, I've added a new build.bat for NativePath that's incoming. |
|
Sorry, don't hesitate to add me as reviewer in case I miss the message next time! |
xen2
left a comment
There was a problem hiding this comment.
LGTM!
Maybe I better create a zig-cross branch on this repo so you that you can merge the successive PR?
I think we can just merge straight into master even with these new commits? |
These changes are required to support building NativePath and Celt for newer platforms like osx-arm64.
This will have no impact on Stride itself until I push the changes to deps, which would be into the Stride repo.