-
Notifications
You must be signed in to change notification settings - Fork 57
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.
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
Relocate shaded libraries to solve third party conflicts #177
Comments
Please file this as a bug, not a feature request. Also, I'd like to say the solution is not quite difficult. At the minimum, a new major version will be required, but a version increment does not translate to difficulty of implementation. Relocating shaded dependencies, even within a library such as eco, is entirely possible. |
It's extremely easy, I do it already with Caffeine and many other libs that are used purely internally with eco. It's about compatibility |
I think you might have forgotten to implement this. Opening up the Eco plugin jar, I find the following shaded libraries:
No relocation is applied to this libraries. The build file confirms this: Line 163 in d826a9f
It's no longer a problem in LibertyBans (whatsoever), however I thought you might find this information useful. |
this really should be fixed in a way, maybe a special relocation prefix only to kotlin and use that prefix on all eco plugins, as we have lots of issues with different plugins because of such behavior, of course we solve them, but it's not a very pleasant experience This is a big minus that stops us from buying other eco depending plugin and makes us think of removing Talismans to lower such issues in future |
Linked to A248/LibertyBans#164.
This is no easy feat haha.
The text was updated successfully, but these errors were encountered: