Skip to content
This repository was archived by the owner on Jul 20, 2018. It is now read-only.

Support lazy compilation in barback #26

Closed
DartBot opened this issue Jun 5, 2015 · 4 comments
Closed

Support lazy compilation in barback #26

DartBot opened this issue Jun 5, 2015 · 4 comments
Labels

Comments

@DartBot
Copy link

DartBot commented Jun 5, 2015

<img src="https://avatars.githubusercontent.com/u/188?v=3" align="left" width="96" height="96"hspace="10"> Issue by nex3
Originally opened as dart-lang/sdk#16077


Barback should have the ability for a transformer to declare that it should be run lazily -- that is, only if its assets are actually requested, either by another transformer or by the user via [getAssetById] or [getAllAssets].

In order to do this, the transformer will need to be able to run enough to say which assets in can produce, without actually producing the assets.

@DartBot
Copy link
Author

DartBot commented Jun 5, 2015

<img src="https://avatars.githubusercontent.com/u/188?v=3" align="left" width="48" height="48"hspace="10"> Comment by nex3


Marked this as blocking dart-lang/pub#815.

@DartBot
Copy link
Author

DartBot commented Jun 5, 2015

<img src="https://avatars.githubusercontent.com/u/188?v=3" align="left" width="48" height="48"hspace="10"> Comment by nex3


Ideally, this would include the ability for a lazy transformer's outputs to be consumed by another lazy (or lazy-compatible) transformer without ever materializing them. They could also be rejected by a transformer's isPrimary even if it isn't explicitly lazy-compatible -- it would just have to not try to access the asset's contents.

In order to just get issue dart-lang/pub#815 solved, though, it should be sufficient to just have lazy assets be materialized if any other transformer has anything to do with them. This means that a lazy transformer will effectively only work if it's in the last phase of a cascade.

@DartBot
Copy link
Author

DartBot commented Jun 5, 2015

<img src="https://avatars.githubusercontent.com/u/3276024?v=3" align="left" width="48" height="48"hspace="10"> Comment by anders-sandholm


Removed Library-Barback label.
Added Pkg-Barback label.

@DartBot
Copy link
Author

DartBot commented Jun 5, 2015

<img src="https://avatars.githubusercontent.com/u/188?v=3" align="left" width="48" height="48"hspace="10"> Comment by nex3


Fixed by dart-lang/sdk@01cadd4.


Added Fixed label.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Development

No branches or pull requests

2 participants