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 libdcrwallet dependency with few implemented methods #8
add libdcrwallet dependency with few implemented methods #8
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Currently unable to build the library with invalid value "c-archive" for flag -buildmode: buildmode c-archive not supported on android/arm
9907980
to
232ea5e
Compare
eedaa19
to
583ff18
Compare
b65ff0a
to
eef0db7
Compare
3ed002d
to
6ed9ab5
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good overall, just a couple of questions
6ed9ab5
to
53bd705
Compare
a57e019
to
6c1843d
Compare
6c1843d
to
6151bb7
Compare
53bd705
to
48e1437
Compare
6151bb7
to
5c18d35
Compare
5c18d35
to
77fcd84
Compare
13fbbde
to
027fd55
Compare
2cb10e8
to
7c9afc1
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Working great!
849750a
to
661b2df
Compare
77fcd84
to
4825ea1
Compare
661b2df
to
61f398c
Compare
rm -fr $LIBWALLET_PATH | ||
fi | ||
mkdir -p $LIBWALLET_PATH | ||
git clone $LIBWALLET_URL $LIBWALLET_PATH --branch cgo |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this should point at the specific commit that this pr should work on. That is possible right? e1b9f6043359d191cfd2fbfaa345396a009d0946
In case we make a breaking change, which I think is fine at this point.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, this makes a lot of sense.
closing because merged with decred_main upstream |
Build steps now include fetching the libwallet go source code from https://github.com/itswisdomagain/libwallet/tree/cgo and generating the appropriate library for the target OS.