Fix the basic templating system such that when the template ends in '$', life continues as normal.

lib/ansible/runner/__init__.py

@@ -399,12 +399,9 @@ class Runner(object):
                 actual_host = delegate_to
 
         try:
-            transport = None # use Runner setting
-            if delegate_to and actual_host in [ '127.0.0.1', 'localhost' ]:
-                transport = 'local'
             if actual_port is not None:
                 actual_port = int(actual_port)
-            conn = self.connector.connect(actual_host, actual_port, transport=transport)
+            conn = self.connector.connect(actual_host, actual_port)
             if delegate_to:
                 conn.delegate = host
 

lib/ansible/runner/connection.py

@@ -40,12 +40,11 @@ class Connection(object):
         self.runner = runner
         self.modules = None
 
-    def connect(self, host, port, transport=None):
+    def connect(self, host, port):
         if self.modules is None:
             self.modules = modules.copy()
             self.modules.update(utils.import_plugins(os.path.join(self.runner.basedir, 'connection_plugins')))
         conn = None
-        if transport is None:
         transport = self.runner.transport
         module = self.modules.get(transport, None)
         if module is None:

lib/ansible/utils.py

@@ -246,6 +246,8 @@ def _varFind(text):
     if start == -1:
         return None
     var_start = start + 1
+    if var_start >= len(text):
+        return None
     if text[var_start] == '{':
         is_complex = True
         brace_level = 1

test/TestUtils.py

@@ -19,6 +19,12 @@ class TestUtils(unittest.TestCase):
 
         assert res == 'hello world'
 
+    def test_varReplace_trailing_dollar(self):
+        template = '$what $who $'
+        vars = dict(what='hello', who='world')
+        res = ansible.utils.varReplace(template, vars)
+        assert res == 'hello world $'
+
     def test_varReplace_multiple(self):
         template = '$what $who'
         vars = {