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.

Sign up for GitHub

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

Jump to bottom

Java 11, gradle shadow & libraries update #8

Merged
merged 6 commits into from
Dec 20, 2019
Merged

Java 11, gradle shadow & libraries update #8

merged 6 commits into from
Dec 20, 2019

Conversation

Shyrogan
Copy link
Contributor

No description provided.

@Shyrogan Shyrogan mentioned this pull request Dec 16, 2019
Closed
@vmarchaud
Copy link
Member

Does this PR works with Bungeecord too ? I believe there was some changes needed to support latest version

@MaximeMichaud
Copy link
Member

MaximeMichaud commented Dec 16, 2019 •
edited
Loading

Oopsi, j'ai discuté dans le mauvais PULL.
Hey ! Intéressant comme commit :p
J'ai quelques questions !
C'est quoi l'intérêt d'avoir mis DEFAULT_JVM_OPTS="-Xmx64m"?
Pour ProtocolLib, bien vue ! Je me demande si cela avait rapport avec d'autres problèmes.

J'aimerais testé et dépendant ce que cela donne, je serais POUR le push.
Et pour le v.3.0 à voir (si avec quelques autres modifications bien sûr)

Does this PR works with Bungeecord too ? I believe there was some changes needed to support latest version

Indeed, BungeeCord was not functionnal with MineWeb_Bridge.
#6
Otherwise, an alternative was possible with BungeeRCON.

And do you think that updating ProtocolLib can fix the problem with this one ? (IDK, I don't think, but it will be really nice) (Problem with Injection)

@Shyrogan
Copy link
Contributor Author

Does this PR works with Bungeecord too ? I believe there was some changes needed to support latest version

I didn't touch anything in the code pretty much, changed everything around.

J'ai quelques questions !
C'est quoi l'intérêt d'avoir mis DEFAULT_JVM_OPTS="-Xmx64m"?
Pour ProtocolLib, bien vue ! Je me demande si cela avait rapport avec d'autres problèmes.

C'est IntelliJ qui fait ça je pense de base

@Eywek
Copy link
Member

Eywek commented Dec 17, 2019

@MaximeMichaud t'as testé de ton côté aussi?
Ca me semble bon en tout cas

@vmarchaud
Copy link
Member

A l'epoque de quand j'avais dev le plugin, le probleme avec Bungee c'est que je patch au runtime son code pour pouvoir injecter mon netty handler. Sauf qu'a chaque fois qu'il change leur code sur cette partie, mon patch devient non-fonctionnel. Sans adapter le patching pour les nouvelles versions ca marchera toujours pas :(

@MaximeMichaud
Copy link
Member

@MaximeMichaud t'as testé de ton côté aussi?
Ca me semble bon en tout cas

Non, je n'ai pas encore eu le temps de testé.
Quand tu dis semble bon, ta compilé & testé ou juste check les commits ?

A l'epoque de quand j'avais dev le plugin, le probleme avec Bungee c'est que je patch au runtime son code pour pouvoir injecter mon netty handler. Sauf qu'a chaque fois qu'il change leur code sur cette partie, mon patch devient non-fonctionnel. Sans adapter le patching pour les nouvelles versions ca marchera toujours pas :(

K, je vois ! Ce serait possible de savoir s'il y a une rétrocompatibilité ? Je veux dire par là, si admettons je patch pour un build de 1.15, est-ce que cela fonctionnera pour tous les build précédents jusqu'au nouveau ?
Si oui, stv, je peux m'en occuper dans le futur. Même si le code ne devrait pas être considéré comme étant une solution à long terme.

@Shyrogan
Copy link
Contributor Author

Même si le code ne devrait pas être considéré comme étant une solution à long terme.
Complètement d'accord pour le coup haha

Perso je l'ai compilé et j'ai lancé un serveur en 1.8 (Spigot) avec Java 8 sans problème. D'après quelqu'un d'autre, ça fonctionne en 1.14.4 (Spigot) avec Java 11

@MaximeMichaud
Copy link
Member

MaximeMichaud commented Dec 18, 2019 •
edited
Loading

Même si le code ne devrait pas être considéré comme étant une solution à long terme.
Complètement d'accord pour le coup haha

Perso je l'ai compilé et j'ai lancé un serveur en 1.8 (Spigot) avec Java 8 sans problème. D'après quelqu'un d'autre, ça fonctionne en 1.14.4 (Spigot) avec Java 11

D'accord, je n’ai pas encore eu le temps de testé, mais j'aimerais bien voir.
Car, je testerai avec Java 11. Et si je peux fix le problème avec le Bungee, cela serait top.
Edit : Je regarde ça Jeudi Soir.

@Eywek
Copy link
Member

Eywek commented Dec 19, 2019

Une fois que @MaximeMichaud a test de son côté aussi je merge

@Shyrogan
Copy link
Contributor Author

Shyrogan commented Dec 19, 2019 •
edited
Loading

Même si le code ne devrait pas être considéré comme étant une solution à long terme.
Complètement d'accord pour le coup haha

Perso je l'ai compilé et j'ai lancé un serveur en 1.8 (Spigot) avec Java 8 sans problème. D'après quelqu'un d'autre, ça fonctionne en 1.14.4 (Spigot) avec Java 11

D'accord, je n’ai pas encore eu le temps de testé, mais j'aimerais bien voir.
Car, je testerai avec Java 11. Et si je peux fix le problème avec le Bungee, cela serait top.
Edit : Je regarde ça Jeudi Soir.

Pour Bungee je vois pas ce qui pourrait avoir changer car ça ne dépend pas directement de Minecraft, les références pour injecter le netty handler restent les mêmes (à vérifier).

@MaximeMichaud
Copy link
Member

Une fois que @MaximeMichaud a test de son côté aussi je merge

Testée avec Java 11
Le plugin fonctionne parfaitement seul.
Par contre, lorsqu'il y a ProtocolLib, celui n'arrive pas à charger.
Normalement, avec le 2.0.0, il devient rouge, mais là, il est juste pas présent.
Faudra voir avec la version 3.0.0 no-injector.

[00:02:17] [Server thread/ERROR]: Could not load 'plugins/ProtocolLib.jar' in folder 'plugins'
org.bukkit.plugin.InvalidPluginException: java.lang.IllegalArgumentException: Plugin already initialized!
	at org.bukkit.plugin.java.JavaPluginLoader.loadPlugin(JavaPluginLoader.java:133) ~[minecraft_server.jar:git-PaperSpigot-"4c7641d"]
	at org.bukkit.plugin.SimplePluginManager.loadPlugin(SimplePluginManager.java:331) ~[minecraft_server.jar:git-PaperSpigot-"4c7641d"]
	at org.bukkit.plugin.SimplePluginManager.loadPlugins(SimplePluginManager.java:254) [minecraft_server.jar:git-PaperSpigot-"4c7641d"]
	at org.bukkit.craftbukkit.v1_8_R3.CraftServer.loadPlugins(CraftServer.java:293) [minecraft_server.jar:git-PaperSpigot-"4c7641d"]
	at net.minecraft.server.v1_8_R3.DedicatedServer.init(DedicatedServer.java:202) [minecraft_server.jar:git-PaperSpigot-"4c7641d"]
	at net.minecraft.server.v1_8_R3.MinecraftServer.run(MinecraftServer.java:563) [minecraft_server.jar:git-PaperSpigot-"4c7641d"]
	at java.base/java.lang.Thread.run(Thread.java:834) [?:?]
Caused by: java.lang.IllegalArgumentException: Plugin already initialized!
	at org.bukkit.plugin.java.PluginClassLoader.initialize(PluginClassLoader.java:123) ~[minecraft_server.jar:git-PaperSpigot-"4c7641d"]
	at org.bukkit.plugin.java.JavaPlugin.<init>(JavaPlugin.java:66) ~[minecraft_server.jar:git-PaperSpigot-"4c7641d"]
	at com.comphenix.protocol.ProtocolLib.<init>(ProtocolLib.java:65) ~[?:?]
	at java.base/jdk.internal.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method) ~[?:?]
	at java.base/jdk.internal.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:62) ~[?:?]
	at java.base/jdk.internal.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45) ~[?:?]
	at java.base/java.lang.reflect.Constructor.newInstance(Constructor.java:490) ~[?:?]
	at java.base/java.lang.Class.newInstance(Class.java:584) ~[?:?]
	at org.bukkit.plugin.java.PluginClassLoader.<init>(PluginClassLoader.java:77) ~[minecraft_server.jar:git-PaperSpigot-"4c7641d"]
	at org.bukkit.plugin.java.JavaPluginLoader.loadPlugin(JavaPluginLoader.java:129) ~[minecraft_server.jar:git-PaperSpigot-"4c7641d"]
	... 6 more
