-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
pub build produces a huge amount of output that seems redundant #15509
Comments
Added Area-Polymer, Triaged labels. |
The first two messages from the path package seem to be fixed in the repo and we just need to update dependencies to use the latest version of 'path'. I'm reassigning the area to Dart2js as most of the messages are 'hints' from Dart2js from uses of dart:mirrors that already have a @MirrorUsed annotation. Removed Area-Polymer label. |
This comment was originally written by carlj...@gmail.com Is there a temporary fix one can do to get past this? |
This comment was originally written by vladimir.matusev...@gmail.com I have the same issue with latest Dart Editor in project that using polymer. |
We have improved our error-reporting. Is this still a problem? Added NeedsInfo label. |
This comment was originally written by garrick.blackberr...@gmail.com I wouldn't describe the output as a problem anymore. Not for me. I do wonder why methods retained by dart:mirrors is a warning and not information. |
As far as dart2js is concerned, the message is a "hint" not a "warning". |
cc @johnniwinther. |
This comment was originally written by @zoechi Added AssumedStale label. |
This issue was originally filed by @zoechi
What steps will reproduce the problem?
1.
run pub build on a polymer project
2.
I had the following messages already before the last updates (since pub build is usable).
Building polymer_elements_examples...........
[Warning in Dart2JS on polymer_elements_examples|web/polymer_file.dart]:
packages/path/path.dart:49:23: Hint: Can't find 'dart.io.Directory.current' in the current library.
Did you forget to add an import?
@MirrorsUsed(targets: 'dart.dom.html.window, '
^^^^^^^^^^^^^^^^^^^^^^^^^
[Warning in Dart2JS on polymer_elements_examples|web/polymer_file.dart]:
packages/path/path.dart:49:23: Hint: Can't find 'dart.io.Platform.operatingSystem' in the current library.
Did you forget to add an import?
@MirrorsUsed(targets: 'dart.dom.html.window, '
^^^^^^^^^^^^^^^^^^^^^^^^^
[Dart2JS on polymer_elements_examples|web/polymer_file.dart]:
packages/polymer/polymer.dart:50:1: Info: Import of 'dart:mirrors'.
import 'dart:mirrors';
^^^^^^^^^^^^^^^^^^^^^^
[Dart2JS on polymer_elements_examples|web/polymer_file.dart]:
packages/observe/src/observable.dart:16:1: Info: Import of 'dart:mirrors'.
import 'dart:mirrors';
^^^^^^^^^^^^^^^^^^^^^^
[Dart2JS on polymer_elements_examples|web/polymer_file.dart]:
packages/observe/src/path_observer.dart:10:1: Info: Import of 'dart:mirrors'.
import 'dart:mirrors';
^^^^^^^^^^^^^^^^^^^^^^
[Dart2JS on polymer_elements_examples|web/polymer_file.dart]:
packages/path/path.dart:52:1: Info: Import of 'dart:mirrors'.
import 'dart:mirrors';
^^^^^^^^^^^^^^^^^^^^^^
[Dart2JS on polymer_elements_examples|web/polymer_file.dart]:
packages/polymer_expressions/eval.dart:12:1: Info: Import of 'dart:mirrors'.
import 'dart:mirrors';
^^^^^^^^^^^^^^^^^^^^^^
[Dart2JS on polymer_elements_examples|web/polymer_file.dart]:
packages/polymer_expressions/src/mirrors.dart:11:1: Info: Import of 'dart:mirrors'.
import 'dart:mirrors';
^^^^^^^^^^^^^^^^^^^^^^
[Dart2JS on polymer_elements_examples|web/polymer_file.dart]:
packages/polymer/deserialize.dart:8:1: Info: Import of 'dart:mirrors'.
import 'dart:mirrors' show TypeMirror;
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[Dart2JS on polymer_elements_examples|web/polymer_file.dart]:
packages/polymer/src/reflected_type.dart:11:1: Info: Import of 'dart:mirrors'.
import 'dart:mirrors';
^^^^^^^^^^^^^^^^^^^^^^
These are new
[Dart2JS on polymer_elements_examples|web/polymer_file.dart]:
../../../3rdparty/google/darteditor/dart-sdk/lib/html/dart2js/html_dart2js.dart:26708:1: Warning: 'Window' doesn't implement 'clearInterval'.
Try adding an implementation of 'clearInterval'.
class Window extends EventTarget implements WindowBase, _WindowTimers, WindowBase64 native "Window,DOMWindow" {
^^^^^
[Warning in Dart2JS on polymer_elements_examples|web/polymer_file.dart]:
../../../3rdparty/google/darteditor/dart-sdk/lib/html/dart2js/html_dart2js.dart:30378:8: Info: This is the method declaration.
void clearInterval(int handle);
^^^^^^^^^^^^^
similar with
instead of clearInterval
These are the one where I'm sure I have something to do.
[Warning in Dart2JS on polymer_elements_examples|web/polymer_selector.html_bootstrap.dart]:
packages/polymer/src/instance.dart:388:31: Hint: Using 'MirrorSystem.getName' may lead to unnecessarily large generated code.
Try adding '@MirrorsUsed(...)' as described at https://goo.gl/Akrrog.
attributes[MirrorSystem.getName(name)] = serializedValue;
3.
What is the expected output? What do you see instead?
Only relevant messages should be logged.
What version of the product are you using? On what operating system?
Dart VM version: 1.0.3.0_r30939 (Fri Dec 6 03:48:53 2013) on "linux_x64"
Please provide any additional information below.
This is a project with about 20 simple pages each using a few polymer elements.
pub build produces 1700+ lines of output.
This huge amount of output is horrible and I don't know if I can do something to fix it.
Some clarification is very much appreciated.
Thanks
Günter
The text was updated successfully, but these errors were encountered: