Clean bash script.

src/main/bash/codiki.bash

@@ -2,31 +2,29 @@
 
 workspace='/opt/codiki'
 codikiScreenName='codiki'
-lockFile='/bin/lock'
+logFile='codiki-'`date '+%Y-%m-%d_%H-%M-%S'`'.log'
-deadFile='/bin/dead'
 screenRegex='codiki.\(([0-9]{2}/){2}[0-9]{4} [0-9]{2}(:[0-9]{2}){2}\)'
 
+# Colors
+RED='\033[0;31m'
+GREEN='\033[0;32m'
+NC='\033[0m'
+
 start() {
-    screen -dmS $codikiScreenName java -jar $workspace/bin/codiki.jar
+    screen -dmS "$codikiScreenName" java -jar $workspace/bin/codiki.jar &>> "$workspace/log/$logFile"
-    if [ $? -eq 0 ]
+    if [ $? -ne 0 ]
     then
-        date > $workspace$lockFile
-        rm -f $workspace$deadFile
-    else
         echo "Service failed to start..."
         exit 1
     fi
 }
 
 stop() {
-    if [ $(status_by_screen) -eq 1 ] && [ -f $workspace$lockFile ]
+    if [ $(status_by_screen) -eq 1 ]
     then
         screen -dr $codikiScreenName -X -S quit
-        if [ $? -eq 0 ]
+        if [ $? -ne 0 ]
         then
-            rm -f $workspace$lockFile
-            date > $workspace$deadFile
-        else
             echo "Service failed to stop..."
             exit 1
         fi
@@ -36,24 +34,20 @@ stop() {
 }
 
 status() {
-    if [ -f $workspace$lockFile ] && [ $(status_by_screen) -eq 1 ]
+    if [ $(status_by_screen) -eq 1 ]
     then
-        tput setaf 2; cat << EOF
+        echo -e " codiki.service - The Codiki web app server"
- codiki.service - The Codiki web app server
+        echo -e "    Active: ${GREEN}active${NC} (running)"
-    Active: active (running) since $(cat $lockFile)
-EOF
         echo ''
     else
-        tput setaf 1; cat << EOF
+        echo -e " codiki.service - The Codiki web app server"
- codiki.service - The Codiki web app server
+        echo -e "    Active: ${RED}inactive${NC} (dead)"
-    Active: inactive (dead) since $(cat $deadFile)
-EOF
         echo ''
     fi
 }
 
 restart() {
-    if [ -f $workspace$lockFile ]
+    if [ $(status_by_screen) -eq 1 ]
     then
         stop
     fi
@@ -79,6 +73,7 @@ case "$1" in
         ;;
     *)
         echo "Usage: $0 {start|stop|status}"
+        echo ''
         exit 1
 esac
 exit 0