Code review comment for lp://staging/~dave-cheney/juju-core/005-cmd-juju-scp

https://codereview.appspot.com/6565055/diff/5001/cmd/juju/scp.go
File cmd/juju/scp.go (right):

https://codereview.appspot.com/6565055/diff/5001/cmd/juju/scp.go#newcode51
cmd/juju/scp.go:51: // BUG(dfc) This will also not work if the local
path has a : in it.
Parsing the value we obtain and checking if it's a number of a
IsUnitName would remove most ambiguities as it will be quite rare to see
a path with a /<number>:/ component. This isn't much worse than the same
issue in the stock scp (<email address hidden> as a path?).

https://codereview.appspot.com/6565055/