two tiny typos fixed

two tiny typos fixed



https://code.launchpad.net/~natefinch/juju-core/015-typos/+merge/186606

(do not edit description out of merge proposal)

[natefinch, 2013-09-19 18:31:44]

Please take a look.

[rog, 2013-09-19 19:26:39]

On 2013/09/19 18:31:44, nate.finch wrote:
> Please take a look.

LGTM

[rog, 2013-09-19 19:29:49]

On brief further reflection, using "choose" as part of a command line flag
description is quite unorthodox.

How about these alternative phrasings?

https://codereview.appspot.com/13791043/diff/1/cmd/juju/synctools.go
File cmd/juju/synctools.go (right):

https://codereview.appspot.com/13791043/diff/1/cmd/juju/synctools.go#newcode58
cmd/juju/synctools.go:58: f.StringVar(&c.source, "source", "", "choose a
location on the file system as source")
"local source directory"

https://codereview.appspot.com/13791043/diff/1/cmd/juju/synctools.go#newcode59
cmd/juju/synctools.go:59: f.StringVar(&c.destination, "destination", "", "choose
a location on the file system as destination")
"local destination directory"

[natefinch, 2013-09-19 19:37:24]

Yep, that looks significantly better. Honestly, I barely read the actual
messages... couldn't think past the typo.  I had approved my branch, so
it's possible the bot is in the process of trying to land it.  I reset it
to needs review, so if it doesn't land it, I'll make that additional change
on my branch.


On Thu, Sep 19, 2013 at 3:29 PM, <rogpeppe@gmail.com> wrote:

> On brief further reflection, using "choose" as part of a command line
> flag description is quite unorthodox.
>
> How about these alternative phrasings?
>
>
>
https://codereview.appspot.**com/13791043/diff/1/cmd/juju/**synctools.go<http...
> File cmd/juju/synctools.go (right):
>
> https://codereview.appspot.**com/13791043/diff/1/cmd/juju/**
>
synctools.go#newcode58<https://codereview.appspot.com/13791043/diff/1/cmd/juju/synctools.go#newcode58>
> cmd/juju/synctools.go:58: f.StringVar(&c.source, "source", "", "choose a
>
> location on the file system as source")
> "local source directory"
>
> https://codereview.appspot.**com/13791043/diff/1/cmd/juju/**
>
synctools.go#newcode59<https://codereview.appspot.com/13791043/diff/1/cmd/juju/synctools.go#newcode59>
> cmd/juju/synctools.go:59: f.StringVar(&c.destination, "destination", "",
>
> "choose a location on the file system as destination")
> "local destination directory"
>
>
https://codereview.appspot.**com/13791043/<https://codereview.appspot.com/137...
>

[natefinch, 2013-09-23 18:54:01]

Please take a look.