Skip to content
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

add JDK 13 community build #873

Closed
SethTisue opened this issue Mar 1, 2019 · 18 comments
Closed

add JDK 13 community build #873

SethTisue opened this issue Mar 1, 2019 · 18 comments
Assignees
Labels

Comments

@SethTisue
Copy link
Member

what, already? sure, why not. setting it up is easy, I know the drill now, and we have enough disk space on the workers.

early access builds are available from https://jdk.java.net/13/

@SethTisue
Copy link
Member Author

JDK 13 is in final release, so it's time for this.

to conserve resources, we can get rid of the JDK 12 jobs.

@eed3si9n
Copy link
Member

It might require sbt 1.3.1 that uses Scala 2.12.10.

@mr-git
Copy link

mr-git commented Sep 30, 2019

and https://jdk.java.net/14/ early builds are available too ;)

@SethTisue
Copy link
Member Author

instructions for installing new OpenJDK versions on the behemoths: #942 (comment)

@SethTisue
Copy link
Member Author

and https://jdk.java.net/14/ early builds are available too ;)

@mr-git I've made a separate ticket on that, though I don't plan to tackle it immediately: #984

@SethTisue
Copy link
Member Author

@SethTisue
Copy link
Member Author

SethTisue commented Oct 3, 2019

EXTRACTION FAILED (failed: blaze, doodle, fs2, genjavadoc, monix, paradox, sbt, sbt-io, sbt-librarymanagement, sbt-util, scala-async, scala-collection-compat, scala-continuations, scala-js, scala-partest, scala-swing, sconfig, sttp, tsec, zinc)

https://scala-ci.typesafe.com/job/scala-2.12.x-jdk13-integrate-community-build/494/artifact/logs/

I'll have to look at these one at a time.

@SethTisue
Copy link
Member Author

repos that don't extract because they're on old sbt 1.x versions that don't work on JDK 13 (as @eed3si9n anticipated): blaze, doodle, fs2, genjavadoc, monix, paradox, sbt, sbt-*, scala-js, sconfig, sttp, tsec, zinc

Scala modules with hardcoded JDK version check needs updating: scala-async, scala-collection-compat, scala-continuations, scala-partest, scala-swing

I don't see any other classes of failure.

SethTisue added a commit to SethTisue/community-build that referenced this issue Oct 3, 2019
@SethTisue
Copy link
Member Author

#986 should fix the modules. after that I'll think about what to do about the others

SethTisue added a commit that referenced this issue Oct 4, 2019
@TimMoore
Copy link

TimMoore commented Oct 9, 2019

I just updated the wiki page (https://github.com/scala/community-builds/wiki) which had some dead links to the jdk12 builds.

@SethTisue
Copy link
Member Author

I'll think about what to do about the others

a couple possible workarounds I've thought of:

  • for the projects that need sbt 1.2, can we override the Scala version that sbt itself uses to be 2.12.10 instead of 2.12.7?
  • maybe we could make sbt-1-2-version be 1.2.8 on most JDKs but 1.3.2 on JDK 13+. the projects that need sbt 1.2 will fail, but at least they should extract, allowing other projects to build

@SethTisue
Copy link
Member Author

SethTisue commented Oct 10, 2019

can we override the Scala version that sbt itself uses to be 2.12.10 instead of 2.12.7?

#991 tries this approach (-Dsbt.scala.version=2.12.10)

@SethTisue
Copy link
Member Author

2.13.x: SUCCEEDED: 138

  • [jawn-0-11] [error] java.lang.IllegalArgumentException: Unsupported class file major version 57
  • [expression-evaluator] [error] java.lang.IllegalArgumentException: Unsupported class file major version 57
  • [log4s] [error] Unsupported class file major version 57
  • [mockito-scala] [info] org.mockito.exceptions.base.MockitoException: Mockito cannot mock this class: class user.org.mockito.scalatest.IdiomaticMockitoTest$Foo. (and many similar)
  • [sbt-io] [error] Error during tests: sbt.io.IOSpecification
  • [scala-async] [error] java.lang.IllegalArgumentException: Unsupported class file major version 57
  • [twitter-util:error] Unrecognized VM option 'AggressiveOpts'
  • scala-js has compile errors from new overloads in Java stdlib

SethTisue added a commit to SethTisue/scala-js that referenced this issue Oct 11, 2019
JDK 13 adds another overload to that method.

This turned up in the context of scala/community-build#873.
@SethTisue
Copy link
Member Author

2.12.x: SUCCEEDED: 141 (https://scala-ci.typesafe.com/view/scala-2.12.x/job/scala-2.12.x-jdk13-integrate-community-build/526/)

failures are about as above, plus:

  • [utest] [error] java.lang.IllegalArgumentException: Unsupported class file major version 57
  • [jawn-0-10] [error] Unsupported class file major version 57
    * [playframework] [error] Failed tests: play.mvc.HttpFormsTest
  • [scala-partest] [error] Unsupported class file major version 57
  • [curryhoward] java.lang.IllegalArgumentException: Unsupported class file major version 57
  • [breeze] [error] Caused by: scala.tools.asm.tree.analysis.AnalyzerException: Error at instruction 3: Unsupported class file major version 57

@SethTisue
Copy link
Member Author

declaring victory, according to our target ambition level

most of the 2.12 and 2.13 builds are green; the failures are don't look concerning, or are known issues in scala/bug

@SethTisue
Copy link
Member Author

SethTisue commented Jan 24, 2020

the 2.13.x-on-JDK-13 build keeps SIGSEGV'ing when running the scalacheck-shapeless tests — not every time, but very often

hoping a version upgrade would fix it, I did sudo apt-get install adoptopenjdk-13-hotspot which updated JDK 13 from a September 2019 build to 13.0.2+8-2 (done on all 3 behemoths)

but the first scalacheck-shapeless run on the new version still crashed. oh well

@SethTisue
Copy link
Member Author

ha! scalacheck-shapeless segfaults even on JDK 14 (14-ea+32-1423) on macOS, even on 2.13.1 and outside dbuild. reported upstream: alexarchambault/scalacheck-shapeless#149

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants