Skip to content

Add/resident compiler #662

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

Merged
merged 6 commits into from
Jun 29, 2015
Merged

Conversation

odersky
Copy link
Contributor

@odersky odersky commented Jun 15, 2015

This adds a resident compiler command. For usage information, see doc comment in Resident.scala.
We'll later refine this to become a general compile server.

@odersky
Copy link
Contributor Author

odersky commented Jun 26, 2015

I'd like to get this in before it starts to rot. Any objections?

odersky added 6 commits June 29, 2015 08:56
 - Make parts more reusable
 - Introduce hook "sourcesRequired" that controls whether
   no sources on the command line give a help message.
Not needed right now, but will be useful later.
Reporters are part of the context which is passed into a compiler run. Passing them
as a separate option is an unnecessary complication.
@odersky odersky force-pushed the add/resident-compiler branch from e8bdccc to 840aa93 Compare June 29, 2015 07:23
@odersky
Copy link
Contributor Author

odersky commented Jun 29, 2015

Rebased to current master

odersky added a commit that referenced this pull request Jun 29, 2015
@odersky odersky merged commit bf6d805 into scala:master Jun 29, 2015
@DarkDimius
Copy link
Contributor

While being rebased this PR removed most of changes that were made in #691. I'll make a PR to bring them back

@xeno-by
Copy link

xeno-by commented Jul 1, 2015

What's the main benefit of having this feature? Is it cutting down the startup time of the compiler or there's more? Also, how does this compare with sbt-server?

@allanrenucci allanrenucci deleted the add/resident-compiler branch December 14, 2017 16:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants