[INFRAUTILS-52] jHades doesn’t support JDK 9+ Created: 24/Sep/18 Updated: 09/Nov/18 Resolved: 09/Nov/18 |
|
| Status: | Resolved |
| Project: | infrautils |
| Component/s: | None |
| Affects Version/s: | Neon |
| Fix Version/s: | Neon |
| Type: | Bug | Priority: | Medium |
| Reporter: | Stephen Kitt | Assignee: | Michael Vorburger |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||
| Description |
|
See https://github.com/jhades/jhades/issues/9 : jHades doesn’t support JDK 9 or later, which means ClasspathHellDuplicatesCheckRule can’t be used as currently implemented. |
| Comments |
| Comment by Michael Vorburger [ 24/Sep/18 ] |
|
I can look into this some time (perhaps more next week?), but one of the options here may be to just ditch it... |
| Comment by Michael Vorburger [ 04/Oct/18 ] |
|
I've concluded that it's non-trivial at all to fix JHades to support Java 9+ (11), see update in jhades #9 and thefore instead just completely re-implemented our ClasspathHellDuplicatesCheckRule based on classgraph (see also classgraph #256) in c/76625. |
| Comment by Michael Vorburger [ 04/Oct/18 ] |
|
Hm, interesting, c/76625 fails with |
| Comment by Michael Vorburger [ 04/Oct/18 ] |
|
classgraph released a new version https://github.com/classgraph/classgraph/releases/tag/classgraph-4.2.7 - I'll use that. |
| Comment by Michael Vorburger [ 30/Oct/18 ] |
|
skitt (or rovarga, if interested), wanna test and confirm that c/76625 works as expected with Java 9/10/11 ? |
| Comment by Michael Vorburger [ 09/Nov/18 ] |
|
|