Commit graph

125 commits

Author SHA1 Message Date
Paul Berry
ceec063f27 Revert "Add dev_compiler to the SDK build process."
This broke the Mac and Windows builds, for reasons I'm still
investigating.

This reverts commit 2ddf244819.

TBR=jmesserly@google.com

Review URL: https://codereview.chromium.org/1813983003 .
2016-03-17 13:46:51 -07:00
Paul Berry
2ddf244819 Add dev_compiler to the SDK build process.
Also update DEPS to bring the latest dev_compiler into the SDK.

R=jmesserly@google.com, whesse@google.com

Review URL: https://codereview.chromium.org/1777173003 .
2016-03-17 11:14:35 -07:00
Paul Berry
fadca3983a Change sdk/bin/dart2js to find an available build.
This parallels the change to sdk/bin/dart made by commit
ac8a228d0b.

R=floitsch@google.com

Review URL: https://codereview.chromium.org/1650383002 .
2016-02-01 08:53:13 -08:00
William Hesse
634e5a1d02 Refs #25328 Use a portable shebang
BSD systems don't place bash in /bin and a lot of the dart tools
hardcode a #!/bin/bash shebang that fails the 'all' target build
(not able to execute dart2js since the interpreter is not found).

Solve the issue by using #!/usr/bin/env as the shebang. For scripts
that need to pass arguments to bash modify the script to use the set
command as the first executed line of shell.

BUG=
R=whesse@google.com

Review URL: https://codereview.chromium.org/1552313002 .
2016-01-05 16:26:28 +01:00
William Hesse
73dfc7afe0 Change to X64 default build in tool scripts in sdk/bin
BUG=
R=sgjesse@google.com

Review URL: https://codereview.chromium.org/1535553003 .
2015-12-17 14:49:19 +01:00
William Hesse
6485590527 Remove old java-based analyzer from test scripts and status files
BUG=
R=sgjesse@google.com

Review URL: https://codereview.chromium.org/1537653002 .
2015-12-17 14:44:57 +01:00
Florian Loitsch
e9ce747dac Run x64 dart2js version by default.
R=sigmund@google.com

Review URL: https://codereview.chromium.org/1484803003 .
2015-12-01 00:50:53 +01:00
pq
9ae1265ff5 analyzer_cli move to SDK.
Tracking bug: https://github.com/dart-lang/sdk/issues/24731

Note, dartium build changes are in a separate CL: https://codereview.chromium.org/1453413006/

Some tests had to be disabled for want of mockito in the SDK; tracking their reimplementation is here: https://github.com/dart-lang/sdk/issues/24994

BUG=24731
R=whesse@google.com

Review URL: https://codereview.chromium.org/1459683003 .
2015-11-19 14:52:54 -08:00
keertip
18b68f7290 remove docgen remnants from repo, update CHANGELOG
BUG=
R=alanknight@google.com, whesse@google.com

Review URL: https://codereview.chromium.org//1361163002 .
2015-09-23 10:27:15 -07:00
keertip
c775149a5a remove docgen source and targets from build
BUG=
R=alanknight@google.com

Review URL: https://codereview.chromium.org//1364553002 .
2015-09-22 15:10:18 -07:00
keertip
fe5550e934 fix dartdoc.bat
BUG=

Review URL: https://codereview.chromium.org//1318603006 .
2015-09-01 10:42:22 -07:00
Stan Manilov
fe402c50ea sdk/dartanalyzer now detects an available build
Similar to CL 1276333002; this allows the dartanalyzer script to pick an
available build, rather than failing if ReleaseIA32 is not available.

BUG=
R=floitsch@google.com

Review URL: https://codereview.chromium.org//1314893003 .
2015-08-25 20:48:14 +02:00
keertip
3865dd8748 create dartdoc snapshot, add to dart-sdk. Also add resources needed to run dartdoc from the sdk.
BUG=
R=whesse@google.com

Review URL: https://codereview.chromium.org//1295073003 .
2015-08-19 08:02:41 -07:00
Stan Manilov
ac8a228d0b sdk/dart now detects an available build
The shell scripts in sdk/sdk are using ReleaseIA32 as the default build
configuration in the case one is not provided as an environment
variable. This is arbitrary so a check of the available builds is
performed and if ReleaseIA32 is available it is selected. Otherwise, an
arbitrary configuration is selected (currently the last one
lexicographically). In the case where there are no subfolders in the
"out" folder, ReleaseIA32 is selected, in order to produce the same
error that used to be produced before (this can be changed to a custom
message for user-friendliness).

