-
Notifications
You must be signed in to change notification settings - Fork 3.1k
SI-874 JSR-223 compliance for the interpreter #2153
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
Conversation
|
review @paulp |
|
PLS SYNCH |
|
(kitty-note-to-self: ignore 13955366) |
|
PLS REBUILD ALL |
|
(kitty-note-to-self: ignore 14186715) |
|
BUILDLOG? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this needs a copyright/author header
|
This is great stuff, thanks @rjolly! (I apologize for not having gotten to this faster.) I'd like to see it split up in several commits, so that if something breaks (one never knows) we can revert more selectively. How about the following breakdown:
Also, review by @jsuereth for build/osgi-related changes. |
Part 1 : reflect.io improvements
Part 2 : jarlister (but not used in build yet)
Part 3 : use jarlister in build
Part 4 : actual JSR-223 implementation
|
subsumed by #2206 |
Usage:
scala/bin/jarlister -o scala-library-listed.jar scala/lib/scala-library.jar
jrunscript -classpath scala/lib/scala-compiler.jar:scala/lib/scala-reflect.jar:scala-library-listed.jar -l scala