Fix rotate_node to return the correct itemstack.

All on_* hooks that return an itemstack are NOT guaranteed to
modify and return the original stack.  We cannot count on the
behavior of the existing builtin definitions not to be overridden
by game/mod logic.

Randomwalk caves: Reduce 'insure' value from 10 to 2 (#9035)

This value reduces how far randomwalk caves can extend into the
mapchunk padding, to avoid 'out of voxelmanip' cave nodes which cause
flattened cave walls.
Testing shows that a value of 2 (instead of 10) is enough to make
'out of area' nodes extremely rare.
Reducing this value results in a higher chance of overlap and
connection with caves of neighbour mapchunks.

Add more visual feedback for button states (#8916)

- Add style properties for overriding the the hovered/pressed state
  - By default, hovered buttons are a lighter version of the base color
  - By default, pressed buttons are a darker version of the base color
- Add hovered bg image support for image buttons (style property)

Continue with 5.2.0-dev

Bump version to 5.1.0

Update minetest.conf.example

Update translation strings

Update from Weblate (hacky)

Fix isDead()/autoforward crash (#9031)

Crash occurred in singleplayer when leaving a world with
autoforward enabled then re-entering a world.

ContentCAO: Fix segfault on quit (attached)

Revert Client::sendPlayerPos optimization (part of 81c7f0a) (#9025)

Comment out the if statement that prevents sending TOSERVER_PLAYERPOS if the player is dead.

Fix upright_sprite lighting when colors are set

fixes #9020

Revert "Fix the bgcolor formspec element (#8716)" (#9018)

This reverts commit 1db3d252cff9e8d61fecf1052d7497813851da51.
Temporary solution until a compatible solution is found to define both - formspec and fullscreen backgrounds.

Fix warnings in guiButton.h

Merge pull request #8166 from Warr1024/fix7020

Fix (or workaround?) for 'Falling nodes with node inventory cause crash when unloaded'

Fix unwanted detaching when damage = 0

Attachments: Fix attachments to temporary removed objects (#8989)

Does not clear the parent's attachment information when the child is deleted locally.
Either it was removed permanently, or just temporary - we don't know, but it's up to the server to send a *detach from child" packet for the parent.

README: Add dependencies for Arch Linux (#8998)

Added Arch Linux dependency list (including base-devel which covers many of them)

All Settings: Set min limit for crtitical settings (#9000)

Document node drop tool filtering by string matching as deprecated (#9009)

Textures: Load base pack only as last fallback (#8974)

label[]: Fix broken colors since 2c9edef

Move Quicktune code to util/ (#8871)

Fix error message caused by adding new parameter to background (#8922)

Adds background9[] element to keep backwards compatibility in formspec prepends.

Allow grep-ing for on_grant and on_revoke (#8958)

Just two code comments are added.

All settings: Fix missing flags checkbox caused by 'possible flags' order (#8997)

Previously, the 'rivers' checkbox was missing for mgcarpathian, caused by the 'possible flags' order: 'caverns,nocaverns,rivers,norivers'.
Also reorder mgcarpathian 'possible flags', but only for consistency.

Simple shader fixes. (#8991)

1. Pass current camera offset to shader, so shader have access to the global coordinates
2. Pass animation timer to fragment shader. C++ code is already there, just wasn't declared in the shader
3. Delay animation timer wrap-around (from 100s to about 16 minutes)

Automatically enable depends of mods when enabling the mods in the world config menu (#3473)

Fix some reference counters (memleak) (#8981)

Fix some reference counters (memleak)
Map::dispatchEvent: Allocation safety using references

Fix broken build

The variable name changed but this didn't cause merge conflicts, so it wasn't caught before.

Rename "private messages" to "direct messages" (#8971)

These messages can be read by server administrators, which makes them
not actually private.

Move the code for meta formspec into bool Game::nodePlacement(const ItemDefinition &selected_def,

Punchwear (improved) (#8959)

Fix AreaStore's IDs persistence (#8888)

Improve documentation
Read old formats
Fix free ID function. Return first gap in map

Wieldhand: Specify which ItemStack to use (#8961)

Makes 'get_wield_item' to return the "main" ItemStack

Add support for per-player FOV overrides and multipliers

Remove incorrect MutexAutoLock

The line declared a variable "m_con" instead of locking m_con.
getClient() doesn't need this anyway, so remove it.

Trigger on_place in many situations even if prediction failed

Also print help commands to chat for server terminal (#8869)

Fix misuse of AsyncTask (#8799)

Settings: Prevent mutex deadlock in remove() (#7803)

Inventory: Properly revert client predictions (#8945)

Caused by incremental inventory sending
Previously everything was overwritten by serializing the entire inventory

Improve undersampling settings

The setting now accepts values between 1 and 8 in the
Advanced Settings menu.
Values 0 and 1 now behave the same way (setting it to 1 won't
disable MSAA anymore), so there's no need to expose 0 as a value.

This closes #8939.

Inventory: Undo prediction on drop

Fix the bgcolor formspec element (#8716)

Fix formspec version backup in prepends losing data

Dungeons: Clean up parameters, improve structure variety (#8918)

While preserving the general character of dungeon structure.
Slightly increase the range of standard room horizontal size, while
preserving the average horizontal size.
Return to classic maximum large room size of 16x16x16.
Make 1 in 4 dungeons have a 1 in 8 chance for each room being 'large',
making multiple large rooms possible for the first time.
Make 1 in 8 dungeons allow diagonal corridors, to make these a little
more common.
Make corridor width vary from 1 to 2, but forced to 2 if diagonal
corridors are allowed, to make them passable.
Add some comments.

Built-in formspecs: Force version 1

client.h: Add missing const's

LocalPlayer: Fix code-style

client.cpp: Fix code-style in Client::sendPlayerPos

Fix LocalPlayer::isDead always returning false if player is immortal

Send ActiveObjects once right after Init2

Formspecs: Introduce formspec_version to mods

CSM: Fix itemstack:get_meta() 'metadata' indexing error

Load CSM environment after the restrictions are known

Safety-guards for CSM callbacks to abort on a bad implementation
Only run callbacks when the mods are loaded (and with it: builtin)

Duplication checks inside constructors

InventoryManager: Fix virtual function overload warning

Send cumulated inventory changes only each step (#8856)

Applies to player and detached inventories

Update minetest.conf.example

Update translation strings

Update from Weblate

Dungeons: Remove non-functional 'projecting dungeons' setting (#8897)

Lua API documentation: Various fixes (#8914)

Remove unusable 'minetest.setting_*' from client_lua_api.txt
lua_api.txt:
- Define the 'mod.conf' format
- More precise 'settingtypes.txt' format reference
- Document special sound files 'player_*damage'
- Group, sort and add 'not_in_creative_inventory' special group
- Define the 'Settings' format
- Warning about incorrect byte saving in 'StorageRef'
- Note about non-persistent player definitions fields
- Better 'leveldiff' and 'level' group description

lua_api.txt: Improve privilege definition (#8912)

Move debug.txt after it grows too big (#8904)

Before opening the file for writing, its file size is tested. If it exceeds 50 MB, it is moved to debut.txt.1, otherwise the log is appended to the old messages. An old debut.txt.1 is removed if it already exists.

label[]: Fix cut-off translated text

Fix Irrlicht 1.9 support

doc/client_lua_api: Clarify how client side api and mods start (#8822)

Force item entities out of solid nodes (simpler) (#8885)

Fix Inventory::moveItemSomewhere()

Inventory: Handle removeItem leftovers (#8884)

'All Settings': Don't use checkboxes for 'no...' mapgen flags (#7847)

Various fixes to lua_api.txt (#8878)

particles.cpp: Fix code-style

Mainly spacing and pointer placement

clientobject.h: Fix code-style

Fix rotation of attached particlespawner

Co-authored-by: ANAND <ClobberXD@gmail.com>

Document world format version 28 (private nodemeta) (#8857)

Document luaentity pitch and roll in world format (#8864)

Clarify docs regarding pointed_thing and get_pointed_thing_position (#8766)

Various settingtypes.txt improvements (#8867)

Improve 'readable' setting names for waving water settings.
Change 'near plane' to 'near clipping plane'.
Document physics settings.

mgv6_spflags:
Change 'new biome system' to 'snowbiomes flag' for clarity.
Add missing 'noflat' flag to default flags.

Inventory: Fix missing setModified calls

Previously inv:remove_item and inv:set_list did not trigger an inventory update

Mapgen v6: Various mudflow code improvements and bugfixes  (#8805)

Calculate 'i' inside 'y' loop to avoid 'i' and 'y' becoming
out of step due to the uses of 'continue'.
Simplify calculation of 'p2d' when coordinates are inverted.
Remove some unnecessary and unreliable 'index out of
voxelmanip' checks.
Move calculation of '&em' out of loops.
For 'Loop further down until not air' code, use y coordinate
to detect being out of voxelmanip, instead of checking index
which did not detect that.
Add and improve comments.
Indent the 'for (;; y--) {' loop.
Improve format and fix codestyle issues.

fontengine: Fix non-gettext builds

Inventory: Send dirty lists where appropriate (#8742)

This change reduces the amount of sent data towards clients. Inventory lists that are already known to the player are skipped, saving quite some data over time.

Raises protocol version to 38 to ensure correct backwards-compatible code.

Add ItemStack:get_description() to get tooltip (#8847)

Improve occlusion culling in corridors with additional check

Restore approximate occlusion check

While less precise, it worked better which is what matters in the end.

Make Mapgen::spreadLight use a queue (#8838)

Fix default hand definition not using wieldhand.png

Occlusion: Check for light_propagates and do mapblock bounds checks

Occlusion: Begin cleanup

Revert "Don't send position update packet if player is dead"

This reverts commit fb6f1fdcbe3ee2321cc33eb00f2c6da86744d00a.

Add player sprite textures back into the engine as a fallback (#8823)

'player.png' and 'player_back.png' are hardcoded into the engine as
the default appearence of the player object, so the textures should
be present in the engine.
This results in a functional fallback sprite player for when a game
has no player appearence related code.

lua_api.txt: Add link to Minetest Modding Book (#8826)

remove_detached_inventory: Fix segfault during mod load

Don't send position update packet if player is dead

Disable autoforward if player is dead

Restore intended functionality to minimap markers (#8819)

Find LuaJIT headers on vcpkg

luajit headers were moved into a subdirectory to avoid conflicts with plain lua

ClientInterface: Use recursive mutex to prevent freeze in on_newplayer() (#8808)