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

Remove greetings printing. #36

Merged
merged 1 commit into from Sep 27, 2011
Merged

Remove greetings printing. #36

merged 1 commit into from Sep 27, 2011

Conversation

gkossakowski
Copy link

They were used mainly to inform people
that compilation is really slow. It got
a lot better so we can safely remove it.

Change-Id: I231c0892d90e48f3eb6269bb67cb00c432e92b0b
Signed-off-by: Grzegorz Kossakowski grzegorz.kossakowski@gmail.com

They were used mainly to inform people
that compilation is really slow. It got
a lot better so we can safely remove it.

Change-Id: I231c0892d90e48f3eb6269bb67cb00c432e92b0b
Signed-off-by: Grzegorz Kossakowski <grzegorz.kossakowski@gmail.com>
@anovstrup
Copy link

Looks fine. Can we further improve compilation speed by compiling ScalaLibrary to a gwtar?

@gkossakowski
Copy link
Author

I don't know details behind gwtar. I guess @stephenh knows more about it.

Nonetheless, we can further improve compilation speed by switching from text protobuf messages to binary ones. See scalagwt/scalagwt-gwt#23.

gkossakowski added a commit that referenced this pull request Sep 27, 2011
@gkossakowski gkossakowski merged commit 2dd4a8c into scalagwt:master Sep 27, 2011
@stephenh
Copy link

Per the gwtar, you just invoke GWT's CompileModule class with the name of the module you want compiled. As long as you've run the scala compiler first for it to find the jribble files, it should work fine:

https://github.com/stephenh/scalagwt-scala/blob/embed/build.xml#L559

@gkossakowski
Copy link
Author

The question is if packaging scala library as gwtar would make compilation faster.

@stephenh
Copy link

Ah, sorry. Yeah, I'm sure it would because then GWT doesn't need to parse the jribble files at all. It will only have to parse the .jribble files for the user's app itself. (My embedding spike used this and it was pretty quick--no multi-minute first compile to populate the unit cache.)

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.

None yet

3 participants