Skip to content

Commit fd6a71c

Browse files
committed
Improve bootstrap script
- Don't run Scaladoc in the first pass of building a module. - Exclude scala-actors-migration module from deletion logic to speed things up
1 parent 08f9b03 commit fd6a71c

File tree

1 file changed

+13
-7
lines changed

1 file changed

+13
-7
lines changed

scripts/jobs/integrate/bootstrap

Lines changed: 13 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@
2222
# - To prevent staging on sonatype (for testing), set publishToSonatype to anything but "yes"
2323
# - Note: After building a release, the jenkins job provides an updated versions.properties file as artifact.
2424
# Put this file in the Scala repo and create a pull request, and also update the file build.number.
25-
#
25+
#
2626
# - Otherwise, a nightly release is built:
2727
# - version number is read from the build.number file, extended with -$sha-nightly
2828

@@ -217,7 +217,9 @@ buildXML() {
217217
then echo "Found scala-xml $XML_VER; not building."
218218
else
219219
update scala scala-xml "$XML_REF" && gfxd
220-
sbtBuild 'set version := "'$XML_VER'-DOC"' $clean doc 'set version := "'$XML_VER'"' test "${buildTasks[@]}"
220+
if [ "$XML_BUILT" == "yes" ]; then doc=doc; else doc='set publishArtifact in packageDoc := false'; fi
221+
sbtBuild 'set version := "'$XML_VER'-DOC"' $clean "$doc" 'set version := "'$XML_VER'"' test "${buildTasks[@]}"
222+
221223
XML_BUILT="yes" # ensure the module is built and published when buildXML is invoked for the second time, see comment above
222224
fi
223225
}
@@ -227,7 +229,8 @@ buildParsers() {
227229
then echo "Found scala-parser-combinators $PARSERS_VER; not building."
228230
else
229231
update scala scala-parser-combinators "$PARSERS_REF" && gfxd
230-
sbtBuild 'set version := "'$PARSERS_VER'-DOC"' $clean doc 'set version := "'$PARSERS_VER'"' test "${buildTasks[@]}"
232+
if [ "$PARSERS_BUILT" == "yes" ]; then doc=doc; else doc='set publishArtifact in packageDoc := false'; fi
233+
sbtBuild 'set version := "'$PARSERS_VER'-DOC"' $clean "$doc" 'set version := "'$PARSERS_VER'"' test "${buildTasks[@]}"
231234
PARSERS_BUILT="yes"
232235
fi
233236
}
@@ -237,7 +240,8 @@ buildPartest() {
237240
then echo "Found scala-partest $PARTEST_VER; not building."
238241
else
239242
update scala scala-partest "$PARTEST_REF" && gfxd
240-
sbtBuild 'set version :="'$PARTEST_VER'"' 'set VersionKeys.scalaXmlVersion := "'$XML_VER'"' 'set VersionKeys.scalaCheckVersion := "'$SCALACHECK_VER'"' $clean test "${buildTasks[@]}"
243+
if [ "$PARTEST_BUILT" == "yes" ]; then doc=doc; else doc='set publishArtifact in packageDoc := false'; fi
244+
sbtBuild 'set version :="'$PARTEST_VER'"' 'set VersionKeys.scalaXmlVersion := "'$XML_VER'"' 'set VersionKeys.scalaCheckVersion := "'$SCALACHECK_VER'"' $clean "$doc" test "${buildTasks[@]}"
241245
PARTEST_BUILT="yes"
242246
fi
243247
}
@@ -256,7 +260,8 @@ buildSwing() {
256260
then echo "Found scala-swing $SWING_VER; not building."
257261
else
258262
update scala scala-swing "$SWING_REF" && gfxd
259-
sbtBuild 'set version := "'$SWING_VER'"' $clean test "${buildTasks[@]}"
263+
if [ "$SCALACHECK_BUILT" == "yes" ]; then doc=doc; else doc='set publishArtifact in packageDoc := false'; fi
264+
sbtBuild 'set version := "'$SWING_VER'"' $clean "$doc" test "${buildTasks[@]}"
260265
SWING_BUILT="yes"
261266
fi
262267
}
@@ -267,7 +272,8 @@ buildScalacheck(){
267272
then echo "Found scalacheck $SCALACHECK_VER; not building."
268273
else
269274
update rickynils scalacheck $SCALACHECK_REF && gfxd
270-
sbtBuild 'set version := "'$SCALACHECK_VER'"' 'set VersionKeys.scalaParserCombinatorsVersion := "'$PARSERS_VER'"' $clean publish # test times out NOTE: never published to sonatype
275+
if [ "$SCALACHECK_BUILT" == "yes" ]; then doc=doc; else doc='set publishArtifact in packageDoc := false'; fi
276+
sbtBuild 'set version := "'$SCALACHECK_VER'"' 'set VersionKeys.scalaParserCombinatorsVersion := "'$PARSERS_VER'"' $clean "$doc" publish # test times out NOTE: never published to sonatype
271277
SCALACHECK_BUILT="yes"
272278
fi
273279
}
@@ -444,7 +450,7 @@ removeExistingBuilds() {
444450
local netrcFile="$HOME/.credentials-private-repo-netrc"
445451

446452
local storageApiUrl=`echo $releaseTempRepoUrl | sed 's/\(scala-release-temp\)/api\/storage\/\1/'`
447-
local scalaLangModules=`curl -s $storageApiUrl/org/scala-lang | jq -r '.children | .[] | "org/scala-lang" + .uri'`
453+
local scalaLangModules=`curl -s $storageApiUrl/org/scala-lang | jq -r '.children | .[] | "org/scala-lang" + .uri' | grep -v scala-actors-migration`
448454

449455
for module in "org/scalacheck" $scalaLangModules; do
450456
local artifacts=`curl -s $storageApiUrl/$module | jq -r ".children | .[] | select(.uri | contains(\"$SCALA_VER\")) | .uri"`

0 commit comments

Comments
 (0)