Skip to content

Added latest Guava and SLF4J compatibility#659

Merged
TuxCoding merged 2 commits intoTuxCoding:mainfrom
AlbeMiglio:main
Dec 1, 2021
Merged

Added latest Guava and SLF4J compatibility#659
TuxCoding merged 2 commits intoTuxCoding:mainfrom
AlbeMiglio:main

Conversation

@AlbeMiglio
Copy link
Copy Markdown
Contributor

(1.18+ breaks support to older guava methods)

Related issue

java.lang.NoSuchMethodError: 'java.lang.String com.google.common.net.HostAndPort.getHostText()'
at com.github.games647.fastlogin.core.shared.FastLoginCore.lambda$load$2(FastLoginCore.java:129) ~[?:?]
at java.util.stream.ReferencePipeline$3$1.accept(ReferencePipeline.java:197) ~[?:?]
at java.util.stream.ReferencePipeline$3$1.accept(ReferencePipeline.java:197) ~[?:?]
at java.util.ArrayList$ArrayListSpliterator.forEachRemaining(ArrayList.java:1625) ~[?:?]
at java.util.stream.AbstractPipeline.copyInto(AbstractPipeline.java:484) ~[?:?]
at java.util.stream.AbstractPipeline.wrapAndCopyInto(AbstractPipeline.java:474) ~[?:?]
at java.util.stream.ReduceOps$ReduceOp.evaluateSequential(ReduceOps.java:913) ~[?:?]
at java.util.stream.AbstractPipeline.evaluate(AbstractPipeline.java:234) ~[?:?]
at java.util.stream.ReferencePipeline.collect(ReferencePipeline.java:682) ~[?:?]
at com.github.games647.fastlogin.core.shared.FastLoginCore.load(FastLoginCore.java:131) ~[?:?]
at com.github.games647.fastlogin.bungee.FastLoginBungee.onEnable(FastLoginBungee.java:88) ~[?:?]
at net.md_5.bungee.api.plugin.PluginManager.enablePlugins(PluginManager.java:315) ~[Waterfall-1.18-461.jar:git:Waterfall-Bootstrap:1.18-R0.1-SNAPSHOT:3e0a6ae:461]
at net.md_5.bungee.BungeeCord.start(BungeeCord.java:290) ~[Waterfall-1.18-461.jar:git:Waterfall-Bootstrap:1.18-R0.1-SNAPSHOT:3e0a6ae:461]
at net.md_5.bungee.BungeeCordLauncher.main(BungeeCordLauncher.java:67) ~[Waterfall-1.18-461.jar:git:Waterfall-Bootstrap:1.18-R0.1-SNAPSHOT:3e0a6ae:461]
at net.md_5.bungee.Bootstrap.main(Bootstrap.java:15) ~[Waterfall-1.18-461.jar:git:Waterfall-Bootstrap:1.18-R0.1-SNAPSHOT:3e0a6ae:461]

(1.18+ breaks support to older guava methods)
@TuxCoding TuxCoding added the bug Something isn't working label Dec 1, 2021
@TuxCoding
Copy link
Copy Markdown
Owner

Thanks, didn't noticed the breaking change. Sadly this breaks 1.8 compat, which this project still targets 😭. Furthermore the velocity proxy also ships an outdated guava version: 25.1-jre. However this supports the new method.

Shading a nearly 3 MiB library for older versions seems so wasteful.

@TuxCoding TuxCoding merged commit 829ba79 into TuxCoding:main Dec 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants