Merge pull request #4069 from guggero/windows-build

travis+lncli: fix windows build
This commit is contained in:
Johan T. Halseth 2020-03-11 14:51:54 +01:00 committed by GitHub
commit 44ba481f56
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 6 additions and 1 deletions

@ -43,6 +43,7 @@ jobs:
- make unit pkg=... case=_NONE_ - make unit pkg=... case=_NONE_
- make lint - make lint
- make btcd - make btcd
- LNDBUILDSYS=windows-amd64 bash ./build/release/release.sh
- stage: Test - stage: Test
script: make travis-cover script: make travis-cover
name: Unit Cover name: Unit Cover

@ -1807,7 +1807,11 @@ func unlock(ctx *cli.Context) error {
// lncli. // lncli.
default: default:
fmt.Printf("Input wallet password: ") fmt.Printf("Input wallet password: ")
pw, err = terminal.ReadPassword(syscall.Stdin)
// The variable syscall.Stdin is of a different type in the
// Windows API that's why we need the explicit cast. And of
// course the linter doesn't like it either.
pw, err = terminal.ReadPassword(int(syscall.Stdin)) // nolint:unconvert
fmt.Println() fmt.Println()
} }
if err != nil { if err != nil {