Currently only the "dart" shell script is changed and if the change is
accepted, the others will be changed too.

R=floitsch@google.com, zerny@google.com

Review URL: https://codereview.chromium.org//1276333002 .
2015-08-12 15:19:23 +02:00
Rico Wind
f3c6339206 Switch over testing pub seperately from the the normal packages
This will call a new test script that Natalie will fill in the blanks of for pub testing.

This also moves pub from third_party/pkg_tested into third_party/pkg.

The corresponding changes to the buildbot is here:
http://src.chromium.org/viewvc/chrome?revision=295555&view=revision

R=nweiz@google.com, whesse@google.com
BUG=

Review URL: https://codereview.chromium.org//1166093002.
2015-06-09 08:12:51 +02:00
pq
2c5e890fc7 Build fixes.
Long tail of fixes related to moving out of `pkg_tested`.

R=paulberry@google.com

Review URL: https://codereview.chromium.org//1162423002
2015-06-03 14:03:15 -07:00
Natalie Weizenbaum
3d8c06c4e9 Start pulling pub from its own repo.
Pub now lives in third_party/pkg/pub (it's in pkg to make it possible to import
using "package:" imports).

R=ricow@google.com, rnystrom@google.com

Review URL: https://codereview.chromium.org//1165473002
2015-05-29 14:50:18 -07:00
pquitslund@google.com
4c03f25626 Analyzer build updates to use analyzer_cli.
The associated change to add analyzer_cli to DEPS is here:
https://codereview.chromium.org/1093173004/

R=brianwilkerson@google.com, paulberry@google.com, ricow@google.com, rnystrom@google.com

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

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@45321 260f80e4-7a28-3924-810f-c04153c831b5
2015-04-21 20:59:31 +00:00
johnniwinther@google.com
1728034084 Make unittests work on Windows (again)
BUG=
R=floitsch@google.com

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

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@44680 260f80e4-7a28-3924-810f-c04153c831b5
2015-03-25 07:59:23 +00:00
rnystrom@google.com
6a3180b6bc Fix path to dart_style.
R=tjblasi@google.com

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

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@44109 260f80e4-7a28-3924-810f-c04153c831b5
2015-02-27 23:26:24 +00:00
rnystrom@google.com
1cae12abdc Switch dartfmt in the SDK to run dart_style.
BUG=https://code.google.com/p/dart/issues/detail?id=22400
R=pquitslund@google.com, ricow@google.com

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

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@44108 260f80e4-7a28-3924-810f-c04153c831b5
2015-02-27 23:00:35 +00:00
rnystrom@google.com
25f3f6733d Remove heap settings from pub scripts for SDK too.
r43821 only does it for the scripts used to run pub from within the source repo.

Also add support to the Windows shell script for passing VM flags to pub similar to
what dart2js and the bash pub scripts allow.

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

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

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@43925 260f80e4-7a28-3924-810f-c04153c831b5
2015-02-20 20:15:25 +00:00
nweiz@google.com
c8de060c54 Revert "Revert "Use native async/await support in pub.""
The VM now fully supports all the weird async stuff pub is doing, and all the
tests pass with native async/await!

Previous CLs were r43484 and r43483.

R=rnystrom@google.com

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

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@43897 260f80e4-7a28-3924-810f-c04153c831b5
2015-02-19 22:28:54 +00:00
nweiz@google.com
0c78059186 Remove --old_gen_heap_size from the pub scripts.
This was added because it used to be necessary to load dart2js, but
dart2js itself no longer uses it and it's limiting the VM's overall
ability to allocate memory.

R=rnystrom@google.com
BUG=22413

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

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@43821 260f80e4-7a28-3924-810f-c04153c831b5
2015-02-17 21:46:13 +00:00
nweiz@google.com
bbe0d9325c Revert "Use native async/await support in pub."
This caused SDK compilation issues that may have obscured pub test failures as
well.

