8000 Add JDK 11 by OpenJDK to Travis CI matrix by eed3si9n · Pull Request #7311 · scala/scala · GitHub
[go: up one dir, main page]
Skip to content

Add JDK 11 by OpenJDK to Travis CI matrix #7311

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 4 commits into from
Closed

Add JDK 11 by OpenJDK to Travis CI matrix #7311

wants to merge 4 commits into from

Conversation

eed3si9n
Copy link
Member
@eed3si9n eed3si9n commented Oct 4, 2018

ref #7026

@scala-jenkins scala-jenkins added this to the 2.12.8 milestone Oct 4, 2018
@eed3si9n
Don't use Travis CI default Scala version
8d2ecf1 
The tests are running on wong starr because Travis CI injects `++$TRAVIS_SCALA_VERSION`.

```
java
-Xms2048M
-Xmx2048M
-Xss6M
-XX:MaxPermSize=512M
-jar
/home/travis/.sbt/launchers/0.13.17/sbt-launch.jar
++2.10.4
test
```
@xuwei-k
Copy link
Contributor
xuwei-k commented Oct 4, 2018 •
edited
Loading

I have fixed some tests in 2.13.x branch. (should be backport to 2.12.x)

still fail some tests scala/scala-dev#559 (comment)

@adriaanm
Copy link
Contributor
adriaanm commented Oct 4, 2018

Given the work on Java 11 compatibility already done in 2.13.x by xuwei-k and others, I propose we focus our effort there, and backport to 2.12.x once it's done.

@eed3si9n
Copy link
Member Author
eed3si9n commented Oct 4, 2018

@adriaanm Yea. I agree doing the actual fixes in 2.13.x. I just want to get to the test phase in Travis CI so we know how many tests fail for 2.12.x on JDK 11.

@eed3si9n eed3si9n mentioned this pull request Oct 4, 2018
Closed
3 tasks
@adriaanm
Copy link
Contributor

Closing, since the experiment has delivered its results. Feel free to reopen for another run, of course.

@adriaanm adriaanm closed this Oct 16, 2018
@SethTisue
Copy link
Member

this finally went forward at #9579

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
2.12.8
Development

Successfully merging this pull request may close these issues.
5 participants
@eed3si9n @xuwei-k @adriaanm @SethTisue @scala-jenkins
0