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

add go bindings #144

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

add go bindings #144

wants to merge 1 commit into from

Conversation

nepet
Copy link

@nepet nepet commented Oct 11, 2021

Hey All,

this PR introduces bindings for Golang (based on the swig framework). Still, an integration in the build process is missing and needed. Maybe someone of you who is more familiar with meson and the gdk build process can help me out here.

Also the notification handler sample is not very useful but we dont need it right now. I might provide a better solution in the future or on request.

Regards,
Peter

@jgriffiths
Copy link
Contributor

Hi @nepet Thanks for this. I'm not familiar with go, would it be possible for you to give the swig, build and test run command lines you are using to get the the test to build/run?

Based on that I should be able to munge it into the meson build, I just need to verify that the resulting test compiles and runs correctly.

Copy link

@SSGoku369 SSGoku369 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok

@SSGoku369
Copy link

ok

@jackstar12
Copy link

jackstar12 commented Oct 2, 2023

@nepet any progress? I need this aswell and would be willing to help

@nepet
Copy link
Author

nepet commented Oct 6, 2023

@jackstar12 I am more than happy if you take over this (2 years old) PR!

@SSGoku369
Copy link

I need assistance adding the go binding. Who may I reach out to?
Please and thanks.

@jgriffiths
Copy link
Contributor

Hi @SSGoku369 if you can answer my question above then I can probably help integrate this.

@SSGoku369
Copy link

No progress. Where is a good starting point?
Thank you for your patience and services.

@SSGoku369
Copy link

SSGoku369 commented Nov 24, 2023 via email

@jgriffiths
Copy link
Contributor

For security reasons we don't give out contact details beyond our github profile and commit email addresses. You can contact me via my commit email or here.

@SSGoku369
Copy link

SSGoku369 commented Nov 24, 2023 via email

@YusukeShimizu
Copy link

YusukeShimizu commented Dec 25, 2023

@jgriffiths

would it be possible for you to give the swig, build and test run command lines you are using to get the the test to build/run?

I think we can run greenaddress_test.go by installing go and running the following command.

  1. Confirm go version.
$ go version

go version go1.21.3 darwin/arm64 is my env.

  1. init
$ go mod init github.com/Blockstream/gdk/src/swig_go
go: creating new go.mod: module github.com/Blockstream/gdk/src/swig_go
go: to add module requirements and sums:
	go mod tidy
  1. add module requirements and sums
$ go mod tidy
  1. Running SWIG with Go
$ swig -go -cgo swig_gasdk.i
  1. test examples
$ go test

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants