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

Review URL: https://codereview.chromium.org//362353003

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge@37961 260f80e4-7a28-3924-810f-c04153c831b5

pub/test/global/deactivate/unexpected_arguments_test
pub/test/global/deactivate/missing_package_arg_test
Started failing after revision 37915

Filed http://dartbug.com/19784 to track this.

I wonder why these did not show up failing running the tests locally (the tests where actually changed)

R=kasperl@google.com

Review URL: https://codereview.chromium.org//366853005

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge@37933 260f80e4-7a28-3924-810f-c04153c831b5

If we can tell (using $include/$exclude) that a dependency's
transformer can't run on any public files in that package, don't
bother loading that transformer at all.

R=rnystrom@google.com
BUG=17534

Review URL: https://codereview.chromium.org//363543003

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge@37925 260f80e4-7a28-3924-810f-c04153c831b5

If a local dependency isn't upgraded to the most recent available
version, pub will now indicate what that most recent version is in its
"pub upgrade" output.

R=rnystrom@google.com
BUG=16020

Review URL: https://codereview.chromium.org//366513002

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge@37924 260f80e4-7a28-3924-810f-c04153c831b5

R=rnystrom@google.com
BUG=17252

Review URL: https://codereview.chromium.org//363573002

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge@37915 260f80e4-7a28-3924-810f-c04153c831b5

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

Review URL: https://codereview.chromium.org//341433009

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge@37837 260f80e4-7a28-3924-810f-c04153c831b5

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

Review URL: https://codereview.chromium.org//331593012

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge@37832 260f80e4-7a28-3924-810f-c04153c831b5

R=rnystrom@google.com

Review URL: https://codereview.chromium.org//335313004

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge@37830 260f80e4-7a28-3924-810f-c04153c831b5

Also make confirmation messages respect --with-prejudice.

R=rnystrom@google.com

Review URL: https://codereview.chromium.org//347863006

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge@37772 260f80e4-7a28-3924-810f-c04153c831b5

Previously, pub would exit immediately after the first load error was
reported. Now it will wait to see if any other transformers have
errors and report those as well.

R=rnystrom@google.com
BUG=14552

Review URL: https://codereview.chromium.org//354053002

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge@37770 260f80e4-7a28-3924-810f-c04153c831b5

R=rnystrom@google.com
BUG=15736

Review URL: https://codereview.chromium.org//356953005

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge@37761 260f80e4-7a28-3924-810f-c04153c831b5

R=rnystrom@google.com
BUG=19565

Review URL: https://codereview.chromium.org//353943002

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge@37756 260f80e4-7a28-3924-810f-c04153c831b5

R=rnystrom@google.com
BUG=13658

Review URL: https://codereview.chromium.org//344413004

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge@37754 260f80e4-7a28-3924-810f-c04153c831b5

R=rnystrom@google.com
BUG=16008

Review URL: https://codereview.chromium.org//352923002

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge@37752 260f80e4-7a28-3924-810f-c04153c831b5

These packages are used in pub's plugin isolate, so pub needs to have
a dependency on them to ensure their APIs don't change out from under
it.

R=rnystrom@google.com
BUG=19574

Review URL: https://codereview.chromium.org//356523003

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge@37751 260f80e4-7a28-3924-810f-c04153c831b5

R=rnystrom@google.com

Review URL: https://codereview.chromium.org//352263002

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge@37715 260f80e4-7a28-3924-810f-c04153c831b5

This change does three things:
* Splits out a WrappedException class from ApplicationException.
* Uses fail() pervasively rather than new ApplicationException() everywhere.
* Moves exception classes from utils.dart to exceptions.dart.

R=rnystrom@google.com

Review URL: https://codereview.chromium.org//354663008

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge@37713 260f80e4-7a28-3924-810f-c04153c831b5

R=nweiz@google.com

Review URL: https://codereview.chromium.org//357563002

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge@37705 260f80e4-7a28-3924-810f-c04153c831b5

This uses the source spans exposed by the yaml package to associate pubspec
errors visually with the section of the pubspec that caused them.

R=rnystrom@google.com
BUG=14999

Review URL: https://codereview.chromium.org//351703004

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge@37675 260f80e4-7a28-3924-810f-c04153c831b5

This is prep work for global packages, which need to ensure their
dependencies are local, but do not want to create symlinks for them.

R=nweiz@google.com

Review URL: https://codereview.chromium.org//357483004

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge@37672 260f80e4-7a28-3924-810f-c04153c831b5

R=rnystrom@google.com

Review URL: https://codereview.chromium.org//345413004

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge@37616 260f80e4-7a28-3924-810f-c04153c831b5

R=nweiz@google.com

Review URL: https://codereview.chromium.org//346003002

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge@37572 260f80e4-7a28-3924-810f-c04153c831b5

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

Review URL: https://codereview.chromium.org//341123002

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge@37561 260f80e4-7a28-3924-810f-c04153c831b5

R=rnystrom@google.com
TBR

Review URL: https://codereview.chromium.org//332003003

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge@37528 260f80e4-7a28-3924-810f-c04153c831b5

In r37514 I incorrectly assumed that the problem was Directory.list
when in fact it's git.

R=rnystrom@google.com

Review URL: https://codereview.chromium.org//349503003

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge@37527 260f80e4-7a28-3924-810f-c04153c831b5

Also add support for --dry-run to pub get.

BUG=dartbug.com/19439 dartbug.com/15587
R=nweiz@google.com

Review URL: https://codereview.chromium.org//342763003

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge@37525 260f80e4-7a28-3924-810f-c04153c831b5

R=rnystrom@google.com
BUG=19554

Review URL: https://codereview.chromium.org//349433003

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge@37514 260f80e4-7a28-3924-810f-c04153c831b5

Previously, two transformers from the same library but with different
configuration was considered completely different from one another. This was not
only inefficient but incorrect: it meant that a transformer could be loaded with
the wrong code.

R=rnystrom@google.com
BUG=19261

Review URL: https://codereview.chromium.org//344673002

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge@37511 260f80e4-7a28-3924-810f-c04153c831b5

R=rnystrom@google.com

Review URL: https://codereview.chromium.org//348463003

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge@37471 260f80e4-7a28-3924-810f-c04153c831b5

In particular, this makes barback and the "pub lish" validators use gitignore
rules.

R=rnystrom@google.com
BUG=12102, 11198

Review URL: https://codereview.chromium.org//342553002

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge@37469 260f80e4-7a28-3924-810f-c04153c831b5

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

Review URL: https://codereview.chromium.org//344493003

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge@37464 260f80e4-7a28-3924-810f-c04153c831b5

Rather than only looking at the package-level dependency graph, pub will now
examine transformer files' imports to determine dependencies between
transformers. This improves both load times and cycle detection. For example, a
package using polymer transformers can now load them in two phases rather than
four, reducing the load time by about half.

In addition, this gives transformer authors more control over how parallel their
transformer graph is. With some $include/$exclude and import tweaks, most
remaining transformer dependencies can be eliminated.

R=rnystrom@google.com

Review URL: https://codereview.chromium.org//331263002

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge@37461 260f80e4-7a28-3924-810f-c04153c831b5

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

Review URL: https://codereview.chromium.org//333963002

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge@37427 260f80e4-7a28-3924-810f-c04153c831b5

R=rnystrom@google.com

Review URL: https://codereview.chromium.org//341643002

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge@37425 260f80e4-7a28-3924-810f-c04153c831b5

R=rnystrom@google.com
BUG=19221

Review URL: https://codereview.chromium.org//334363004

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge@37424 260f80e4-7a28-3924-810f-c04153c831b5

R=rnystrom@google.com

Review URL: https://codereview.chromium.org//331663006

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge@37373 260f80e4-7a28-3924-810f-c04153c831b5

R=nweiz@google.com

Review URL: https://codereview.chromium.org//332833006

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge@37337 260f80e4-7a28-3924-810f-c04153c831b5

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

Review URL: https://codereview.chromium.org//330183003

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge@37322 260f80e4-7a28-3924-810f-c04153c831b5

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

Review URL: https://codereview.chromium.org//329303006

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge@37319 260f80e4-7a28-3924-810f-c04153c831b5

Transformers that are used in the package that defines them will be loaded in
the same isolate only if they're in the same phase. Otherwise, all transformers
within a given package will be loaded in the same isolate.

A future CL will take more advantage of this by more aggressively determining
when transformers can be safely loaded in parallel.

R=rnystrom@google.com

Review URL: https://codereview.chromium.org//323263002

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge@37296 260f80e4-7a28-3924-810f-c04153c831b5