-
Notifications
You must be signed in to change notification settings - Fork 3.1k
Merge msil and genjvm deletions. #1724
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
Get rid of GenJVM and everything that refers to it. Also get rid of GenAndroid since it's dead code that refers to GenJVM.
It lives on in a branch born from this commit's parent. It's abrupt; no attempt is made to offer a "smooth transition" for the serious msil userbase, population zero. If anyone feels very strongly that such a transition is necessary, I will be happy to talk you into feeling differently.
* commit 'refs/pull/1717/head': SI-6769 Removes GenJVM backend Conflicts: src/compiler/scala/tools/nsc/backend/jvm/GenAndroid.scala
* commit 'refs/pull/1718/head': Expunged the .net backend. Conflicts: build.detach.xml build.examples.xml build.xml project/Build.scala src/compiler/scala/tools/ant/Scalac.scala src/compiler/scala/tools/nsc/Global.scala src/compiler/scala/tools/nsc/settings/StandardScalaSettings.scala src/compiler/scala/tools/nsc/symtab/clr/TypeParser.scala src/compiler/scala/tools/nsc/transform/Mixin.scala src/intellij/compiler.iml.SAMPLE tools/buildcp
Apparently this thing depends on trailing spaces because I can't change it in my trailing-space-stripping editor without the build failing with this informative message: BUILD FAILED /scala/trunk/build.xml:1403: Existing manifest /scala/trunk/build/pack/META-INF/MANIFEST.MF is invalid The whole diff between working and not working is whitespace: --- i/META-INF/MANIFEST.MF +++ w/META-INF/MANIFEST.MF @@ -4,11 +4,11 @@ Bundle-Name: Scala Distribution Bundle-SymbolicName: org.scala-ide.scala.compiler;singleton:=true Bundle-Version: 2.10.0.alpha Eclipse-LazyStart: true -Bundle-ClassPath: +Bundle-ClassPath: ., bin, lib/jline.jar, -Export-Package: +Export-Package: scala.tools.nsc, scala.tools.nsc.ast, scala.tools.nsc.ast.parser, @@ -45,7 +45,7 @@ Export-Package: scala.reflect.runtime, scala.reflect.internal.transform, scala.reflect.api, -Require-Bundle: +Require-Bundle: org.apache.ant, org.scala-ide.scala.library - +
|
Started jenkins job pr-rangepos at https://scala-webapps.epfl.ch/jenkins/job/pr-rangepos/1096/ |
|
jenkins job pr-rangepos: Success - https://scala-webapps.epfl.ch/jenkins/job/pr-rangepos/1096/ |
|
Started jenkins job pr-scala-testsuite-linux-opt at https://scala-webapps.epfl.ch/jenkins/job/pr-scala-testsuite-linux-opt/1806/ |
|
jenkins job pr-scala-testsuite-linux-opt: Success - https://scala-webapps.epfl.ch/jenkins/job/pr-scala-testsuite-linux-opt/1806/ |
|
LGTM |
|
Cue "Taps." |
This PR merges and supersedes #1717 and #1718.