Skip to content

Commit 69249df

Browse files
sigmundchCommit Bot
authored and
Commit Bot
committed
[web] remove top-level scripts for dartdevc and dart2js.
This completes the breaking change that was previously announced in #46100 Change-Id: Ie81665e7932fe00a8c1fee9f67ee8294cd4c89e9 Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/248342 Reviewed-by: Joshua Litt <[email protected]> Commit-Queue: Sigmund Cherem <[email protected]> Reviewed-by: Nicholas Shahan <[email protected]>
1 parent 3099911 commit 69249df

File tree

7 files changed

+14
-229
lines changed

7 files changed

+14
-229
lines changed

CHANGELOG.md

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -140,6 +140,13 @@ them, you must set the lower bound on the SDK constraint for your package to
140140
141141
### Tools
142142
143+
#### Dart command line
144+
145+
- **Breaking change** [#46100](https://github.com/dart-lang/sdk/issues/46100):
146+
The standalone `dart2js` and `dartdevc` tools have been removed as previously
147+
announced. `dart2js` is replaced by the `dart compile js` command, `dartdevc`
148+
is no longer exposed as a command-line tool.
149+
143150
#### Linter
144151
145152
Updated the Linter to `1.25.0`, which includes changes that

pkg/dev_compiler/tool/ddb

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -174,7 +174,7 @@ void main(List<String> args) async {
174174
mode: ProcessStartMode.inheritStdio, environment: environment);
175175
}
176176

177-
Future<void> runDdc(String command, List<String> args) async {
177+
Future<void> runDdc(List<String> args) async {
178178
if (debug) {
179179
// Use unbuilt script. This only works from a source checkout.
180180
var vmServicePort = options.wasParsed('vm-service-port')
@@ -188,14 +188,14 @@ void main(List<String> args) async {
188188
'--pause-isolates-on-start',
189189
],
190190
'--enable-asserts',
191-
p.join(ddcPath, 'bin', '$command.dart')
191+
p.join(ddcPath, 'bin', 'dartdevc.dart')
192192
]);
193-
command = dartBinary;
194193
} else {
195194
// Use built snapshot.
196-
command = p.join(dartSdk, 'bin', command);
195+
args.insertAll(
196+
0, [p.join(dartSdk, 'bin', 'snapshots', 'dartdevc.dart.snapshot')]);
197197
}
198-
var process = await startProcess('DDC', command, args, <String, String>{
198+
var process = await startProcess('DDC', dartBinary, args, <String, String>{
199199
if (options['compile-vm-options'] != null)
200200
'DART_VM_OPTIONS': options['compile-vm-options'] as String
201201
});
@@ -249,7 +249,7 @@ void main(List<String> args) async {
249249
out,
250250
entry
251251
];
252-
await runDdc('dartdevc', ddcArgs);
252+
await runDdc(ddcArgs);
253253
}
254254

255255
if (run) {

sdk/BUILD.gn

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -90,11 +90,7 @@ declare_args() {
9090
# Scripts that go under bin/
9191
_platform_sdk_scripts = [ "dartanalyzer" ]
9292

93-
_full_sdk_scripts = [
94-
"dart2js",
95-
"dartanalyzer",
96-
"dartdevc",
97-
]
93+
_full_sdk_scripts = [ "dartanalyzer" ]
9894

9995
# Snapshots that go under bin/snapshots
10096
_platform_sdk_snapshots = [

sdk/bin/dart2js_sdk

Lines changed: 0 additions & 58 deletions
This file was deleted.

sdk/bin/dart2js_sdk.bat

Lines changed: 0 additions & 57 deletions
This file was deleted.

sdk/bin/dartdevc_sdk

Lines changed: 0 additions & 41 deletions
This file was deleted.

sdk/bin/dartdevc_sdk.bat

Lines changed: 0 additions & 62 deletions
This file was deleted.

0 commit comments

Comments
 (0)