-
Notifications
You must be signed in to change notification settings - Fork 928
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
Automatically choose proper compiler bridge for dotty #2902
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.
I'll defer to @eed3si9n, as he understands how compiler bridges are meant to be used and customized.
The single test failure appears to be spurious and is unrelated to this PR: [info] Could not create file /tmp/sbt_ecf6163e/java-mixed/target/streams/$global/ivySbt/$global/streams/outjava.io.IOException: No such file or directory |
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's generally ok, but could you add release notes in https://github.com/sbt/sbt/tree/0.13/notes/0.13.14 and clarify in there that Dotty support in sbt is an experimental feature that's controlled by EPFL, plz?
Release notes added! |
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!
|
Oops, that commit should not be there |
Before this commit, using dotty in your sbt project required to add: scalaCompilerBridgeSource := ("ch.epfl.lamp" % "dotty-sbt-bridge" % scalaVersion.value % "component").sources() in your build.sbt. We might as well automatically do this, this reduces the boilerplate for using dotty in your project to: scalaOrganization := "ch.epfl.lamp" scalaVersion := "0.1.1-SNAPSHOT" scalaBinaryVersion := "2.11" // dotty itself is only published as a // 2.11 artefact currently
41e3b4e
to
3c6db1c
Compare
Fixed |
Before this commit, using dotty in your sbt project required to add:
in your build.sbt. We might as well automatically do this, this reduces
the boilerplate for using dotty in your project to:
Note: I did not add a test for this since there is no released version of dotty yet, we could have a test that depends on a nightly build of dotty (http://central.maven.org/maven2/ch/epfl/lamp/dotty_2.11/) but I'm not sure if maven keeps old nightly builds around forever.