Update start.sh (#2147)

* Update start.sh

- the codes cds into its folder which makes it possible to run script from any directory

- saves the return code from previous commands to a variable and returns it at exit. Without saving the exit code into a variable exit $? return the return code of the echo statement which is always 0.

* doc: add comment explaining why we cd into the parent folder in start.sh

* set the parent of the script as current location

---------

Co-authored-by: Pamela Fox <[email protected]>

Author: yavuzkaymak

app/start.ps1

@@ -1,3 +1,6 @@
+# set the parent of the script as the current location.
+Set-Location $PSScriptRoot
+
 Write-Host ""
 Write-Host "Loading azd .env file from current environment"
 Write-Host ""

app/start.sh

@@ -1,5 +1,9 @@
 #!/bin/sh
 
+# cd into the parent directory of the script, 
+# so that the script generates virtual environments always in the same path.
+cd "${0%/*}" || exit 1
+
 cd ../
 echo 'Creating python virtual environment ".venv"'
 python3 -m venv .venv
@@ -9,9 +13,10 @@ echo "Restoring backend python packages"
 echo ""
 
 ./.venv/bin/python -m pip install -r app/backend/requirements.txt
-if [ $? -ne 0 ]; then
+out=$?
+if [ $out -ne 0 ]; then
     echo "Failed to restore backend python packages"
-    exit $?
+    exit $out
 fi
 
 echo ""
@@ -20,19 +25,21 @@ echo ""
 
 cd app/frontend
 npm install
-if [ $? -ne 0 ]; then
+out=$?
+if [ $out -ne 0 ]; then
     echo "Failed to restore frontend npm packages"
-    exit $?
+    exit $out
 fi
 
 echo ""
 echo "Building frontend"
 echo ""
 
 npm run build
-if [ $? -ne 0 ]; then
+out=$?
+if [ $out -ne 0 ]; then
     echo "Failed to build frontend"
-    exit $?
+    exit $out
 fi
 
 echo ""
@@ -44,7 +51,8 @@ cd ../backend
 port=50505
 host=localhost
 ../../.venv/bin/python -m quart --app main:app run --port "$port" --host "$host" --reload
-if [ $? -ne 0 ]; then
+out=$?
+if [ $out -ne 0 ]; then
     echo "Failed to start backend"
-    exit $?
+    exit $out
 fi