Skip to content
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: universe.objc.h file copy fails on case-sensitive system #28335

Closed
tadovas opened this issue Oct 23, 2018 · 1 comment
Closed

x/mobile: universe.objc.h file copy fails on case-sensitive system #28335

tadovas opened this issue Oct 23, 2018 · 1 comment

Comments

@tadovas
Copy link

@tadovas tadovas commented Oct 23, 2018

Please answer these questions before submitting your issue. Thanks!

What version of Go are you using (go version)?

go version go1.11 linux/amd64

Does this issue reproduce with the latest release?

yes

What operating system and processor architecture are you using (go env)?

GOARCH="amd64"
GOBIN=""
GOCACHE="/root/.cache/go-build"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/go"
GOPROXY=""
GORACE=""
GOROOT="/usr/local/go"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/linux_amd64"
GCCGO="gccgo"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build230554153=/tmp/go-build -gno-record-gcc-switches"

What did you do?

gomobile bind -x -v -work --target=ios/arm64

If possible, provide a recipe for reproducing the error.
A complete runnable program is good.
A link on play.golang.org is best.

What did you expect to see?

Successful gomobile bind on linux

What did you see instead?

gomobile bind tries to copy universe.objc.h file (instead of generated Universe.objc.h).
cp $WORK/src/gobind/universe.objc.h mobile.framework/Versions/A/Headers/universe.objc.h
This works on darwin (somehow), but obviously fails on linux machines (because of upper/lower case mismatch.
It's a single line fix. PR is coming

@gopherbot

This comment has been minimized.

Copy link

@gopherbot gopherbot commented Oct 23, 2018

Change https://golang.org/cl/144117 mentions this issue: gomobile: Start universe filename uppercase

@ALTree ALTree changed the title x/mobile x/mobile: universe.objc.h file copy fails on case-sensitive system Oct 23, 2018
@ALTree ALTree added the NeedsFix label Oct 23, 2018
@golang golang locked and limited conversation to collaborators Oct 26, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants
You can’t perform that action at this time.