From df1aaab3df4ad8b1a9be283ceae4685bbfead1ea Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?K=C3=A9vin=20Subileau?= Date: Mon, 27 Aug 2018 10:58:15 +0200 Subject: [PATCH] win_nssm: fix incorrect separator in doc for service dependencies The dependencies parameter works with space as separator, but not with comma as shown in the documentation (cherry picked from commit ddd4b4bea69a39e8e2aadac9d23c1f64bd1355f5) --- lib/ansible/modules/windows/win_nssm.py | 4 ++-- test/integration/targets/win_nssm/tasks/tests.yml | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/lib/ansible/modules/windows/win_nssm.py b/lib/ansible/modules/windows/win_nssm.py index f896d1e3f95..2fb8928f0b9 100644 --- a/lib/ansible/modules/windows/win_nssm.py +++ b/lib/ansible/modules/windows/win_nssm.py @@ -58,7 +58,7 @@ options: - Use either this or C(app_parameters), not both. dependencies: description: - - Service dependencies that has to be started to trigger startup, separated by comma. + - Service dependencies that has to be started to trigger startup, separated by space. user: description: - User to be used for service startup. @@ -140,7 +140,7 @@ EXAMPLES = r''' - win_nssm: name: foo application: C:\windows\foo.exe - dependencies: 'adf,tcpip' + dependencies: 'adf tcpip' # Install and start the foo service with dedicated user - win_nssm: diff --git a/test/integration/targets/win_nssm/tasks/tests.yml b/test/integration/targets/win_nssm/tasks/tests.yml index 1ce69a11e71..b38884a4642 100644 --- a/test/integration/targets/win_nssm/tasks/tests.yml +++ b/test/integration/targets/win_nssm/tasks/tests.yml @@ -61,7 +61,7 @@ name: '{{ test_service_name }}' application: C:\Windows\System32\cmd.exe start_mode: manual - dependencies: 'tcpip,dnscache' + dependencies: 'tcpip dnscache' user: '{{ test_win_nssm_username }}' password: '{{ test_win_nssm_password }}' stdout_file: '{{ test_win_nssm_path }}\log.txt'