Commits · 47abd53705a679630661764ebc31f13275a535d0 · fuchsia-mirror / dart.googlesource.com-pub

May 14, 2015

Fix some pub analyzer warnings. · 47abd537

nweiz@google.com authored 10 years ago

R=rnystrom@google.com
TBR

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

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

47abd537

Precompute untransformed packages when determining trnasformer dependencies. · 40a34747

nweiz@google.com authored 10 years ago

This takes "pub run polymer:new_element --help" from 0.96s to 0.75s.

R=rnystrom@google.com
BUG=20859

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

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

40a34747

Tell the user if the binstub directory is not on their path. · 7bcddc32

rnystrom@google.com authored 10 years ago

R=nweiz@google.com

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

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

7bcddc32

Don't use .gitignore when determining what to build/serve/run. · ac9be18a

rnystrom@google.com authored 10 years ago

(Still use it for publishing.)

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

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

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

ac9be18a

Fix a couple minor bugs/typos in pub. · f88ec5ca

nweiz@google.com authored 10 years ago

R=rnystrom@google.com

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

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

f88ec5ca

Don't load transformers that aren't going to be used for an executable. · f44636f1

nweiz@google.com authored 10 years ago

A package's executables may not use all of that package's transformers. This is
especially true for browser frameworks where there's a clear divide between
client-side and server-side code. This uses pub's existing dependency-sniffing
infrastructure to figure out which transformers can be omitted.

This takes "pub run polymer:new_element --help" from 1.52s to 0.96s.

R=rnystrom@google.com
BUG=20859

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

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

f44636f1

Remove unused parameter. · 46967af9

rnystrom@google.com authored 10 years ago

This also fixes an analyzer warning since the param had the wrong type.

R=nweiz@google.com

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

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

46967af9

Fix the listing of built-in packages for "pub build". · 70ef0265

nweiz@google.com authored 10 years ago

R=rnystrom@google.com
BUG=21068

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

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

70ef0265

Add Shebang to Global Package binstubs · 201b9e5d

rnystrom@google.com authored 10 years ago

BUG=
R=rnystrom@google.com

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

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

201b9e5d

Remove disable of GZIP, as it's no longer on by default anymore. · 479c8236

ajohnsen@google.com authored 10 years ago

BUG=
R=rnystrom@google.com

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

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

479c8236

Fix a warning introduced by the previous CL. · 9e321f5c

nweiz@google.com authored 10 years ago

R=rnystrom@google.com
BUG=

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

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

9e321f5c

Fix an error-swallowing bug in pub and several bugs it was hiding. · 88625697

nweiz@google.com authored 10 years ago

We were passing an error to [completer.completer] rather than
[completer.completeError] and this was hiding several behavioral bugs.

R=rnystrom@google.com
TBR

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

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

88625697

Have barback lazily load assets from static pub packages. · 7a44321d

nweiz@google.com authored 10 years ago

This takes "pub run new_element --help" from about 1.75s to about 1.60s.

R=rnystrom@google.com
BUG=20859

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

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

7a44321d

Regenerate a snapshot if the first isolate fails to load. · 6c6beb55

rnystrom@google.com authored 10 years ago

R=nweiz@google.com

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

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

6c6beb55

Ensure dart and pub are on PATH when spawning binstub processes in test. · 851d9f95

rnystrom@google.com authored 10 years ago

BUG=
R=nweiz@google.com

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

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

851d9f95

Pass a package root to spawned isolates in pub. · 5d6952a9

nweiz@google.com authored 10 years ago

This allows pub to get rid of the RewriteImportTransformer. This reduces the
runtime of "pub run polymer:new_element --help" by about 270ms.

R=rnystrom@google.com
BUG=20859

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

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

5d6952a9

Use precompiled dependencies in pub. · 3451d3f0

nweiz@google.com authored 10 years ago

BUG=20859
R=rnystrom@google.com

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

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

3451d3f0

Fix type annotation. · c2a904b1

rnystrom@google.com authored 10 years ago