Caused by: java.lang.IllegalStateException: Initial initialization
	at org.bukkit.plugin.java.PluginClassLoader.initialize(PluginClassLoader.java:126) ~[minecraft_server.jar:git-PaperSpigot-"4c7641d"]
	at org.bukkit.plugin.java.JavaPlugin.<init>(JavaPlugin.java:66) ~[minecraft_server.jar:git-PaperSpigot-"4c7641d"]
	at fr.vmarchaud.mineweb.bukkit.BukkitCore.<init>(BukkitCore.java:72) ~[?:?]
	at java.base/jdk.internal.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method) ~[?:?]
	at java.base/jdk.internal.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:62) ~[?:?]
	at java.base/jdk.internal.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45) ~[?:?]
	at java.base/java.lang.reflect.Constructor.newInstance(Constructor.java:490) ~[?:?]
	at java.base/java.lang.Class.newInstance(Class.java:584) ~[?:?]
	at org.bukkit.plugin.java.PluginClassLoader.<init>(PluginClassLoader.java:77) ~[minecraft_server.jar:git-PaperSpigot-"4c7641d"]
	at org.bukkit.plugin.java.JavaPluginLoader.loadPlugin(JavaPluginLoader.java:129) ~[minecraft_server.jar:git-PaperSpigot-"4c7641d"]
	... 6 more

@Eywek Eywek merged commit 3150a8e into MineWeb:master Dec 20, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Eywek Eywek Eywek approved these changes

Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@Shyrogan @vmarchaud @MaximeMichaud @Eywek