Skip to content

x/mobile/cmd/gomobile: build fails with SDK 34 #67496

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.

Sign up for GitHub

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

Jump to bottom
Closed
hajimehoshi opened this issue May 18, 2024 · 2 comments
Closed

x/mobile/cmd/gomobile: build fails with SDK 34 #67496

hajimehoshi opened this issue May 18, 2024 · 2 comments
Labels
mobile Android, iOS, and x/mobile NeedsFix The path to resolution is known, but the work has not been done.
Milestone
Unreleased

Comments

@hajimehoshi
Copy link
Member

Go version

go version go1.22.3 darwin/arm64

Output of go env in your module/workspace:

GO111MODULE=''
GOARCH='arm64'
GOBIN=''
GOCACHE='/Users/hajimehoshi/Library/Caches/go-build'
GOENV='/Users/hajimehoshi/Library/Application Support/go/env'
GOEXE=''
GOEXPERIMENT=''
GOFLAGS=''
GOHOSTARCH='arm64'
GOHOSTOS='darwin'
GOINSECURE=''
GOMODCACHE='/Users/hajimehoshi/go/pkg/mod'
GONOPROXY=''
GONOSUMDB=''
GOOS='darwin'
GOPATH='/Users/hajimehoshi/go'
GOPRIVATE=''
GOPROXY='https://proxy.golang.org,direct'
GOROOT='/usr/local/go'
GOSUMDB='sum.golang.org'
GOTMPDIR=''
GOTOOLCHAIN='auto'
GOTOOLDIR='/usr/local/go/pkg/tool/darwin_arm64'
GOVCS=''
GOVERSION='go1.22.3'
GCCGO='gccgo'
AR='ar'
CC='clang'
CXX='clang++'
CGO_ENABLED='1'
GOMOD='/Users/hajimehoshi/go-x-mobile/go.mod'
GOWORK=''
CGO_CFLAGS='-O2 -g'
CGO_CPPFLAGS=''
CGO_CXXFLAGS='-O2 -g'
CGO_FFLAGS='-O2 -g'
CGO_LDFLAGS='-O2 -g'
PKG_CONFIG='pkg-config'
GOGCCFLAGS='-fPIC -arch arm64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -ffile-prefix-map=/var/folders/cj/73zbb35j0qx5t4b6rnqq0__h0000gn/T/go-build3108044952=/tmp/go-build -gno-record-gcc-switches -fno-common'

What did you do?

  1. Prepare Android SDK 34. (If there are older versions, this bug might not be reproduced.)
  2. Install gomobile (a1a533f289d32c6297d63bfdc30e42c80b5a1f55)
  3. Run gomobile build -androidapi=34 ./example/basic at the gomobile directory

For 1, I installed Android SDK 34 or newer only:

image

What did you see happen?

Error:

gomobile: go build -buildmode=c-shared -o /var/folders/cj/73zbb35j0qx5t4b6rnqq0__h0000gn/T/gomobile-work-3204247024/lib/armeabi-v7a/libbasic.so golang.org/x/mobile/example/basic failed: exit status 1
# golang.org/x/mobile/app
app/android.go:368:6: could not determine kind of name for C.ALooper_pollAll

What did you expect to see?

The build succeeds.

The related issues are:
@gopherbot gopherbot added the mobile Android, iOS, and x/mobile label May 18, 2024
@gopherbot gopherbot added this to the Unreleased milestone May 18, 2024
@hajimehoshi
Copy link
Member Author

https://developer.android.com/ndk/reference/group/looper#alooper_pollall

ALooper_pollAll should be replaced with ALooper_pollOnce.

@gopherbot
Copy link
Contributor

Change https://go.dev/cl/586595 mentions this issue: all: replaced ALooper_pollAll with ALooper_pollOnce

hajimehoshi added a commit to ebitengine/gomobile that referenced this issue May 18, 2024
@hajimehoshi
all: replaced ALooper_pollAll with ALooper_pollOnce
e863328 
ALooper_pollAll is no longer available as of Android SDK 34.

Updates hajimehoshi/ebiten#2992
Updates golang/go#67496
@dmitshur dmitshur added the NeedsFix The path to resolution is known, but the work has not been done. label May 18, 2024
enjloezz added a commit to enjloezz/gomobile that referenced this issue Sep 7, 2024
@enjloezz
all: replace ALooper_pollAll with ALooper_pollOnce

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
GPG key ID: B5690EEEBB952194
Verified
Learn about vigilant mode
c4bd37a 
ALooper_pollAll is no longer available as of Android SDK 34.

Fixes golang/go#67496
mzki added a commit to mzki/erago that referenced this issue Oct 14, 2024
@mzki
.github: Update gomobile commit hash to include bug fix
Loading
Loading status checks…
c35c671 
See golang/go#67496
mzki added a commit to mzki/erago that referenced this issue Oct 14, 2024
@mzki
go.mod: Update gomobile commit hash to include bug fix
37b92c1 
See golang/go#67496
mzki added a commit to mzki/erago that referenced this issue Oct 14, 2024
@mzki
Fix mobile build fail
Loading
Loading status checks…
bcb3703 
This commit includes below updates:
* mobile/model/v2: Update minimum Android API level
* .github: Update gomobile commit hash to include bug fix
* go.mod: Update gomobile and its dependency to include bug fix

See golang/go#67496
@gabyhelp gabyhelp mentioned this issue Feb 8, 2025
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
mobile Android, iOS, and x/mobile NeedsFix The path to resolution is known, but the work has not been done.
Projects
None yet
Milestone
Unreleased
Development

No branches or pull requests
3 participants
@hajimehoshi @dmitshur @gopherbot