Add support for MacOS build on sample build scripts

NordSecurity / uniffi-bindgen-go

Uniffi bindings generator for Golang

Mozilla Public License 2.0

65 stars 18 forks source link

Add support for MacOS build on sample build scripts #51

Closed pacu closed 4 weeks ago

pacu commented 1 month ago

closes #50

This adds a few conditionals on build.sh, build_bindings.sh and test_bindings.sh to detect MacOS as host architecture and change build paths.

For some reason cargo ends up building intel 64-bit on Apple silicon instead of arm64 if no target is specified. This makes the scripts less simple but will work for anyone regardless whether they are building on Linux or MacOS.

arg0d commented 4 weeks ago

I just tried building and running the test on a fresh ARM Mac and everything worked fine. Maybe you have changed the default target triplet on your system? What is the output of rustup default on your system? I think rustup default stable should fix the problem. If this is indeed the case, I see no point to make the proposed changes to the scripts.

pacu commented 4 weeks ago

This indeed was a rustup burp. I don't know how that changed because I never did and all the other projects I worked in built just fine