Merge pull request #6841 from s0undt3ch/develop

Fix bug introduced in fcfe3a9 when changing argspec to use the named tuple
Merge pull request #6834 from jacksontj/develop

Move fileclient local hashing to use utils
Keep old behaviour if `gitfs_root` is not set or is set to an empty string.
strip both spaces and * from Group names
Merge pull request #6840 from s0undt3ch/hotfix/fix-gitfs-root

Keep old behaviour if `gitfs_root` is not set or is set to an empty string
Merge pull request #6836 from basepi/contains6817

Make sure text is a string for file.contains, Fix #6817
improve group mgmnt in user.present on Windows
Set groups and full name properly first run
Flush spooled temporary files before reading back the data contained.
Merge pull request #6842 from UtahDave/fix_windows_user.present

Fix windows user.present so groups are set correctly the first run
Add debug print to check if communicate is doin' the right thing.
Add more information on overriding providers

As noted in IRC, if you are using a custom linux spin or derivative, it
may not be apparent which provider to use when overriding providers in
the minion config file. This commit adds a massive table for the pkg,
service, user, and group providers which explains how they are used, to
help guide the user to the correct decision when specifying a provider
override.
Fix sphinx build warning
Added hgfs values to config
Fixed not returning virtual
move dom import to try/except.

Windows barfs on from xml.dom import minidom
Add virtual function so it doesn't blow up windows
Add virtual function to not blow up Windows
Add csv to items that are blowing up Windows. :)
Change to HAS_ALL_IMPORTS for clarity