juju-core

Code review comment for lp:~axwalk/juju-core/lp1274780-osuser-fallback

  1. lp1274780-osuser-fallback
  2. Merge into trunk
Revision history for this message
Tim Penhey (thumper) wrote :

https://codereview.appspot.com/58970043/diff/1/provider/local/environprovider.go
File provider/local/environprovider.go (right):

https://codereview.appspot.com/58970043/diff/1/provider/local/environprovider.go#newcode60
provider/local/environprovider.go:60: username = u.Username
should we lowercase this?

https://codereview.appspot.com/58970043/

« Back to merge proposal