bspkrs / Treecapitator

DaftPVF's Treecapitator mod for Minecraft. Chops down trees in a single blow. Requires bspkrsCore.
http://www.minecraftforum.net/topic/1009577-
42 stars 36 forks source link

Adding the axes from AlexDGr8r's Falling Meteors mod to the default config. #16

Closed airbreather closed 11 years ago

airbreather commented 11 years ago

Not tested at all, since I don't really feel like setting up a build environment for this right now. FWIW, I did painstakingly compare this addition with the Mekanism version and the .cfg file that gets generated for it, so that I'm as sure as I can be about its correctness.

I am not AlexDGr8r, nor have I contacted him about this.

bspkrs commented 11 years ago

Can you confirm that the Mod ID is correct? I'm not 100% positive, but I think FML mod IDs are case-sensitive. Same with the config filename.

airbreather commented 11 years ago

Per this: https://github.com/AlexDGr8r/MeteorsMod/blob/master/src/minecraft/net/meteor/common/MeteorsMod.java#L46 it looks like it is. "meteors.cfg" was what was generated on my box, and this got those axes working for me.

Looks like the modid used to be "Meteors", but it was changed to "meteors" in the update for 1.6.2. Made a comment about that here: https://github.com/AlexDGr8r/MeteorsMod/commit/4818f3684981d9ccccf224984a92bf79c5ff5abf#commitcomment-3998251

Lunatrius commented 11 years ago

It was intentional. If you search through the files you'll notice that "meteors" occurs in quite a few places (resource loading, mcmod.info etc).