Skip to content
This repository has been archived by the owner on Jan 24, 2019. It is now read-only.

Panic on startup without GPM #216

Open
RobCherry opened this issue Feb 20, 2016 · 3 comments
Open

Panic on startup without GPM #216

RobCherry opened this issue Feb 20, 2016 · 3 comments
Labels

Comments

@RobCherry
Copy link

Built via go get github.com/bitly/oauth2_proxy command provided in the readme.

Appears to have been caused by mreiferson/go-options#4.

panic: interface conversion: *main.StringArray is not flag.Getter: missing method Get

goroutine 1 [running]:
github.com/mreiferson/go-options.Resolve(0x8692c0, 0xc8200c8580, 0xc820037ee8, 0xc82008af90)
    /go/src/github.com/mreiferson/go-options/options.go:86 +0xe7c
main.main()
    /go/src/github.com/bitly/oauth2_proxy/main.go:93 +0x15e6

The installation steps in the readme need to be updated to reference using https://github.com/pote/gpm, or changes need to be made to support the latest version of go-options.

@mreiferson
Copy link

whooops

@mreiferson
Copy link

You should probably use godep to build the binary, though, so you don't pickup random changes to master in its dependencies.

@RobCherry
Copy link
Author

@mreiferson Yeah, using https://github.com/pote/gpm I was able to get it building again. I am going to update the issue to reflect that this project will not run with the latest versions of https://github.com/mreiferson/go-options.

@RobCherry RobCherry changed the title Panic on startup Panic on startup without GPM Feb 21, 2016
@jehiah jehiah added the bug label Feb 25, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Development

No branches or pull requests

3 participants