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

[backport] upgrade to ASM 7 #7780

Merged
merged 2 commits into from Mar 1, 2019
Merged

[backport] upgrade to ASM 7 #7780

merged 2 commits into from Mar 1, 2019

Conversation

SethTisue
Copy link
Member

@SethTisue SethTisue commented Feb 22, 2019

2.12.x backport of #7384

goal: improve compatibility with JDK 12, as per scala/community-builds#864 and scala/bug#11372

fixes scala/bug#11372

@SethTisue
Copy link
Member Author

SethTisue commented Feb 22, 2019

in a local partial community build run on JDK 12, I verified that several projects that failed before are green now: scalaz,scala-async,scala-partest

but let's do a full JDK 12 run on Jenkins, too, just in case: https://scala-ci.typesafe.com/view/scala-2.12.x/job/scala-2.12.x-jdk12-integrate-community-build/24/

@SethTisue
Copy link
Member Author

SethTisue commented Feb 22, 2019

I verified locally that scala/bug#11372 goes away with 2.12.9-bin-1220d3c-SNAPSHOT

@SethTisue SethTisue closed this Feb 22, 2019
@SethTisue SethTisue reopened this Feb 22, 2019
@SethTisue
Copy link
Member Author

SethTisue commented Feb 22, 2019

let's do a full JDK 12 run on Jenkins, too, just in case

the results show that this PR raises the number of green projects in the community build from 98 to 139, and without introducing any additional failures

@adriaanm @retronym shall we merge?

@SethTisue
Copy link
Member Author

SethTisue commented Feb 22, 2019

(one additional piece of due diligence: https://asm.ow2.io/versions.html shows that 7.0 is still the most recent ASM version)

@smarter
Copy link
Member

smarter commented Feb 24, 2019

As far as I can tell, the compilation errors only happen when the backend optimizer is turned on, since that's what triggers ASM looking into classfiles from your dependencies, why not degrade gracefully and have the optimizer not look into classfiles it can't read ? Or if that's not possible, at least make the optimizer output an error message explaining to the user that he needs to downgrade his JVM, disable the optimizer, or upgrade his scalac.

@SethTisue
Copy link
Member Author

SethTisue commented Feb 24, 2019

As far as I can tell, the compilation errors only happen when the backend optimizer is turned on

yes, likely correct, I didn't get to the bottom of every individual failure, but every one I did look at it turned out to involved the optimizer

why not degrade gracefully

seems like a good suggestion, but it exceeds my own ambitions for this PR. perhaps someone else will take it up

@SethTisue
Copy link
Member Author

SethTisue commented Mar 1, 2019

I'm going to go ahead and merge this because I think we should have nightly builds that work on JDK 12. And for the community build.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-notes worth highlighting in next release notes
4 participants