R=rnystrom@google.com

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

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@43484 260f80e4-7a28-3924-810f-c04153c831b5
2015-02-04 21:15:44 +00:00
nweiz@google.com
1835c63d0a Use native async/await support in pub.
R=rnystrom@google.com

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

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@43483 260f80e4-7a28-3924-810f-c04153c831b5
2015-02-04 20:45:49 +00:00
floitsch@google.com
05c54a2ab3 dart2js: Remove heap-growth-flag that was passed to the VM.
R=iposva@google.com

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

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@43441 260f80e4-7a28-3924-810f-c04153c831b5
2015-02-03 22:24:47 +00:00
nweiz@google.com
c3ab5d7d7d Don't use pub_packages for running the Windows pub script.
R=rnystrom@google.com
TBR

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

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@42473 260f80e4-7a28-3924-810f-c04153c831b5
2014-12-18 19:41:00 +00:00
nweiz@google.com
80e6b5be2c Remove the http package from the repo.
This also does away with the pub_packages build target.

R=ahe@google.com, rnystrom@google.com, sigmund@google.com

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

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@42469 260f80e4-7a28-3924-810f-c04153c831b5
2014-12-18 19:04:04 +00:00
rnystrom@google.com
379ee11f6d Don't run async/await compiler from bin/pub.
R=nweiz@google.com

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

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@42051 260f80e4-7a28-3924-810f-c04153c831b5
2014-12-01 22:26:11 +00:00
alanknight@google.com
48ff3ccd27 Rename snapshot, other user-visible parts of docgen to dartdocgen.
BUG=
R=ricow@google.com

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

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

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

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@41573 260f80e4-7a28-3924-810f-c04153c831b5
2014-11-06 17:41:25 +00:00
alanknight@google.com
57aede2e75 "Reverting 41571"
BUG=

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

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@41572 260f80e4-7a28-3924-810f-c04153c831b5
2014-11-06 17:27:52 +00:00
alanknight@google.com
b9be0df603 Rename snapshot, other user-visible parts of docgen to dartdocgen.
BUG=
R=ricow@google.com

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

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

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@41571 260f80e4-7a28-3924-810f-c04153c831b5
2014-11-06 17:15:10 +00:00
alanknight@google.com
844c55185c "Reverting 41529"
BUG=

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

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@41531 260f80e4-7a28-3924-810f-c04153c831b5
2014-11-05 21:08:12 +00:00
alanknight@google.com
935058524b Rename snapshot, other user-visible parts of docgen to dartdocgen.
BUG=
R=ricow@google.com

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

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@41529 260f80e4-7a28-3924-810f-c04153c831b5
2014-11-05 20:23:08 +00:00
johnniwinther@google.com
09fbe78e71 Move dart2js from sdk/lib/_internal/compiler to pkg/compiler
BUG=

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

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@41514 260f80e4-7a28-3924-810f-c04153c831b5
2014-11-05 08:51:01 +00:00
johnniwinther@google.com
8637cfd322 Revert "Move dart2js from sdk/lib/_internal/compiler to pkg/compiler"
This reverts commit r41512.

BUG=

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

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@41513 260f80e4-7a28-3924-810f-c04153c831b5
2014-11-05 08:15:26 +00:00
johnniwinther@google.com
2b336e17f0 Move dart2js from sdk/lib/_internal/compiler to pkg/compiler
BUG=

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

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@41512 260f80e4-7a28-3924-810f-c04153c831b5
2014-11-05 07:09:04 +00:00
paulberry@google.com
f7db779401 Clean up scripts to run Java-based analyzer.
sdk/bin/dartanalyzer_developer is renamed sdk/bin/dartanalyzer_java
(replacing the old sdk/bin/dartanalyzer_java, which was unused), and
the test infrastructure is modified to use sdk/bin/dartanalyzer_java
to start the Java-based analyzer.

This frees up sdk/bin/dartanallyzer_developer to be the --host-checked
version of sdk/bin/dartanalyzer (paralleling what we do for
/sdk/bin/dart2js_developer).

R=ricow@google.com

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

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@41227 260f80e4-7a28-3924-810f-c04153c831b5
2014-10-21 13:02:37 +00:00
paulberry@google.com
b9116653bd Modify dartanalyzer scripts to look more like dart2js scripts.
The dartanalyzer script (and its Windows companion, dartanalyzer.bat)
now support:

