neos-modding-group / NeosModLoader

A mod loader for Neos VR
GNU Lesser General Public License v3.0
61 stars 22 forks source link

disable loading from subdirectories #101

Open art0007i opened 1 year ago

art0007i commented 1 year ago

I find this feature annoying more often than I find it useful. For example if I want to test something with only a single mod and temporarily disable every other mod I would just take all the current mods and throw them into a different folder. The easiest option would be just a subdirectory within mods called like "disabled" and then take them back out of there later.

I don't know if anyone at all uses the subdirectories feature at all. I assume it's there to allow better organization but it actually prevents me from organizing mods how I want (right now I need to put a disabled mods directory OUTSIDE the mods folder which gets kind of annoying to navigate).

A good middle ground would be to have some way to specify a folder shouldn't be searched by the mod loader so for example a dot. That way I could make a folder called ".disabled" and anything inside of there would be ignored, but also I think completely removing this feature is a good option, unless there are actually people who use that to organize their mods.

XDelta commented 1 year ago

I personally don't use this either and found it some amount annoying as I'd sometimes do the same. The main benefit I suppose would be for mods that are packaged as zips or otherwise have multiple files associated with them can be grouped together in a folder.

EIA485 commented 1 year ago

maybe we could add an option to nml's config to disable it. if we want to be generic then we could maybe make this a regex string for what sub directories to ignore

zkxs commented 1 year ago

For reference, the PR that added recursive mod loading: #51

I think adding a NML config option to opt out of this behavior would be fine.

art0007i commented 1 year ago

I'd say ignoring folders with a dot at the beginning is the best option. tying this behavior to config would mean that any mod managers/updaters have to respect that option and it could get messy.

Or just revert the PR :)