Skip to content

make pub-build robust to accept transformer code with bugs or errors #14206

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

Closed
sigmundch opened this issue Oct 18, 2013 · 7 comments
Closed

make pub-build robust to accept transformer code with bugs or errors #14206

sigmundch opened this issue Oct 18, 2013 · 7 comments
Labels
closed-not-planned Closed as we don't intend to take action on the reported issue

Comments

@sigmundch
Copy link
Member

I saw this error as a result of issue #14205:

Because pub tries to load 'polymer.dart', the vm gives this error and pub-build fails immediately:

IsolateSpawnException: 'The built-in library 'dart:html' is not available on the stand-alone VM.
'http://127.0.0.1:45866/packages/polymer/polymer.dart': Error: line 44 pos 1: library handler failed
import 'dart:html';
^
'file:///tmp/pub_DuxgFe/runInIsolate.dart': Error: line 1 pos 1: library handler failed
import "http://127.0.0.1:45866/packages/polymer/polymer.dart";import 'dart:async';
^'

@nex3
Copy link
Member

nex3 commented Oct 18, 2013

Making pub behave in a way that's any better than crash here would require infrastructure for reloading transformers after the initialization, which isn't planned for the near future.


Removed this from the M8 milestone.
Added this to the Later milestone.

@munificent
Copy link
Member

Removed Priority-Unassigned label.
Added Priority-Medium label.

@nex3
Copy link
Member

nex3 commented Jun 26, 2014

Marked this as being blocked by #19163.

@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.

@munificent
Copy link
Member

I'm going to close this because we don't have any plans to change this behavior any time soon. If we do eventually support reloading transformers, this will get fixed implicitly by that.


Added NotPlanned label.

@DartBot
Copy link

DartBot commented Jun 5, 2015

This issue has been moved to dart-lang/pub#675.

This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
closed-not-planned Closed as we don't intend to take action on the reported issue
Projects
None yet
Development

No branches or pull requests

5 participants