Skip to content

Commit a75845f

Browse files
authored
Migrate from manual build script to copy builder (#2226)
- Remove the `tool/build.dart` file with hardcoded builders. - Update to the latest `build_web_compilers`. - Add a copy builder to move the `.js` files from `web/` to `lib/` and into the source tree. - Change to use `-O4` dart2js arg. - Update tests to expect `.js` for `.browser_test.dart` files.
1 parent fc3d135 commit a75845f

File tree

13 files changed

+6719
-6902
lines changed

13 files changed

+6719
-6902
lines changed

.travis.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -21,9 +21,9 @@ jobs:
2121
env: PKGS="build build_config build_daemon build_modules build_resolvers build_runner build_runner_core build_test build_vm_compilers build_web_compilers example scratch_space"
2222
script: ./tool/travis.sh dartfmt dartanalyzer_0
2323
- stage: analyze_and_format
24-
name: "SDK: 2.1.0; PKGS: build, build_daemon, build_resolvers, build_runner, build_runner_core; TASKS: `dartanalyzer --fatal-warnings .`"
24+
name: "SDK: 2.1.0; PKGS: build, build_daemon, build_resolvers, build_runner_core; TASKS: `dartanalyzer --fatal-warnings .`"
2525
dart: "2.1.0"
26-
env: PKGS="build build_daemon build_resolvers build_runner build_runner_core"
26+
env: PKGS="build build_daemon build_resolvers build_runner_core"
2727
script: ./tool/travis.sh dartanalyzer_1
2828
- stage: analyze_and_format
2929
name: "SDK: 2.2.0; PKGS: build_config, build_test; TASKS: `dartanalyzer --fatal-warnings .`"

build_runner/build.yaml

Lines changed: 25 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,25 @@
1+
targets:
2+
$default:
3+
builders:
4+
build_web_compilers:entrypoint:
5+
options:
6+
compiler: dart2js
7+
dart2js_args:
8+
- -O4
9+
generate_for:
10+
- web/graph_viz_main.dart
11+
- web/hot_reload_client.dart
12+
build_runner:client_js_copy_builder:
13+
enabled: true
14+
builders:
15+
client_js_copy_builder:
16+
import: "tool/builders.dart"
17+
builder_factories:
18+
- copyCompiledJs
19+
build_extensions:
20+
web/graph_viz_main.dart.js:
21+
- lib/src/server/graph_viz_main.dart.js
22+
web/hot_reload_client.dart.js:
23+
- lib/src/server/build_updates_client/hot_reload_client.dart.js
24+
auto_apply: none
25+
build_to: source

0 commit comments

Comments
 (0)