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

allow dart2js compile multiple source files at once #10711

Closed
DartBot opened this issue May 17, 2013 · 7 comments
Closed

allow dart2js compile multiple source files at once #10711

DartBot opened this issue May 17, 2013 · 7 comments
Labels
closed-obsolete Closed as the reported issue is no longer relevant P2 A bug or feature request we're likely to work on type-enhancement A request for a change that isn't a bug web-dart2js

Comments

@DartBot
Copy link

DartBot commented May 17, 2013

This issue was originally filed by email...@daniel-zwicker.de


We have a about 84 dart files in our project with out the libs inside the package folders. There from we have to use dart2js for 15 of them. As a compile to js takes about 10 sec we have tried to speed up this.

We edit the dart2js.dart and changed the 'api.compile(uri....' call to a hard coded URI and double it. So one call to dart2js compile 2 files at once (Yes the out.js will be overwritten).

We mesure the following times (disabled snapshot use of utils_wrapper.dart.snapshot):

file1: 0m8.869s
file2: 0m8.299s

together: 0m12.732s

So there is a speed up of around 50% for the second file. We believe this is because the compiler is loaded.

So wouldn't it be possible to allow to define a directory which dart2js compile all *.dart recursive? This would speed up our builds extreme.

@anders-sandholm
Copy link
Contributor

Removed Type-Defect label.
Added Type-Enhancement, Area-Dart2JS, Triaged labels.

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

@kevmoo kevmoo added P2 A bug or feature request we're likely to work on type-enhancement A request for a change that isn't a bug and removed triaged labels Feb 29, 2016
@matanlurey matanlurey added the closed-obsolete Closed as the reported issue is no longer relevant label Jun 22, 2018
@matanlurey
Copy link
Contributor

We require a build system now

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
closed-obsolete Closed as the reported issue is no longer relevant P2 A bug or feature request we're likely to work on type-enhancement A request for a change that isn't a bug web-dart2js
Projects
None yet
Development

No branches or pull requests

5 participants