Skip to content
This repository has been archived by the owner on Aug 23, 2022. It is now read-only.

Use icedtea-web from the new source #125

Closed
fennibay opened this issue Jan 12, 2021 · 5 comments
Closed

Use icedtea-web from the new source #125

fennibay opened this issue Jan 12, 2021 · 5 comments

Comments

@fennibay
Copy link

icedtea-web was moved from https://icedtea.classpath.org/wiki/IcedTea-Web to github https://github.com/AdoptOpenJDK/IcedTea-Web and is now being maintained there, but ojdkbuild is still using an older version cloned at https://github.com/ojdkbuild/lookaside_icedtea-web

Are there any plans to move to using the latest upstream?

Specifically, I need the fix in AdoptOpenJDK/IcedTea-Web#679, but this PR not merged to the currently used lookaside_icedtea-web

Many thanks for maintaining this package.

@ojdkbuild
Copy link
Owner

Hi, thanks for the report!

IcedTea-Web build in this project predates the "modern" ITW on AdoptOpenJDK (wth Windows support etc). Unfortunately there are no plans for major ITW updates for jdk8 of for adding ITW to jdk11+ in this project. This ITW remains in a "deep maintenance".

Though the specific patch you pointed out looks to be small and non-intrusive. I've backported it and it is going to be included with the upcoming 8u282 release.

If you can look at this pre-release build (EA, does NOT include 8u282 security fixes) to check whether the patch fixes the actual problem - that would be great.

@fennibay
Copy link
Author

Hi, many thanks for the quick fix. Do you also have a zipped build instead of MSI?

@ojdkbuild
Copy link
Owner

Uploaded the ZIP with the same files as MSI above (excluding Mission Control) - https://drive.google.com/file/d/1eXw08ujrVKQkY5ucyd2wRLmzexHhA6aw/view?usp=sharing

@fennibay
Copy link
Author

fennibay commented Jan 17, 2021

Thank you. I just tried with 1.8.0.282-0 and the problem seems to be fixed.

To make sure I also tried again with 1.8.0.275-1, and could reproduce the problem with that.

So, it looks good. Thanks again! Shall I close the issue?

@ojdkbuild
Copy link
Owner

Sure, thanks for confirming, feel free to close it. Release version of 8u282 should be ready around the end of next week.

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

No branches or pull requests

2 participants