Skip to content

Print terse stack traces for uncaught errors #1139

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

Merged
merged 2 commits into from
Mar 9, 2018
Merged

Conversation

natebosch
Copy link
Member

Closes #1135

Exceptions from Builders should be caught by a runZoned call. The only
exceptions we expect to escape the Isolate are bugs within build_runner.

  • Add a message saying that an issue should be file.
  • Parse the trace and print the Trace.terse version

Closes #1135

Exceptions from Builders should be caught by a `runZoned` call. The only
exceptions we expect to escape the Isolate are bugs within build_runner.

- Add a message saying that an issue should be file.
- Parse the trace and print the Trace.terse version
@googlebot googlebot added the cla: yes Google is happy with the PR contributors label Mar 9, 2018
@natebosch
Copy link
Member Author

The error from #1134 now looks like:



You have hit a bug in build_runner
Please file an issue with reproduction steps at https://github.com/dart-lang/build/issues


FormatException: Invalid character (at character 13)
e2e_example/#strange_file_name
            ^

dart:core                                                  new _Uri
package:build/src/asset/id.dart 102:13                     AssetId.uri
package:build_runner/src/watcher/graph_watcher.dart 84:54  PackageGraphWatcher._watch.<fn>.<fn>

@matanlurey matanlurey merged commit d34b262 into master Mar 9, 2018
@matanlurey matanlurey deleted the uncaught-errors branch March 9, 2018 21:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes Google is happy with the PR contributors
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants