Also fixes a template path lookup issue when using an include

Fixes #12277

Python 3: use six.string_types instead of basestring

* Turn strings into byte strings before passing to shlex and turn them
  back into unicode strings after they are retyurned from there

  Fixes #12257

Reduces the number of failing tests from 35 to 17.

Python 3: fix TypeError: unorderable types in test

s/rpmbuild/rpm-build

When I followed these instructions, the generated path was 'rpm-build', not 'rpmbuild'. My rpm-build version is rpm-build-4.11.1-25.el7.x86_64 if that's relevant. Maybe this is 'just me', but wanted to feed back in case it's the same for everyone.

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

The full error was

======================================================================
ERROR: test_task_executor_execute (units.executor.test_task_executor.TestTaskExecutor)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/mg/src/ansible/test/units/executor/test_task_executor.py", line 252, in test_task_executor_execute
    mock_action.run.return_value = dict(ansible_facts=dict())
  File "/home/mg/src/ansible/lib/ansible/executor/task_executor.py", line 317, in _execute
    if self._task.async > 0:
TypeError: unorderable types: MagicMock() > int()

----------------------------------------------------------------------

Experiments show that Python 2 MagicMock() > 0 is true, so I'm setting
the async property on mock_task to 1.  (If I set it to 0, the test fails
anyway.)

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