From 31454700828cb48cc751d8bc72f9bd7602111c1e Mon Sep 17 00:00:00 2001
From: Ugo <ugogon@hotmail.de>
Date: Thu, 12 Sep 2019 15:35:21 +0200
Subject: [PATCH] added exit errors in install script and updated README

---
 README.md  |  6 ++++--
 install.sh | 21 ++++++++++++++-------
 2 files changed, 18 insertions(+), 9 deletions(-)

diff --git a/README.md b/README.md
index c3278c7..d497e6f 100644
--- a/README.md
+++ b/README.md
@@ -2,10 +2,12 @@
 
 ## Installationsanleitung
 
-Voraussetzung ist Python in einer Version >=3.3. Normalerweise reicht es einfach folgenden Befehl auszuführen:
+Voraussetzung ist Python in einer Version >=3.3 (bzw. venv). Normalerweise reicht es einfach folgenden Befehl auszuführen:
 
 ```$ ./install.sh```
 
+**INFO:** Manche Debian/Ubuntu basierten Systeme trennen Python3 und venv. Sollte dieser Fehler auftreten installieren Sie bitte venv nach(`apt-get install python3-venv`).
+
 Sollte es Probleme mit der Python-Instanz geben kann diese manuell in der 4. Zeile von `install.sh` geändert werden.
 
 Gestartet wird der Server per:
@@ -18,6 +20,6 @@ Das startet ein Webserver der auf Port 5000 lauscht.
 
 Da mir gesagt wurde overachieving ist u.U. auch nicht gern gesehen (getreu dem Motto: "Zu früh kommen ist auch unpünktlich.") habe ich wirklich nur die Anforderungen implementiert.
 
-Um dennoch zu demonstrieren, dass ich mir Gedanken um die simple Erweiterung von Funktionalitäten gemacht habe, habe ich eine weitere CRUD Funktion auskommentiert hinzugefügt. Diese ist im Branch [delete_edge](https://git.finnendahl.de/Yugon/TU_Assignment/commit/0d5778ada4c85305e4f63ab189c243b299a7ae31) zu begutachten 
+Um dennoch zu demonstrieren, dass ich mir Gedanken um die simple Erweiterung von Funktionalitäten gemacht habe, habe ich eine weitere CRUD Funktion auskommentiert hinzugefügt. Diese ist im Branch [delete_edge](https://git.finnendahl.de/Yugon/TU_Assignment/commit/0d5778ada4c85305e4f63ab189c243b299a7ae31) zu begutachten
 
 Ach ja, und da zu der layout Methode nicht viel gesagt wurde hab ich mich mit force-directed-graphs etwas ausgetobt. Zu sehen per Klick auf Automatic Layout rechts unten.
diff --git a/install.sh b/install.sh
index 5309ece..81c273a 100755
--- a/install.sh
+++ b/install.sh
@@ -8,23 +8,30 @@ cd "$(dirname "$0")"
 version=$($PYTHON3_PATH -V 2>&1 | grep -Po '(?<=Python )(.+)')
 if [[ -z "$version" ]]
 then
-    echo "Given PYTHON3_PATH is no Python3! Please edit paths.config" 
+    echo "Given PYTHON3_PATH is no Python3! Please edit paths.config"
     exit 1
 fi
 
 parsedVersion=$(echo "${version//./}")
 if [[ "$parsedVersion" -lt "330" ]]
-then 
-    echo "Python version must be >=3.3" 
+then
+    echo "Python version must be >=3.3"
     exit 1
 fi
 
 echo "Creating virtual enviroment"
 python3 -m venv venv
-echo "Installing created virtual enviroment"
+if [[ $? -eq 0 ]]; then
+  echo "Successfully created virtual enviroment"
+else
+  exit 1
+fi
 
 echo "Installing dependencies"
 venv/bin/pip install flask
-
-echo "Successfully installed webserver"
-echo "Start webserver with \"venv/bin/python main.py\""
+if [[ $? -eq 0 ]]; then
+  echo "Successfully installed webserver"
+  echo "Start webserver with \"venv/bin/python main.py\""
+else
+  exit 1
+fi