- Executing from snapshot if present, otherwise from source (needed
  for buildbot tests).
- Passing in the appropriate package root when executind from source.
- Passing in VM tuning parameters to speed up analysis.
- Passing through extra DART_VM_OPTIONS to the VM.

Also, the old editor/tools/analyzer script has been removed and
the test infrastructure now uses sdk/bin/dartanalyzer, in the
same way that it works for dart2js.

R=ahe@google.com, johnniwinther@google.com, ricow@google.com

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

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@41150 260f80e4-7a28-3924-810f-c04153c831b5
2014-10-16 14:21:37 +00:00
rnystrom@google.com
80a6e8270c Get binstubs working on Windows.
Review URL: https://codereview.chromium.org//599343003

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@40667 260f80e4-7a28-3924-810f-c04153c831b5
2014-09-25 05:43:21 +00:00
rnystrom@google.com
17c1642748 Store the async-await compiled pub code directly in the repo.
R=nweiz@google.com

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

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@40041 260f80e4-7a28-3924-810f-c04153c831b5
2014-09-09 16:55:16 +00:00
rnystrom@google.com
dafe5c5ee4 Run async compiler in pub Windows batch files.
R=nweiz@google.com

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

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@39533 260f80e4-7a28-3924-810f-c04153c831b5
2014-08-25 21:56:13 +00:00
rnystrom@google.com
5167458d68 Skeleton code for running the forthcoming async/await compiler on pub.
R=nweiz@google.com

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

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@39530 260f80e4-7a28-3924-810f-c04153c831b5
2014-08-25 20:38:58 +00:00
ricow@google.com
46738fbe54 Increase vm memory for docgen.
We are running out of memory on the bots generating the docs, see e.g.
http://build.chromium.org/p/client.dart/builders/new_analyzer-linux-release-be/builds/9085/steps/build/logs/stdio

I have filed issue 20308 to track this

R=whesse@google.com

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

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@38825 260f80e4-7a28-3924-810f-c04153c831b5
2014-08-01 08:43:19 +00:00
alanknight@google.com
06ca853a8a Make docgen script work for directories with spaces in their names
BUG=
R=efortuna@google.com

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

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@38587 260f80e4-7a28-3924-810f-c04153c831b5
2014-07-25 19:58:24 +00:00
nweiz@google.com
d8663eeeda Create a separate packages directory for running pub.
Once issue 6943 is fixed, the http package will start using pub's
cross-platform library support to work with both dart:io and
dart:html. However, this package is also used *within* pub, so we need
a way for pub to import it that doesn't require it being run through
pub.

This change supports this by stripping out and dart:html imports from
the http package and creating a special packages directory only used
by pub that uses this modified version.

BUG=20068
R=floitsch@google.com

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

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@38255 260f80e4-7a28-3924-810f-c04153c831b5
2014-07-15 20:28:08 +00:00
whesse@google.com
dce5bc1322 Allow space in path in some dartanalyzer wrappers.
BUG=dartbug.com/17272
R=danrubel@google.com, ricow@google.com

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

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@36490 260f80e4-7a28-3924-810f-c04153c831b5
2014-05-22 12:31:13 +00:00
ricow@google.com
d0e8a0b8a6 Add dart-sdk argument when calling the dart based analyzer in the created sdk
I am not even sure why we need to pass in this argument, but the tool requires this or it will exit with "Usage: dartanalyzer: no Dart SDK found."

This is a fix so that we can get a release out, I will leave it to the analyzer people to figure out if this is needed after all.

R=kasperl@google.com

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

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@35889 260f80e4-7a28-3924-810f-c04153c831b5
2014-05-08 08:18:48 +00:00
danrubel@google.com
521aef0d96 Replace Java based analyzer with Dart based analyzer when building SDK
* include analyzer dart files and snapshot in SDK
* modified bin/dartanalyzer to call dart based analyzer
* removed jar files in utils directory

BUG=dartbug.com/18289
R=ricow@google.com, scheglov@google.com

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

git-svn-id: https://dart.googlecode.com/svn/branches/bleeding_edge/dart@35604 260f80e4-7a28-3924-810f-c04153c831b5
2014-04-30 23:09:14 +00:00