R=nweiz@google.com

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

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

c2a904b1

Make binstubs run snapshots directly when possible. · 7569ed87

rnystrom@google.com authored 10 years ago

R=nweiz@google.com

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

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

7569ed87

Allow ".dart" extension in pub run. · ec510f8d

rnystrom@google.com authored 10 years ago

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

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

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

ec510f8d

Get rid of Chain.track in pub. · 381ec7a9

nweiz@google.com authored 10 years ago

R=rnystrom@google.com

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

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

381ec7a9

Fix binstub test on windows. · f3173651

rnystrom@google.com authored 10 years ago

R=nweiz@google.com

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

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

f3173651

Precompile immutable dependencies on "pub get". · 5e2cdf8a

nweiz@google.com authored 10 years ago

R=rnystrom@google.com
BUG=20859

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

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

5e2cdf8a

Create binstubs for executables when activating a package. · 73f82a4b

rnystrom@google.com authored 10 years ago

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

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

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

73f82a4b

Cache snapshots of (mostly) immutable transformer phases. · 1ddae179

nweiz@google.com authored 10 years ago

This shaves off a pretty significant amount of time for packages that are
transformed but whose transformers never change (I saw a ~450ms gain in one of
my tests).

R=rnystrom@google.com
BUG=20859

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

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

1ddae179

Add a --mode flag to pub run and pub global run. · ef58d6df

nweiz@google.com authored 10 years ago

pub run now defaults to --mode=debug for local executables.

R=rnystrom@google.com
BUG=20898

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

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

ef58d6df

Parse the repo's version file when running from the repo. · b6a00125

rnystrom@google.com authored 10 years ago

R=nweiz@google.com

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

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

b6a00125

Revert "try again to not auto start editor port" · 53ee254d

sethladd@google.com authored 10 years ago

This reverts commit 043c5de2d213a3f3dd9c9b023eb42e1badefb8f0.

BUG=

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

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

53ee254d

try again to not auto start editor port · e05e6f89

sethladd@google.com authored 10 years ago

BUG=

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

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

e05e6f89

Revert "only run the admin server if admin-port is specified" · 7ca6897b

sethladd@google.com authored 10 years ago

This reverts commit 9ed84a9d820a57132d915cd069b47b33ed6f5631.

BUG=
R=scheglov@google.com

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

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

7ca6897b

only run the admin server if admin-port is specified · ca5e9300

sethladd@google.com authored 10 years ago

BUG=
R=nweiz@google.com

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

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

ca5e9300

Don't try to load unnecessary transformers when precompiling executables. · 51f6dbe3

nweiz@google.com authored 10 years ago

R=jmesserly@google.com
BUG=20917

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

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

51f6dbe3

Add suppression to pub that got lost in move of status file in revision 40123 · 57817684

ricow@google.com authored 10 years ago

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

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

57817684

Delete stale compiled files. · 71360222

rnystrom@google.com authored 10 years ago

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

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

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

71360222

Add the ability for a PackageProvider to declare static packages. · 648c7872

nweiz@google.com authored 10 years ago

Once pub gets rid of RewriteTransformer, this will allow it to avoid
passing in assets to Barback unnecessarily.

R=rnystrom@google.com
BUG=20859

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

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

648c7872

Stop working around issue 20504 in pub. · e13aff17

nweiz@google.com authored 10 years ago

R=rnystrom@google.com

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

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

e13aff17

Add a missing .url. · 3253317f

rnystrom@google.com authored 10 years ago

BUG=

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

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

3253317f

Make compiler imports relative in async-compiled code. · 1a3c2afb

rnystrom@google.com authored 10 years ago

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

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

1a3c2afb

Store the async-await compiled pub code directly in the repo. · aa14da36

rnystrom@google.com authored 10 years ago

R=nweiz@google.com

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

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

aa14da36

Fix an analyzer error. · 787117e3

nweiz@google.com authored 10 years ago

R=rnystrom@google.com
TBR

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

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

787117e3