minetest-tools / mcresconvert

Convert Minecraft Resource and Texture Packs to Minetest Texture Packs
35 stars 14 forks source link

readme and script update #15

Closed ensonic closed 7 years ago

ensonic commented 8 years ago

Tried to convert a bunch of packs. A few small patches below to update the README and improve the script a bit.

ensonic commented 8 years ago

Sorry, I don't understand. I did not change anything w.r.t a default directory. The script is using '~/.minetest/textures' which is the default for the local mintest dir. It would not write the e.g. /usr{/local,}/share/minetest/ even if you run it as root.

sofar commented 8 years ago

The script is written to fail when ~/.minetest does not exist. It is done so to prevent writing textures in folders that are irrelevant (in case people have installed minetest differently).

If you mkdir -p ~/.minetest/textures, then it is 100% guaranteed that the folder is not going to be found by any existing minetest installation on the system.

This is the reason why it currently is fatal. Making it non-fatal should at least print out a significant warning message.

ensonic commented 7 years ago

Please take another look.