Skip to content

Commit

Permalink
Merge pull request #10668 from SethTisue/remove-sbt-workaround
Browse files Browse the repository at this point in the history
Remove no-longer-needed JDK 21 workaround in build
  • Loading branch information
SethTisue committed Jan 19, 2024
2 parents 6c5f388 + 9ee7c1e commit 03396e9
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions .travis.yml
Expand Up @@ -44,9 +44,9 @@ jobs:
name: "JDK 8 pr validation"
if: type = pull_request
script:
- sbt -Dsbt.io.jdktimestamps=true -Dsbt.scala.version=2.12.18 -warn setupPublishCore generateBuildCharacterPropertiesFile headerCheck publishLocal
- sbt -Dsbt.io.jdktimestamps=true -warn setupPublishCore generateBuildCharacterPropertiesFile headerCheck publishLocal
- STARR=`cat buildcharacter.properties | grep ^maven.version.number | cut -d= -f2` && echo $STARR
- sbt -Dsbt.io.jdktimestamps=true -Dsbt.scala.version=2.12.18 -Dstarr.version=$STARR -warn setupValidateTest test:compile info testAll
- sbt -Dsbt.io.jdktimestamps=true -Dstarr.version=$STARR -warn setupValidateTest test:compile info testAll

# build the spec using jekyll
- stage: build
Expand Down
2 changes: 1 addition & 1 deletion scripts/common
Expand Up @@ -19,7 +19,7 @@ SBT_VERSION=`grep sbt.version $WORKSPACE/project/build.properties | sed -n 's/sb
SBT_CMD=${SBT_CMD-sbt}
# the jdktimestamps thing is to work around https://github.com/sbt/sbt/issues/7463 --
# it can be removed again once we're on an sbt version with a fix
SBT_CMD="$SBT_CMD -Dsbt.io.jdktimestamps=true -Dsbt.scala.version=2.12.18 -sbt-version $SBT_VERSION"
SBT_CMD="$SBT_CMD -Dsbt.io.jdktimestamps=true -sbt-version $SBT_VERSION"

# repo to publish builds
integrationRepoUrl=${integrationRepoUrl-"https://scala-ci.typesafe.com/artifactory/scala-integration/"}
Expand Down

0 comments on commit 03396e9

Please sign in to comment.