Rename "install" -> "get" and "update" -> "upgrade".

Rename "install" -> "get" and "update" -> "upgrade".

BUG=https://code.google.com/p/dart/issues/detail?id=7351
R=nweiz@google.com

Committed: https://code.google.com/p/dart/source/detail?r=28330

[Bob Nystrom, 2013-10-07 16:45:50]

I went through everything and removed all uses of "install" and "update" that
referred to the old commands. My goal was to get the codebase looking like "get"
and "upgrade" were how we had always referred to them and not have any
historical cruft visible.

- Everything referring to the "get" pub command or setting up a package so that
an entrypoint package can use it is "get".
- Everything referring to acquiring a package and placing it in the system cache
is "download".
- Everything referring to updating a package to the latest version is "upgrade".

Sorry for the giant patch, but it's all just simple renames and some
documentation rewording.

I didn't do "deploy" -> "build" in this patch. I'll do that separately.

[nweiz, 2013-10-07 21:25:14]

A couple of suggestions, otherwise lgtm.

https://codereview.chromium.org/26284002/diff/1/sdk/lib/_internal/pub/lib/src...
File sdk/lib/_internal/pub/lib/src/command/serve.dart (right):

https://codereview.chromium.org/26284002/diff/1/sdk/lib/_internal/pub/lib/src...
sdk/lib/_internal/pub/lib/src/command/serve.dart:129: log.message("Dependencies
have changed, getting...");
Not super related to this CL, but can we make this message a little more
detailed? The mailing list thread made it pretty clear that people didn't
understand that this was doing a minimal amount of work, and a different message
might clear that up.

https://codereview.chromium.org/26284002/diff/1/sdk/lib/_internal/pub/lib/src...
File sdk/lib/_internal/pub/lib/src/command/upgrade.dart (right):

https://codereview.chromium.org/26284002/diff/1/sdk/lib/_internal/pub/lib/src...
sdk/lib/_internal/pub/lib/src/command/upgrade.dart:16: "Upgrade the current
package's dependencies to latest versions.";
Why was "the" removed here?

https://codereview.chromium.org/26284002/diff/1/sdk/lib/_internal/pub/lib/src...
File sdk/lib/_internal/pub/lib/src/entrypoint.dart (right):

https://codereview.chromium.org/26284002/diff/1/sdk/lib/_internal/pub/lib/src...
sdk/lib/_internal/pub/lib/src/entrypoint.dart:45: /// directory, or have already
been downloaded.
It seems weird that the docstring says "download" but the variable name says
"get".

https://codereview.chromium.org/26284002/diff/1/sdk/lib/_internal/pub/lib/src...
sdk/lib/_internal/pub/lib/src/entrypoint.dart:108: /// Gets all dependencies of
the [root] package and places them in its
"Gets" -> "Downloads", since placing them in the "packages" directory is defined
as the difference between the two. Same for [upgradeDependencies] and
[upgradeAllDependencies].

[Bob Nystrom, 2013-10-07 22:24:39]

Committed patchset #1 manually as r28330 (presubmit successful).

[Bob Nystrom, 2013-10-07 22:32:24]

https://codereview.chromium.org/26284002/diff/1/sdk/lib/_internal/pub/lib/src...
File sdk/lib/_internal/pub/lib/src/command/serve.dart (right):

https://codereview.chromium.org/26284002/diff/1/sdk/lib/_internal/pub/lib/src...
sdk/lib/_internal/pub/lib/src/command/serve.dart:129: log.message("Dependencies
have changed, getting...");
On 2013/10/07 21:25:15, nweiz wrote:
> Not super related to this CL, but can we make this message a little more
> detailed? The mailing list thread made it pretty clear that people didn't
> understand that this was doing a minimal amount of work, and a different
message
> might clear that up.

Done.

https://codereview.chromium.org/26284002/diff/1/sdk/lib/_internal/pub/lib/src...
File sdk/lib/_internal/pub/lib/src/command/upgrade.dart (right):

https://codereview.chromium.org/26284002/diff/1/sdk/lib/_internal/pub/lib/src...
sdk/lib/_internal/pub/lib/src/command/upgrade.dart:16: "Upgrade the current
package's dependencies to latest versions.";
On 2013/10/07 21:25:15, nweiz wrote:
> Why was "the" removed here?

To keep the usage under 80 chars. "upgrade" was just long enough to push it
over.

https://codereview.chromium.org/26284002/diff/1/sdk/lib/_internal/pub/lib/src...
File sdk/lib/_internal/pub/lib/src/entrypoint.dart (right):

https://codereview.chromium.org/26284002/diff/1/sdk/lib/_internal/pub/lib/src...
sdk/lib/_internal/pub/lib/src/entrypoint.dart:45: /// directory, or have already
been downloaded.
On 2013/10/07 21:25:15, nweiz wrote:
> It seems weird that the docstring says "download" but the variable name says
> "get".

Yeah. Reworded.

https://codereview.chromium.org/26284002/diff/1/sdk/lib/_internal/pub/lib/src...
sdk/lib/_internal/pub/lib/src/entrypoint.dart:108: /// Gets all dependencies of
the [root] package and places them in its
On 2013/10/07 21:25:15, nweiz wrote:
> "Gets" -> "Downloads", since placing them in the "packages" directory is
defined
> as the difference between the two. Same for [upgradeDependencies] and
> [upgradeAllDependencies].

I've tried to have "download" consistently mean "put in system cache" and "get"
mean "make available for use by a package" (which may sometimes require
downloading first).

In the case where all of your dependencies are path, none of these functions
will hit the system cache, so I think "get" is right here.

[nweiz, 2013-10-07 23:23:07]

https://codereview.chromium.org/26284002/diff/1/sdk/lib/_internal/pub/lib/src...
File sdk/lib/_internal/pub/lib/src/entrypoint.dart (right):

https://codereview.chromium.org/26284002/diff/1/sdk/lib/_internal/pub/lib/src...
sdk/lib/_internal/pub/lib/src/entrypoint.dart:108: /// Gets all dependencies of
the [root] package and places them in its
On 2013/10/07 22:32:25, Bob Nystrom wrote:
> On 2013/10/07 21:25:15, nweiz wrote:
> > "Gets" -> "Downloads", since placing them in the "packages" directory is
> defined
> > as the difference between the two. Same for [upgradeDependencies] and
> > [upgradeAllDependencies].
> 
> I've tried to have "download" consistently mean "put in system cache" and
"get"
> mean "make available for use by a package" (which may sometimes require
> downloading first).
> 
> In the case where all of your dependencies are path, none of these functions
> will hit the system cache, so I think "get" is right here.

I feel like if you're going to use "get" here you shouldn't also explicitly call
out that it "places them in its 'packages' directory", since that's part of what
"get" means. Saying it explicitly makes the distinction less clear.

[Bob Nystrom, 2013-10-08 21:15:03]

https://codereview.chromium.org/26284002/diff/1/sdk/lib/_internal/pub/lib/src...
File sdk/lib/_internal/pub/lib/src/entrypoint.dart (right):

https://codereview.chromium.org/26284002/diff/1/sdk/lib/_internal/pub/lib/src...
sdk/lib/_internal/pub/lib/src/entrypoint.dart:108: /// Gets all dependencies of
the [root] package and places them in its
On 2013/10/07 23:23:08, nweiz wrote:
> On 2013/10/07 22:32:25, Bob Nystrom wrote:
> > On 2013/10/07 21:25:15, nweiz wrote:
> > > "Gets" -> "Downloads", since placing them in the "packages" directory is
> > defined
> > > as the difference between the two. Same for [upgradeDependencies] and
> > > [upgradeAllDependencies].
> > 
> > I've tried to have "download" consistently mean "put in system cache" and
> "get"
> > mean "make available for use by a package" (which may sometimes require
> > downloading first).
> > 
> > In the case where all of your dependencies are path, none of these functions
> > will hit the system cache, so I think "get" is right here.
> 
> I feel like if you're going to use "get" here you shouldn't also explicitly
call
> out that it "places them in its 'packages' directory", since that's part of
what
> "get" means. Saying it explicitly makes the distinction less clear.

Done. See: https://codereview.chromium.org/26284002