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

Build with Gomobile #959

Closed
wants to merge 1 commit into from
Closed

Build with Gomobile #959

wants to merge 1 commit into from

Conversation

Nutomic
Copy link
Contributor

@Nutomic Nutomic commented Sep 28, 2017

I have started to try and make our build work with gomobile. If we can get this to work, it would give us a bunch of advantages over the existing build process:

There are still some problems with this. First of all, the build isn't working and throwing errors (output is here). But I have checked it with golang.org/x/mobile/example/bind/hello instead of Syncthing and it worked, so it seems to be a problem in Syncthing?

Also, gomobile doesn't seem to support building ARMv5 binaries, and creates only ARMv7 instead. I've created a question on Stackoverflow for that.

Setup instructions

Setup Go and Android build tools first. Then run:

go get golang.org/x/mobile/cmd/gomobile
go get golang.org/x/mobile/cmd/gobind
gomobile init -ndk /path/to/ndk
go get github.com/syncthing/syncthing/cmd/syncthing

Then just compile with ./gradlew assembleDebug.

ping @calmh @Zillode @AudriusButkevicius

@Nutomic
Copy link
Contributor Author

Nutomic commented Oct 8, 2017

Gomobile officially requires ARMv7: golang/go#22108 (comment). That means we will have to drop support for those devices after switching to Gomobile.

@Nutomic Nutomic force-pushed the gomobile branch 2 times, most recently from 480aab5 to 1b0a0b9 Compare October 10, 2017 13:48
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants