-
Notifications
You must be signed in to change notification settings - Fork 4
Mixin complains about missing mixin host service #13
Comments
Hi! Any update on this issue ? Trying to find the cause, works fine in dev env but not in production. |
There are no updates right now. #12 or should look into pointing Mixin to continuing using LegacyLauncher reliably. If you know a proper fix, then PR is appreciated. I haven't used Orion myself for a while, hence haven't had much interest in fixing it. |
I found the cause ! Need to test more to ensure all is working, after a lot of step debugging i found that the error was caused by a missing library from // Mixin required deps
downloadDep(group: 'org.ow2.asm', name: 'asm-analysis', version: '6.2')
downloadDep(group: 'org.ow2.asm', name: 'asm-util', version: '6.2')
downloadDep(group: 'org.ow2.asm', name: 'asm-commons', version: '6.2') This was working in dev because the deps where present due to gradle sub-dependencies, in production this was causing If all of this works i'll make a PR 👍 |
Aren't org.ow2.asm dependencies being pulled as a transitive dependencies from |
In ideal whole Gradle stuff should get an overhaul, writing everything into Kotlin and making API a lot lighter (dropping Shuriken etc)... |
Agree yes, transitive dependencies are not downloaded properly for mixin apparently, maybe related to PicoMaven config. Apart of that i successfully ported the |
Sticking with Orion was my best option instead of re-writing everything you did a good job here ! |
Lol didn't noticed that PicoMaven is your project, there is an issue open mikroskeem/PicoMaven#5 I see you worked on it, maybe Orion uses an old version. |
Orion does indeed use older version. However as PicoMaven needs more work then it should be dropped right now, and an alternative solution should be used (shading or getting Gradle to spit out all dependencies with direct links + checksum) |
Also i noticed an issue when loading Dynmap plugin under Orion launcher context, it seems that plugins can't access This is the log:
Issue seems present only for plugins that access minecraft classes |
This is a completely different issue, does not belong here. Anyway,
That's correct? Spigot and inherently Paper relocate nms and cb packages.
However if dynmap works fine without Orion then there are some issues again with LegacyLauncher classloading I bet... |
Thanks for the answer, yes different issue, i messaged you on discord to not bloat the issue. 👍 |
The text was updated successfully, but these errors were encountered: