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/cmd/gomobile: bind -target=ios produces an invalid framework #12604

Closed
hyangah opened this issue Sep 13, 2015 · 3 comments
Closed

x/mobile/cmd/gomobile: bind -target=ios produces an invalid framework #12604

hyangah opened this issue Sep 13, 2015 · 3 comments

Comments

@hyangah
Copy link
Contributor

hyangah commented Sep 13, 2015

It results in an error: bundle format unrecognized, invalid, or unsuitable

As s@ pointed out, it's missing info.plist in Resources file.
https://groups.google.com/forum/#!topic/golang-nuts/NqYUz0dq034

We didn't notice this problem because when releasing Ivy, we didn't use the framework - we used the fat .a file and header files directly.

/cc @crawshaw

@sridharv
Copy link

You should be able to build an IPA which depends on the framework without this as well. As far as I can see it is only needed if you attempt to codesign the framework manually.

@scisci
Copy link

scisci commented Sep 13, 2015

Correct, its still possible to link against the framework even though it is incomplete. When I brought up this issue, I was actually trying to embed the framework into my binary which was bringing up these errors. That was the incorrect approach to begin with, so I would say this is low priority.

However, if you want to make a well formed framework you need to make sure to symlink the Resources folder to the root level and also have an Info.plist in there.

But again, its unnecessary if you are just linking against a static framework.

@gopherbot
Copy link

CL https://golang.org/cl/14543 mentions this issue.

@golang golang locked and limited conversation to collaborators Sep 22, 2016
imWildCat pushed a commit to imWildCat/go-mobile that referenced this issue Apr 10, 2021
Fixes golang/go#12604.

Change-Id: I94e94c32ce42022b58097c349fcbb90edb0a2f13
Reviewed-on: https://go-review.googlesource.com/14543
Reviewed-by: David Crawshaw <crawshaw@golang.org>
imWildCat pushed a commit to imWildCat/go-mobile that referenced this issue Apr 11, 2021
Fixes golang/go#12604.

Change-Id: I94e94c32ce42022b58097c349fcbb90edb0a2f13
Reviewed-on: https://go-review.googlesource.com/14543
Reviewed-by: David Crawshaw <crawshaw@golang.org>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants