minetest-mods / 3d_armor

Visible player armor & wielded items for minetest
Other
17 stars 39 forks source link

remove pointless dependency on bakedclay #90

Closed fluxionary closed 1 year ago

fluxionary commented 1 year ago

so far as i can tell, bakedclay isn't used by 3d_armor or any of the other mods in the pack.

this will partially fix #89

i'm not entirely certain why the api part of the modpack also depends on content mods (default, nether, moreores, ethereal, fire). i can try to excise those as well, but it seems like there's at least references to them (e.g. as materials).

BuckarooBanzay commented 1 year ago

so far as i can tell, bakedclay isn't used by 3d_armor or any of the other mods in the pack.

yeah, doesn't look like it is used 🤷

$ grep baked . -Rn
./3d_armor/mod.conf:3:optional_depends = player_monoids, armor_monoid, pova, fire, ethereal, bakedclay, moreores, nether
./3d_armor/depends.txt:7:bakedclay?

i'm not entirely certain why the api part of the modpack also depends on content mods (default, nether, moreores, ethereal, fire). i can try to excise those as well, but it seems like there's at least references to them (e.g. as materials).

78 removed some of the dependencies in the api, i think those shouldn't be needed anymore, can't really test right now but let me know if you have some infos 👍