Skip to content

Conversation

@paulp
Copy link
Contributor

@paulp paulp commented Dec 6, 2012

This PR merges and supersedes #1717 and #1718.

James Iry and others added 6 commits December 5, 2012 14:36
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
-
+
@scala-jenkins
Copy link

Started jenkins job pr-rangepos at https://scala-webapps.epfl.ch/jenkins/job/pr-rangepos/1096/

@scala-jenkins
Copy link

jenkins job pr-rangepos: Success - https://scala-webapps.epfl.ch/jenkins/job/pr-rangepos/1096/

@scala-jenkins
Copy link

Started jenkins job pr-scala-testsuite-linux-opt at https://scala-webapps.epfl.ch/jenkins/job/pr-scala-testsuite-linux-opt/1806/

@scala-jenkins
Copy link

jenkins job pr-scala-testsuite-linux-opt: Success - https://scala-webapps.epfl.ch/jenkins/job/pr-scala-testsuite-linux-opt/1806/

@JamesIry
Copy link
Contributor

JamesIry commented Dec 7, 2012

LGTM

@paulp
Copy link
Contributor Author

paulp commented Dec 9, 2012

Cue "Taps."

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants