-
Notifications
You must be signed in to change notification settings - Fork 1.7k
IsolateSpawnException should expose more metadata #17501
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
Comments
Added Area-Library label. |
Marked this as blocking #17200. |
Er, blocking dart-lang/pub#896 |
This is not information that we can generally have. If dart2js fails to spawn an isolate, it's either because the JS files didn't exist or it had an error. I think each platform should have a best-effort approach to reporting the errors, but it's not something we can generally add as fields of a the error. |
I'm fine if there are fields that are |
Reassigning to VM. This kind of information would need to come from the VM/embedder. |
dart-lang/pub#896 is closed, so I assume we can close this. |
Currently, IsolateSpawnException only exposes a [toString] method (which is poorly formatted -- see issue #12689). This makes it very difficult to report isolate errors, such as those from "pub serve", in a user-friendly manner.
For an error caused by a library failing to load, it would be useful to have, in order of descending usefulness:
* The URI of the library that failed to load.
The text was updated successfully, but these errors were encountered: