Also fixes a couple of bugs that popped up when using the proxy class

Update intro_windows.rst

Fix typo that broke RST -> html parsing.

added Manjaro as an Archlinux family

Update doc to use 'become' instead of 'sudo'

Fixes #12245

Potential fix for #12260

Potential fix for amazon linux's rpm doing something strange with the python_sitelib macro #12166

Fix rsync connections to IPv6 addresses

Similar to https://github.com/ansible/ansible/pull/11816 we can unconditionally
wrap the host address in square brackets. This is required by rsync for IPv6
addresses.

Required some rewiring in inventory code to make sure we're using
the DataLoader class for some data file operations, which makes mocking
them much easier.

Also identified two corner cases not currently handled by the code, related
to inventory variable sources and which one "wins". Also noticed we weren't
properly merging variables from multiple group/host_var file locations
(inventory directory vs. playbook directory locations) so fixed as well.

Fixes #12191

Use parse_kv to correctly parse parameters from password lookup

Fixes https://github.com/ansible/ansible-modules-core/issues/1968

Become clean

simplify become testing and handling, we had drifted and were doulbe checking prompt, become and become_pass
fixed tests to conform to new signature and now tests both with and w/o password
now we are more explicit about self.prompt

remove closing connections after every task, this goes against conneciton caching and was not expected behaviuor nor inhertited from v1

remove closing connections after every task, this goes against conneciton caching and was not expected behaviuor nor inhertited from v1

Python 3: use six.text_type instead of unicode

Fixes the bug I introduced in my previous commit (six.string_types is
(basestring,) on Python 2).  Thanks @abadger for noticing!

Replace 'unicode' with six.text_type, everywhere but in module_utils.

Go back to using ~/.ansible/cp as the ControlPath

Fixes #12198
Fixes #12241

later

This was commented out earlier because of the lack of interprocess
locking and prepare_writeable_dir in v2.

The locking was not needed: it could only protect against other siblings
of this process (since they were all locking a temporary file that was
opened in the parent), and those would be running as the same user and
with the same umask. Also, os.makedirs() tolerates intermediate paths
being created by other processes. For any other kind of error, both
locking and non-locking code paths would fail in the same way.

So all we really need to do is make sure we have write permissions.

(We also move the cp_dir handling code to where we actually set the
ControlPath ourselves; if the user has set it via ssh_*args already,
we don't need to bother.)

Fixes #12213