Add moonflower mod

[RealBadAngel]

Adding flower that opens only at night
Source of fluorescent dye (to come with separate patch)

[Zeg9]

This mod is so great...
Fluorescent dye is a good idea, maybe wait for it before merging this?

[RealBadAngel]

The dye have to be separate patch because it will modify dye mod. I want rebuild crafting of dyes to use mortar + pestle item instead of cookin (which is an absurd). We dont have to follow stupid mc ways.

[PilzAdam]

Why? This just adds (another) flower that is kinda useless since we already can get all dyes.
It also doesnt change anything at all in the gameplay, so why should this be merged?

[RealBadAngel]

This adds an original content, and makes way to obtain fluorescent dye. Nothing like this is aviable in game atm.
So, THATS WHY.
And ofc it willl be useseless without further usage of it. Thats my point.
Collect flowers that glow at night, get the dye, craft dye +glass for example to get glowing glass...

[PilzAdam]

If we follow this as a guideline to merge mods upstream then we would have to take the whole Mod Release forum and throw it in. This doesnt end good.
Content should not just be added because its "original" or "not MC like". It should make a game together with the other things.
Also, I dont see anything about new dye or glowing glass in this pull request.

[HybridDog]

I think here it would be better to use PseudoRandom instead of math.random.

[Zeg9]

@HybridDog (don't know where I should write you that): could you reopen your pull requests about obsidian brick and lavacooling in minetest_game ?

[HybridDog]

@Zeg9 Thanks for remembering me.

[HybridDog]

@Zeg9 Now I need to tell you that I haven't downloaded your obsidianbrick texture and the ompldr doesn't work.

[Jordach]

#169 does this already and has glowing dye + glowing mese.

[HybridDog]

#175 (comment)

[C1ffisme]

This dosen't need the "duplicate" label anymore. The duplicate has been closed.

[paramat]

Lovely idea but i think a mod thing, 👎 from PilzAdam and myself. Other devs opinions? shall we close this now?

[MirceaKitsune]

Years ago when I implemented the initial Moonflower, it was intended for inclusion into minetest_game. It wanted it to be an unique item that gave the default Minetest a personality of its own, similar to what redstone or zombie pigmen are for Minecraft.

With this considered, it's of course the choice of other developers whether it's good for default MTG or not. Personally I think it works best as a rare item that spawns occasionally on specific biomes.

[RHRhino]

It's ridiculous that minetest devs need YEARS to decide for pull requests and issues - moonflower mod is just one example for that ... 😠

[MirceaKitsune]

@RHRhino Things do happen a bit slowly, I agree. I can understand caution in some decisions, but Minetest could advance a lot faster if pull requests were solved sooner.

[RHRhino]

A bit slowly? This pull request is from March 2013 - as slow as a dead snail would be more appropriate! I think it's just fustrating for people who try to help to develope minetest if they have to wait ages just to get their pull request merged or a clear answer that it doesn't fit into minetest or minetest_game.

[paramat]

Yes this is why today i'm giving old PRs some attention. The other MTGame devs seem to be very busy at the moment so i'm doing a lot of dev work and asking for approvals from them.

[HybridDog]

This pull request is from March 2013 - as slow as a dead snail would be more appropriate!

l think a dead snail disappears in less then a year.
If pull requests of people don't get attention, the people don't make new ones, e.g. because of possible conflicts.

[rubenwardy]

I like this, 👍
I like Jordach's PR more, however.

[paramat]

Hi @RealBadAngel just notifiying you because it's over a year since your last comment.

[RHRhino]

@rubenwardy what does Jordach's PR have different?

[paramat]

Closing due to age of PR and 2 👎