Paper icon indicating copy to clipboard operation
Paper copied to clipboard

Filter plugin.yml's libraries for existing libs

Open velnias75 opened this issue 3 years ago • 5 comments

Is your feature request related to a problem?

In plugin.yml you can define libraries to get downloaded and installed into the libraries folder. I have a plugin that uses this feature to load the adevnture-api which is already included in paper.

Describe the solution you'd like.

It would be nice to have a way to avoid such doubled libraries.

Describe alternatives you've considered.

none

Other

No response

velnias75 avatar Jun 13 '22 05:06 velnias75

Isn't this related to #7955?

Andre601 avatar Jun 13 '22 15:06 Andre601

its related but different. #7955 is allowing the dev to have a different entry point with different settings, one of the benefits would be not downloading libs that paper provides. this issue here suggests to do that automatically without the dev needing to do anything

MiniDigger avatar Jun 13 '22 15:06 MiniDigger

I'm not fond of the notation of having what would potentially be a hardcoded list of libs not to allow loading

electronicboy avatar Jun 13 '22 16:06 electronicboy

Same. It would make sence on one side, but also require more maintenance for devs here.

Andre601 avatar Jun 13 '22 16:06 Andre601

Functionality-wise, #8108 will cover this by being able to use the plugin loader system which allows you to dynamically add libraries during runtime. This will allow you to do any sort of checks if the library already exists.

Owen1212055 avatar Aug 09 '22 02:08 Owen1212055

Resolved by https://github.com/PaperMC/Paper/pull/8108

Owen1212055 avatar Feb 19 '23 14:02 Owen1212055