minetest-mods / magma_conduits

A minetest mod that adds vertical magma conduits to mapgen
Other
5 stars 4 forks source link

attempt to call field 'parse_lines' (a nil value) #10

Open HeyITGuyFixIt opened 4 years ago

HeyITGuyFixIt commented 4 years ago

When I use this mod with the namegen mod (and its undeclared dependency respawn), magma_condiuts attempts to call field "parse_lines" from the namegen mod in volcanoes.lua on line 126. However, this does not exist in the namegen mod. Is there another namegen mod not in the Content DB that provides this?

rusty-snake commented 4 years ago

Hi, I discovered the same bug. There are at least there namegen mods. https://gitlab.com/ksangeelee/namegen https://gitlab.com/cronvel/mt-namegen (contentDB) https://github.com/FaceDeer/namegen (linked from magma_conduits and dfcaverns)

Using the last (which in not in the contentDB) it works.

QuoteNat commented 3 years ago

rusty-snake's solution worked for me.

FaceDeer commented 3 years ago

I've actually just renamed my namegen mod to name_generator and got it included in contentDB, I've been meaning to update all my various mods that use it to the new name. Thanks for commenting here, that reminded me to do this.

FaceDeer commented 3 years ago

Oh, looks like I already did this. This issue should be resolved. @QuoteMasterN , I assume you were using an older version of magma_conduits somehow? The latest version should be referencing the mod name_generator, which is the correct namegen mod now over on contentDB.