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

dart2js not able to analyze code with "dart-ext:XXX" imports #10616

Closed
mkustermann opened this issue May 13, 2013 · 9 comments
Closed

dart2js not able to analyze code with "dart-ext:XXX" imports #10616

mkustermann opened this issue May 13, 2013 · 9 comments
Assignees

Comments

@mkustermann
Copy link
Member

When using native C extensions dart2js is not able to analyze an application. Instead it will abort as soon as it sees a "dart-ext:XXX" import.

This prevents us from using dart2js as an analyzer. Maybe it should just print out a warning if dart2js is in "analyze only" mode.

~/golem4 $ ~/dart/sdk/bin/dart2js --analyze-all --categories=all tools/scheduler.dart
tools/sha256_ext/sha256_ext.dart:5:8: Illegal argument(s): Unknown scheme in uri 'dart-ext:sha256_ext'
import "dart-ext:sha256_ext";
       ^^^^^^^^^^^^^^^^^^^^^
Error: Compilation failed.

@kasperl
Copy link

kasperl commented May 23, 2013

Added this to the Later milestone.

@kasperl
Copy link

kasperl commented May 23, 2013

Added TriageForM5 label.

@kasperl
Copy link

kasperl commented May 28, 2013

Removed TriageForM5 label.

@kasperl
Copy link

kasperl commented Jul 10, 2014

Removed this from the Later milestone.
Added Oldschool-Milestone-Later label.

@kasperl
Copy link

kasperl commented Aug 4, 2014

Removed Oldschool-Milestone-Later label.

@DartBot
Copy link

DartBot commented Oct 22, 2014

This comment was originally written by osstek...@gmail.com


Docgen and other programs still rendered un-useable due to this issue. Could/should have ignored the statement, or issued a warning, or maybe treat it as a comment and moved on temporarily until a permanent solution. A year and a half and still no patch? really?

@johnniwinther
Copy link
Member

Set owner to @johnniwinther.

@johnniwinther
Copy link
Member

Added Started label.

@johnniwinther
Copy link
Member

Fixed by https://codereview.chromium.org/745533002/


Added Fixed label.

This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants