-
Notifications
You must be signed in to change notification settings - Fork 17.9k
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
x/mobile: gomobile init no longer works due to openal-soft and Go changes #46943
Comments
I've been dealing with these issues in our Android/iOS library we build using |
I have the same issue. Pointing out ndk tool chain makes it work. But then I only get 32bit library that is incompatible with my arm64 so file. So I then need to modify the cmake file to build 64bit. Then it works but this feels like something is not up to date as it requires changes to gomobile cmd source. Anyone else solved this in a better and more generic way? |
I tried changing the install line in diff --git a/init.go b/init.go
index 172d015..234b1c1 100644
--- a/init.go
+++ b/init.go
@@ -78,7 +78,7 @@ func runInit(cmd *command) error {
}()
// Make sure gobind is up to date.
- if err := goInstall([]string{"golang.org/x/mobile/cmd/gobind"}, nil); err != nil {
+ if err := goInstall([]string{"golang.org/x/mobile/cmd/gobind@latest"}, nil); err != nil {
return err
} I ran
|
I managed to build OpenAL etc. It required some changes to |
As I see it, this issue may be a duplicate of #50994 which was closed, and the |
Let me close this assuming this is duplicated with #50994. |
What version of Go are you using (
go version
)?Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (
go env
)?go env
OutputWhat did you do?
What did you expect to see?
It builds the shared library.
What did you see instead?
and if I make a go.mod file to avoid that error:
The text was updated successfully, but these errors were encountered: