Buildsystem/CMake: Drop a FATAL_ERROR-message if the git binary is not found or outdated.
Allow for it to be pointed to through the ADVANCED settings in the CMake GUI if the above is true.
Core/Sockets: Always try to send MSG_NOSIGNAL in peer().send()

Fixes RASocket::authenticate crash

"MSG_NOSIGNAL:
If you send() to a remote host which is no longer recv()ing, you'll typically get the signal SIGPIPE. Adding this flag prevents that signal from being raised."

Closes #5040
Thanks to @derex for the hint
DB/NPCs: Set trigger flags_extra to a bunch of quest credit markers

God damn Aokromes, always making me do work and stuff...
Core/RealmSocket: Remove a not required #ifndef MSG_NOSIGNAL / #define MSG_NOSIGNAL 0

Somewhere in ACE overloads of send(), flags are default'ed  to 0 if not provided
Buildsystem/CMake: Allow building the core without the gitchecks, this is done in the GUI, or through the DISABLEGITCHECK parameter on *nix-platforms.
DO NOTE THAT WE WILL NOT GIVE SUPPORT ON ANY CORE-VERSION COMPILED WITH THIS FLAG, AS WE CAN NOT EFFECTIVELY FIND OUT WHAT REVISION YOU ARE ON
Buildsystem/CMake: Fix an issue with MSVC not finding git-binary when updating the revisionhash/revisiondate + rename DISABLEGITCHECK parameter to WITHOUT_GIT (to adhere to our standard)
Also cleaned up the checks slightly + moved git-specific testing to its own macro (minor changes to shaurens original patch applied)
Buildsystem/CMake: Forgot to change variable in main CMakeLists.txt
Buildsystem/CMake: Correct a few mistakes regarding WITHOUT_GIT, and add a better warning during configure-phase.
Buildsystem: Do not print warning that user needs to do a git pull with tags if configured to not use git at all
Scripts/Icecrown Citadel: Fixed possible crash for Necrotic Plague (if cast outside of ICC)

Closes #9188
Core/Spells: Fixed charge effects with non-explicit targets
Core/Loot: Only 3 items of the same entry can drop on 25 man raid difficulties and 1 everywhere else (non-raid or 10 man)
Core/Loot: Minor change to previous commit, thanks Vincent_Michael for reminding me about this method in Map class
Core/RA: Fix a possible crash

Caused when RASocket::handle_close (event-driven) would delete the underlying object before RASocket::commandFinished callback was executed for that object. Dereferencing freed pointers is bad.
Core/RA: Addition to previous crashfix

To make sure it also works for sessions that use more than 1 command
before closing.
Core/LFG: Fix a crash

Fix a crash in case a player is recognized as in a LFG dungeon, but without a valid group
Core/Handlers: Fix a crash in HandleCharFactionOrRaceChange

Ensure there's always valid character name data present, even for
characters that were manually inserted into the database, by adding
character name data on char enum if needed.
Core/Handlers: Prevent some cheating in CharacterHandler

Prevents factionchanging and char customising of characters that do not
belong to the current account.
Core/Handlers: Missing returns after KickPlayer() call
Core/Maps: Prevent creating maps without a valid Map.dbc entry

Closes #9181