From 9cba434b228e33f62e01460abe186fb56038a70e Mon Sep 17 00:00:00 2001 From: Lukas Rytz Date: Tue, 21 Apr 2015 13:38:55 +0200 Subject: Remove the continuations plugin module dependency --- build-ant-macros.xml | 8 ++--- build.xml | 35 ++-------------------- scripts/jobs/integrate/bootstrap | 35 ++-------------------- src/build/bnd/scala-continuations-library.bnd | 7 ----- src/build/bnd/scala-continuations-plugin.bnd | 7 ----- src/build/maven/scala-dist-pom.xml | 6 ---- src/build/maven/scala-library-all-pom.xml | 10 ------- src/library/rootdoc.txt | 2 -- .../reflect/internal/pickling/UnPickler.scala | 1 - versions.properties | 5 ---- 10 files changed, 7 insertions(+), 109 deletions(-) delete mode 100644 src/build/bnd/scala-continuations-library.bnd delete mode 100644 src/build/bnd/scala-continuations-plugin.bnd diff --git a/build-ant-macros.xml b/build-ant-macros.xml index 259d6a6eb6..0b990c0e28 100644 --- a/build-ant-macros.xml +++ b/build-ant-macros.xml @@ -469,9 +469,7 @@ - - - + @@ -595,9 +593,7 @@ - - - + diff --git a/build.xml b/build.xml index 1f51ae9078..fa5316815a 100755 --- a/build.xml +++ b/build.xml @@ -75,8 +75,8 @@ TODO: - - + + @@ -311,9 +311,6 @@ TODO: necessary cross suffix (usually something like "_2.11.0-M6". --> - - - @@ -347,8 +344,6 @@ TODO: - - @@ -367,8 +362,6 @@ TODO: - - @@ -559,11 +552,8 @@ TODO: - - - @@ -574,11 +564,8 @@ TODO: - - - @@ -627,18 +614,6 @@ TODO: - - - - - - - - - - - - @@ -659,7 +634,7 @@ TODO: - + @@ -1297,8 +1272,6 @@ TODO: - - @@ -1328,8 +1301,6 @@ TODO: - - diff --git a/scripts/jobs/integrate/bootstrap b/scripts/jobs/integrate/bootstrap index 34cca4e980..d4fadf061d 100755 --- a/scripts/jobs/integrate/bootstrap +++ b/scripts/jobs/integrate/bootstrap @@ -182,6 +182,7 @@ sbtBuild() { sbtResolve() { cd $baseDir/resolutionScratch_ touch build.sbt + # TODO lry: not needed after removing continuations cross=${4-binary} # Disabled / binary / full echo "### sbtResolve: $sbtCmd $sbtArgs " "${scalaVersionTasks[@]}" "\"$1\" % \"$2\" % \"$3\" cross CrossVersion.$cross" $sbtCmd $sbtArgs "${scalaVersionTasks[@]}" \ @@ -245,27 +246,6 @@ buildPartest() { # fi # } -buildContinuations() { - if [ "$CONT_PLUG_BUILT" != "yes" ] && [ "$forceRebuild" != "yes" ] && ( sbtResolve "org.scala-lang.plugins" "scala-continuations-plugin" $CONTINUATIONS_VER full ) - then echo "Found scala-continuations-plugin $CONTINUATIONS_VER; not building." - else - update scala scala-continuations $CONTINUATIONS_REF && gfxd - - $sbtCmd $sbtArgs 'project plugin' "${scalaVersionTasks[@]}" "${publishTasks[@]}" \ - 'set version := "'$CONTINUATIONS_VER'"' $clean "compile:package" test "${buildTasks[@]}" # https://github.com/scala/scala-continuations/pull/4 - CONT_PLUG_BUILT="yes" - fi - - if [ "$CONT_LIB_BUILT" != "yes" ] && [ "$forceRebuild" != "yes" ] && ( sbtResolve "org.scala-lang.plugins" "scala-continuations-library" $CONTINUATIONS_VER ) - then echo "Found scala-continuations-library $CONTINUATIONS_VER; not building." - else - update scala scala-continuations $CONTINUATIONS_REF && gfxd - $sbtCmd $sbtArgs 'project library' "${scalaVersionTasks[@]}" "${publishTasks[@]}" \ - 'set version := "'$CONTINUATIONS_VER'"' $clean test "${buildTasks[@]}" - CONT_LIB_BUILT="yes" - fi -} - buildSwing() { if [ "$SWING_BUILT" != "yes" ] && [ "$forceRebuild" != "yes" ] && ( sbtResolve "org.scala-lang.modules" "scala-swing" $SWING_VER ) then echo "Found scala-swing $SWING_VER; not building." @@ -283,7 +263,7 @@ buildActorsMigration(){ update scala actors-migration "$ACTORS_MIGRATION_REF" && gfxd # not running tests because # [error] Test scala.actors.migration.NestedReact.testNestedReactAkka failed: java.util.concurrent.TimeoutException: Futures timed out after [20 seconds] - sbtBuild 'set version := "'$ACTORS_MIGRATION_VER'"' 'set VersionKeys.continuationsVersion := "'$CONTINUATIONS_VER'"' $clean "${buildTasks[@]}" + sbtBuild 'set version := "'$ACTORS_MIGRATION_VER'"' $clean "${buildTasks[@]}" ACTORS_MIGRATION_BUILT="yes" fi } @@ -302,7 +282,6 @@ buildScalacheck(){ buildModules() { buildXML buildParsers - buildContinuations buildSwing buildActorsMigration buildScalacheck @@ -385,7 +364,6 @@ deriveModuleVersions() { # use versions.properties as defaults when no version specified on command line XML_VER=${XML_VER-$scala_xml_version_number} PARSERS_VER=${PARSERS_VER-$scala_parser_combinators_version_number} - CONTINUATIONS_VER=${CONTINUATIONS_VER-$scala_continuations_plugin_version_number} SWING_VER=${SWING_VER-$scala_swing_version_number} ACTORS_MIGRATION_VER=${ACTORS_MIGRATION_VER-$actors_migration_version_number} PARTEST_VER=${PARTEST_VER-$partest_version_number} @@ -394,7 +372,6 @@ deriveModuleVersions() { # If a _VER was not specified, the corresponding _REF will be non-empty by now (as specified, or HEAD) XML_REF=${XML_REF-"v$XML_VER"} PARSERS_REF=${PARSERS_REF-"v$PARSERS_VER"} - CONTINUATIONS_REF=${CONTINUATIONS_REF-"v$CONTINUATIONS_VER"} SWING_REF=${SWING_REF-"v$SWING_VER"} ACTORS_MIGRATION_REF=${ACTORS_MIGRATION_REF-"v$ACTORS_MIGRATION_VER"} PARTEST_REF=${PARTEST_REF-"v$PARTEST_VER"} @@ -403,7 +380,6 @@ deriveModuleVersions() { else XML_VER=${XML_VER-$(deriveVersion scala scala-xml "$XML_REF")} PARSERS_VER=${PARSERS_VER-$(deriveVersion scala scala-parser-combinators "$PARSERS_REF")} - CONTINUATIONS_VER=${CONTINUATIONS_VER-$(deriveVersion scala scala-continuations "$CONTINUATIONS_REF")} SWING_VER=${SWING_VER-$(deriveVersion scala scala-swing "$SWING_REF")} ACTORS_MIGRATION_VER=${ACTORS_MIGRATION_VER-$(deriveVersion scala actors-migration "$ACTORS_MIGRATION_REF")} PARTEST_VER=${PARTEST_VER-$(deriveVersion scala scala-partest "$PARTEST_REF")} @@ -411,7 +387,6 @@ deriveModuleVersions() { XML_REF=${XML_REF-"HEAD"} PARSERS_REF=${PARSERS_REF-"HEAD"} - CONTINUATIONS_REF=${CONTINUATIONS_REF-"HEAD"} SWING_REF=${SWING_REF-"HEAD"} ACTORS_MIGRATION_REF=${ACTORS_MIGRATION_REF-"HEAD"} PARTEST_REF=${PARTEST_REF-"HEAD"} @@ -421,7 +396,6 @@ deriveModuleVersions() { echo "Module versions (versioning strategy: $moduleVersioning):" echo "ACTORS_MIGRATION = $ACTORS_MIGRATION_VER at $ACTORS_MIGRATION_REF" - echo "CONTINUATIONS = $CONTINUATIONS_VER at $CONTINUATIONS_REF" echo "PARSERS = $PARSERS_VER at $PARSERS_REF" echo "PARTEST = $PARTEST_VER at $PARTEST_REF" echo "SCALACHECK = $SCALACHECK_VER at $SCALACHECK_REF" @@ -438,8 +412,6 @@ constructUpdatedModuleVersions() { # either because the variables (XML_VER) were provided, or because we're building the modules from HEAD. # in the common case, the values are the same as in versions.properties. updatedModuleVersions=("${updatedModuleVersions[@]}" "-Dactors-migration.version.number=$ACTORS_MIGRATION_VER") - updatedModuleVersions=("${updatedModuleVersions[@]}" "-Dscala-continuations-library.version.number=$CONTINUATIONS_VER") - updatedModuleVersions=("${updatedModuleVersions[@]}" "-Dscala-continuations-plugin.version.number=$CONTINUATIONS_VER") updatedModuleVersions=("${updatedModuleVersions[@]}" "-Dscala-parser-combinators.version.number=$PARSERS_VER") updatedModuleVersions=("${updatedModuleVersions[@]}" "-Dscala-swing.version.number=$SWING_VER") updatedModuleVersions=("${updatedModuleVersions[@]}" "-Dscala-xml.version.number=$XML_VER") @@ -506,10 +478,7 @@ bootstrap() { cd $baseDir rm -rf build/ # must leave everything else in $baseDir for downstream jobs - # scala.full.version determines the dependency of scala-dist on the continuations plugin, - # which is fully cross-versioned (for $SCALA_VER, the version we're releasing) ant -Dstarr.version=$SCALA_VER\ - -Dscala.full.version=$SCALA_VER\ -Dextra.repo.url=$releaseTempRepoUrl\ -Dmaven.version.suffix=$SCALA_VER_SUFFIX\ ${updatedModuleVersions[@]} \ diff --git a/src/build/bnd/scala-continuations-library.bnd b/src/build/bnd/scala-continuations-library.bnd deleted file mode 100644 index b36718cc5b..0000000000 --- a/src/build/bnd/scala-continuations-library.bnd +++ /dev/null @@ -1,7 +0,0 @@ -Bundle-Name: Scala Delimited Continuations Library -Bundle-SymbolicName: org.scala-lang.plugins.scala-continuations-library -ver: @CONTINUATIONS_LIBRARY_VERSION@ -Bundle-Version: ${ver} -Export-Package: *;version=${ver} -Import-Package: scala.*;version="${range;[==,=+);@VERSION@}",* -Bundle-RequiredExecutionEnvironment: JavaSE-1.6, JavaSE-1.7 diff --git a/src/build/bnd/scala-continuations-plugin.bnd b/src/build/bnd/scala-continuations-plugin.bnd deleted file mode 100644 index 2f2464b452..0000000000 --- a/src/build/bnd/scala-continuations-plugin.bnd +++ /dev/null @@ -1,7 +0,0 @@ -Bundle-Name: Scala Delimited Continuations Compiler Plugin -Bundle-SymbolicName: org.scala-lang.plugins.scala-continuations-plugin -ver: @CONTINUATIONS_PLUGIN_VERSION@ -Bundle-Version: ${ver} -Export-Package: *;version=${ver} -Import-Package: scala.*;version="${range;[==,=+);@VERSION@}",* -Bundle-RequiredExecutionEnvironment: JavaSE-1.6, JavaSE-1.7 diff --git a/src/build/maven/scala-dist-pom.xml b/src/build/maven/scala-dist-pom.xml index 9477e14285..1f6b6710ac 100644 --- a/src/build/maven/scala-dist-pom.xml +++ b/src/build/maven/scala-dist-pom.xml @@ -44,12 +44,6 @@ scalap @VERSION@ - - org.scala-lang.plugins - - scala-continuations-plugin_@SCALA_FULL_VERSION@ - @CONTINUATIONS_PLUGIN_VERSION@ - - - org.scala-lang.plugins - scala-continuations-library_@SCALA_BINARY_VERSION@ - @CONTINUATIONS_LIBRARY_VERSION@ - org.scala-lang.modules scala-swing_@SCALA_BINARY_VERSION@ diff --git a/src/library/rootdoc.txt b/src/library/rootdoc.txt index e84942b8c4..0e9a175fcc 100644 --- a/src/library/rootdoc.txt +++ b/src/library/rootdoc.txt @@ -44,8 +44,6 @@ Additional parts of the standard library are shipped as separate libraries. Thes - [[scala.reflect `scala.reflect`]] - Scala's reflection API (scala-reflect.jar) - [[scala.xml `scala.xml`]] - XML parsing, manipulation, and serialization (scala-xml.jar) - [[scala.swing `scala.swing`]] - A convenient wrapper around Java's GUI framework called Swing (scala-swing.jar) - - [[scala.util.continuations `scala.util.continuations`]] - Delimited continuations using continuation-passing-style - (scala-continuations-library.jar, scala-continuations-plugin.jar) - [[scala.util.parsing `scala.util.parsing`]] - [[scala.util.parsing.combinator Parser combinators]], including an example implementation of a [[scala.util.parsing.json JSON parser]] (scala-parser-combinators.jar) - [[scala.actors `scala.actors`]] - Actor-based concurrency (deprecated and replaced by Akka actors, diff --git a/src/reflect/scala/reflect/internal/pickling/UnPickler.scala b/src/reflect/scala/reflect/internal/pickling/UnPickler.scala index 1fc7aebab0..464bbad2cd 100644 --- a/src/reflect/scala/reflect/internal/pickling/UnPickler.scala +++ b/src/reflect/scala/reflect/internal/pickling/UnPickler.scala @@ -239,7 +239,6 @@ abstract class UnPickler { if (missing.startsWith("scala.xml")) Some(("org.scala-lang.modules", "scala-xml")) else if (missing.startsWith("scala.util.parsing")) Some(("org.scala-lang.modules", "scala-parser-combinators")) else if (missing.startsWith("scala.swing")) Some(("org.scala-lang.modules", "scala-swing")) - else if (missing.startsWith("scala.util.continuations")) Some(("org.scala-lang.plugins", "scala-continuations-library")) else None (module map { case (group, art) => diff --git a/versions.properties b/versions.properties index cdba83df41..44fc6f15a7 100644 --- a/versions.properties +++ b/versions.properties @@ -17,15 +17,10 @@ starr.version=2.11.6 # the binary version is 2.x. For milestones and RCs, modules are cross-built against the # full version, so the value, so the value is the full version (e.g. 2.12.0-M1). scala.binary.version=2.11 -# e.g. 2.11.0-RC1, 2.11.0, 2.11.1-RC1, 2.11.1 -# this defines the dependency on scala-continuations-plugin in scala-dist's pom -scala.full.version=2.11.6 # external modules shipped with distribution, as specified by scala-library-all's pom scala-xml.version.number=1.0.3 scala-parser-combinators.version.number=1.0.3 -scala-continuations-plugin.version.number=1.0.2 -scala-continuations-library.version.number=1.0.2 scala-swing.version.number=1.0.1 akka-actor.version.number=2.3.4 actors-migration.version.number=1.1.0 -- cgit v1.2.3