-
Notifications
You must be signed in to change notification settings - Fork 31
usejavacp leaks a lot of dependency into the benchmarked project #46
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
Comments
Of those, we should only need |
I've gone for option A.The |
@retronym Would you accept a PR adding xml to the classpath? It's necessary to compile some projects in the community, and may prove useful to others. |
I'd expect that to work by adding it as a dependency, as we do for scala-asm in the
|
See https://github.com/scala/compiler-benchmark/pull/60/files for an unrelated fix that solves this issue too. |
Uh oh!
There was an error while loading. Please reload this page.
I noticed this while trying to benchmark akka-actors, and the version of Typesafe Config used by Akka is newer than what
infrastructure
depends on, so compilation fails. It seems dangerous to include so many dependencies in every benchmark.Is there a good reason to set
settings.usejavacp.value = true
? (I guess there must be, since it's explicitly added).The text was updated successfully, but these errors were encountered: