mitchellh / goamz

Golang Amazon Library
Other
672 stars 216 forks source link

SharedAuth: use os/user to locate home directory instead of HOME #229

Closed sinbad closed 9 years ago

sinbad commented 9 years ago

HOME isn't necessarily defined on Windows machines, per AWS docs %USERPROFILE% is the root used on that platform. For better compatibility don't manually use environment at all & use os/user package instead.

mitchellh commented 9 years ago

os/user requires cgo, which is why we don't do that. If you switch it to https://github.com/mitchellh/go-homedir then I'll merge.

sinbad commented 9 years ago

Thanks, re-submitted as suggested.