Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

dartdevc_patch_sdk build target fails #42825

Closed
whesse opened this issue Jul 24, 2020 · 2 comments
Closed

dartdevc_patch_sdk build target fails #42825

whesse opened this issue Jul 24, 2020 · 2 comments
Labels
area-web Use area-web for Dart web related issues, including the DDC and dart2js compilers and JS interop. dev-compiler-build P3 A lower priority bug or feature request web-dev-compiler

Comments

@whesse
Copy link
Contributor

whesse commented Jul 24, 2020

The build target dartdevc_patch_sdk fails to build.
No build target that we commonly use includes that target, maybe it is unused.
The build targets create_sdk, runtime, and most do not build it.
Running build.py with no target specified runs all targets in our build files, and so runs it and fails.
Of course, nobody should be running build.py with no target, but people do by accident.

The build fails with:
sdk$ tools/build.py -mrelease dartdevc_patch_sdk

FAILED: gen/utils/dartdevc/patched_sdk/version
python ../../build/gn_run_binary.py compiled_action /src/dart/sdk/tools/sdks/dart-sdk/bin/dart -Dsdk_hash=4c74ebb367 --dfe=/src/dart/sdk/tools/sdks/dart-sdk/bin/snapshots/kernel-service.dart.snapshot /src/dart/sdk/pkg/dev_compiler/tool/patch_sdk.dart --libraries /src/dart/sdk/sdk/lib/libraries.json --target dartdevc --out /src/dart/sdk/out/ReleaseX64/gen/utils/dartdevc/patched_sdk/
Command failed: /src/dart/sdk/tools/sdks/dart-sdk/bin/dart -Dsdk_hash=4c74ebb367 --dfe=/src/dart/sdk/tools/sdks/dart-sdk/bin/snapshots/kernel-service.dart.snapshot /src/dart/sdk/pkg/dev_compiler/tool/patch_sdk.dart --libraries /src/dart/sdk/sdk/lib/libraries.json --target dartdevc --out /src/dart/sdk/out/ReleaseX64/gen/utils/dartdevc/patched_sdk/
output: Unhandled exception:
Invalid argument(s): Content produced diagnostics when parsed
#0 parseString (package:analyzer/dart/analysis/utilities.dart:134:5)
#1 _parseString (file:///src/dart/sdk/pkg/dev_compiler/tool/patch_sdk.dart:477:10)
#2 main (file:///src/dart/sdk/pkg/dev_compiler/tool/patch_sdk.dart:64:14)
#3 _startIsolate. (dart:isolate-patch/isolate_patch.dart:299:32)
#4 _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:168:12)

@whesse whesse added P3 A lower priority bug or feature request dev-compiler-build labels Jul 24, 2020
@vsmenon vsmenon added area-web Use area-web for Dart web related issues, including the DDC and dart2js compilers and JS interop. web-dev-compiler labels Jul 27, 2020
@vsmenon
Copy link
Member

vsmenon commented Jul 27, 2020

@nshahan @sigmundch - is this dead code now with ddk?

@nshahan
Copy link
Contributor

nshahan commented Jul 27, 2020

We are still using that script in that target to create a patched SDK to check for analysis errors but AFAIK we are not using that build target. I'll talk with @sigmundch and make sure we do some cleanup.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-web Use area-web for Dart web related issues, including the DDC and dart2js compilers and JS interop. dev-compiler-build P3 A lower priority bug or feature request web-dev-compiler
Projects
None yet
Development

No branches or pull requests

3 participants