Skip to content
New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom

dart2js in darteditor failed without any sensible error message #3062

Closed
peter-ahe-google opened this issue May 15, 2012 · 1 comment
Closed

dart2js in darteditor failed without any sensible error message #3062

peter-ahe-google opened this issue May 15, 2012 · 1 comment
Assignees
@peter-ahe-google
Labels
web-dart2js

Comments

@peter-ahe-google
Copy link
Contributor

The following was reported on misc@dartlang:

dart2js in darteditor failed without any sensible error message
@peter-ahe-google
Copy link
Contributor Author

I have submitted two changes to fix this problem. Since I don't have much information to go on, I cannot guarantee that this solved the problem reported on misc@dartlang. But these two changes should address all known issues with nonsensical error messages.

The changes are:

http://code.google.com/p/dart/source/detail?r=7660 (ensure that all diagnostics have location information)

http://code.google.com/p/dart/source/detail?r=7661 (attempt to diagnose what method was being compiled if the compiler crashes)

Added Fixed label.

@peter-ahe-google peter-ahe-google self-assigned this May 15, 2012
dart-bot pushed a commit that referenced this issue Sep 13, 2021
@sigurdm
Roll pub
a704116 
New commits in this version:

git -C third_party/pkg/pub log --oneline cd7a43f2109f7e5eb22e73c7f4e15d25fd57598e..d95c5713dda518ed53ada70e00789e6aadbfbe48
d95c5713 (HEAD, origin/master, origin/HEAD) Remove duplicate global invocation on Windows (#3055)
4c5198df master->main (#3101)
e793fd52 More tool/test.dart tweaks (#3097)
1b228edd Report retracted packages (#3093)
4fedb6c5 Tweak strict dependencies error message for `_validateBenchmarkTestTool` (#3087)
e608ab6e Improve test script (#3091)
abf702c4 Detect potential leaks in `dart pub publish`. (#3049)
9941c1f9 Fix broken simplification of prerelease constraints (#3078)
f0cdfa27 format (#3086)
5a1242c6 Fix unicode terminal detection windows (#2933)
58e2296d Dart format (#3084)
1426601c use incremental compilation in the tool/test.dart script (#3075)
9954f851 Fix a typo (#3062)
44489b31 Use relative import for path.dart (#2959)
77702ab1 Always precompile scripts before running them (#3074)

Change-Id: I913ab9e3b5bc7181d483a2de96ec4ad917028b75
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/213262
Reviewed-by: Jonas Jensen <jonasfj@google.com>
Commit-Queue: Sigurd Meldgaard <sigurdm@google.com>
This issue was closed.
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees

@peter-ahe-google peter-ahe-google

Labels
web-dart2js
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@peter-ahe-google