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

GWT 2.9 has dependancies which are not supported in latest Java versions. #9750

Closed
wrbst opened this issue Jan 21, 2022 · 5 comments
Closed
Milestone

Comments

@wrbst
Copy link

wrbst commented Jan 21, 2022

GWT version: 2.9
Browser (with version): Chrome (latest)
Operating System: Windows Server 2016


Description

Hello mates,
Me and my colleagues are bumping our existing project from java 8 to java 16, later to 17 (due yet unsupported libs/plugins, etc.).
And today we hit a problem with the ASM, the lib which comes with GWT 2.9 is version 7.1 (from 2019), which supports up to Java 13 - the latest version is 9.2.

Can I kindly request for a bump. As well probably and other dependencies will need to be updated in order to support the latest Java LTS version - 17.

Steps to reproduce

Use Java 17.

Known workarounds
Links to further discussions
@shrhoads
Copy link

shrhoads commented Jan 21, 2022 via email

@wrbst
Copy link
Author

wrbst commented Jan 21, 2022

Hello Shawn,

Thank you for the quick answer, but we cannot use private projects/repos.
As well we don't use Jetty.

Into the maven POM, we have tried to exclude bundled extension ASM 7.1 and then include the new version 9.2 as dependency, but then another lib -> Typostry "blow's up" ( java.lang.NoClassDefFoundError: org/apache/tapestry/util/text/LocalizedProperties) - they also use ASM as dependency and they have introduce support for ASM 9.2 in the latest version 5.8.0.
Even trying to updating the Typostry the same way as ASM we still got this error during compile: java.lang.NoClassDefFoundError: org/apache/tapestry/util/text/LocalizedProperties

So some more general solution will be needed.

@niloc132
Copy link
Member

niloc132 commented Jan 21, 2022

There is a patch under review right now (it has been +2'd, but I haven't had the chance to return to it to finalize it) which updates to latest Jetty 9.4, and asm 9.2.

With that said, can you confirm that you are specifically failing when trying to run your server app inside GWT's dev mode? If so, this is not encouraged (and the jetty/asm updates are to allow htmlunit to be made more recent, not to target your use case) - it is far better to run your own server, and to run DevMode/CodeServer separately, with the -war/-launchDir argument to build JS where your usual server can see it, and provide it to the browser. That guarantees that problems like this or jetty specific configuration issues cannot affect you.

See also:
https://gwt-review.googlesource.com/c/gwt/+/23840
#9720

From that last link, try adding https://repo.vertispan.com/gwt-snapshot/ as a maven repository, and use gwt version 2.10.0-htmlunit-upgrade-SNAPSHOT. This might be slightly out of date at this point, but still includes a very recent jetty/asm.

@wrbst
Copy link
Author

wrbst commented Jan 21, 2022

Hello Colin,

Thanks for very much for you answer as well.
We use the LocalizableResource in a very custom way. The first try described in my previous comment was in gwt-dev.
But changing the dependencies in gwt-user and "reverting" gwt-dev seem to solve the problem for now.

Still... work in progress.

You have done a lot of work as I saw the commits here. Good job! Hope to see new GWT version soon.

@niloc132 niloc132 added this to the 2.10 milestone Apr 19, 2022
@niloc132
Copy link
Member

niloc132 commented Jun 9, 2022

@wrbst if you're having other issues outside of the jetty version, can you file a new issue?

GWT 2.10 will be released in the next few days, marking this as closed.

@niloc132 niloc132 closed this as completed Jun 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants