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

README still mentions glide, while project is using go modules now #588

Open
Sosthene00 opened this issue Jan 30, 2019 · 3 comments
Open

Comments

@Sosthene00
Copy link

Hi I experienced some trouble with LND recently, and I was adviced to scan my wallet file using dropwtxmgr in btcwallet.

I'm following install instructions here, download btcwallet git directory, but when I run glide install I encounter this error :

[ERROR]	Failed to find glide.yaml file in directory tree: Cannot resolve parent of /

I'm not very familiar with Go, but I highly suspect it is related to the fact that I use the latest Go1.11, while it is advised in the README file to use 1.5 or 1.6.

Is there something I can do about it except downgrading for the time to run it and upgrading again?

uname -a: Linux Celeri 4.15.0-38-generic #41-Ubuntu SMP Wed Oct 10 10:59:38 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux

Thanks!

@wpaulino
Copy link
Contributor

The project uses go modules for dependencies now. You can build the binary with GO111MODULE=on go install ./dropwtxmgr.

@Sosthene00
Copy link
Author

Ok it worked, thanks. I will update the README file, ok?

@Sosthene00 Sosthene00 changed the title glide install [ERROR] Failed to find glide.yaml file in directory tree: Cannot resolve parent of / with Go1.11 README still mentions glide, while project is using go modules now Jan 30, 2019
wallclockbuilder added a commit to wallclockbuilder/btcwallet that referenced this issue Feb 20, 2019
Glide was removed by btcsuite@b523fd5 in favor of go modules. 
Update readme to reflect current reality.
 Also fixes btcsuite#588
@BernhardFuchs
Copy link

As of writing this the correct command is GO111MODULE=on go install -v . ./cmd/... run from /btcwallet. Found it in this PR.
Would be nice to finally merge this.

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 a pull request may close this issue.

4 participants
@BernhardFuchs @wpaulino @Sosthene00 and others