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

Conversation

Projects
None yet
2 participants
@Nutomic
Copy link
Member

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:

  • no more hacks needed to make DNS work
  • we can get rid of all the custom bash scripts to build Syncthing
  • fixes issues #706, #873, #960
  • we can eventually add sdcard support using gobind (allows calling Android Java APIs from Go)

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

This comment has been minimized.

Copy link
Member Author

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 Oct 10, 2017

@Nutomic Nutomic force-pushed the gomobile branch from 1b0a0b9 to f48462c Oct 10, 2017

@Nutomic Nutomic closed this Oct 13, 2017

@Nutomic Nutomic deleted the gomobile branch Oct 17, 2017

@syncthing syncthing locked and limited conversation to collaborators Nov 9, 2018

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
You can’t perform that action at this time.