From b75f7cc22bdf8366669e253363f099d1378962fa Mon Sep 17 00:00:00 2001
From: gfrank <gfrank@chathamfinancial.com>
Date: Thu, 20 Aug 2015 13:57:03 -0400
Subject: [PATCH] Move comments to the documentation

---
 windows/win_nssm.ps1 | 9 ---------
 windows/win_nssm.py  | 6 ++++++
 2 files changed, 6 insertions(+), 9 deletions(-)

diff --git a/windows/win_nssm.ps1 b/windows/win_nssm.ps1
index f99eec66f0a..841bc3aa3fd 100644
--- a/windows/win_nssm.ps1
+++ b/windows/win_nssm.ps1
@@ -39,10 +39,6 @@ If ($params.state)
 {
     $state = $params.state.ToString().ToLower()
     $validStates = "present", "absent", "started", "stopped", "restarted"
-    
-    # These don't really fit the declarative style of ansible
-    # If you need to do these things, you can just write a command for it
-    # "paused", "continued", "rotated"
 
     If ($validStates -notcontains $state)
     {
@@ -138,11 +134,6 @@ Function Nssm-Install
         [string]$application
     )
 
-    #note: the application name must look like the following, if the directory includes spaces:
-    # nssm install service "c:\Program Files\app.exe" """C:\Path with spaces"""
-    #see https://git.nssm.cc/?p=nssm/nssm.git;a=commit;h=0b386fc1984ab74ee59b7bed14b7e8f57212c22b for more info
-
-
     if (!$application)
     {
         Throw "Error installing service ""$name"". No application was supplied."
diff --git a/windows/win_nssm.py b/windows/win_nssm.py
index 5647c60b5a3..86adacf1a7c 100644
--- a/windows/win_nssm.py
+++ b/windows/win_nssm.py
@@ -39,8 +39,10 @@ options:
   state:
     description:
       - State of the service on the system
+      - Note that NSSM actions like "pause", "continue", "rotate" do not fit the declarative style of ansible, so these should be implemented via the ansible command module
     required: false
     choices:
+      - present
       - started
       - stopped
       - restarted
@@ -50,6 +52,10 @@ options:
   application:
     description:
       - The application binary to run as a service
+      - Specify this whenever the service may need to be installed (state: present, started, stopped, restarted)
+      - Note that the application name must look like the following, if the directory includes spaces:
+      - nssm install service "c:\Program Files\app.exe" """C:\Path with spaces"""
+      - See commit 0b386fc1984ab74ee59b7bed14b7e8f57212c22b in the nssm.git project for more info (https://git.nssm.cc/?p=nssm.git;a=commit;h=0b386fc1984ab74ee59b7bed14b7e8f57212c22b)
     required: false
     default: null