-
-
Notifications
You must be signed in to change notification settings - Fork 652
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
Add utility to extract macro classes from a classpath. #8201
Conversation
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.
The CI failures seem unrelated to this change.
import java.io.File | ||
|
||
class FindMacrosSuite extends FunSuite with DiffAssertions { | ||
test("expect") { |
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.
I think it might be worth using ignore
for this test since it downloads quite a lot of jars from Maven Central and I estimate the FindMacros
class won't change so frequently.
Sorry: looks like you were struck by #8127. |
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.
Thanks gang! Before landing this, it would be good to see the dependent PR that will consume it (or a sketch of a design), so that we can know how it will fit into the workflow we're going for.
Closing since #8145 got closed as well. |
Problem
To build a native image for the Scala compiler, we need to know the names of the classes on the compile classpath that define Scala macros.
Solution
This PR introduces a new
find-macros
command-line tool that takes a classpath and prints out the names of the classes that define Scala macros. The tool finds class names by looking at the Scala signatures produced by the Scala compiler.