Closed ferexio closed 3 years ago
Hejo hejo jako może propozycję dodałbym żeby te numery portów dodawać bezpośrednio w trakcie odpalenia skryptu albo jako $1, $2. Na przykład ./chce_domoticz.sh 99 789
. A jeżeli nie będzie chociaż jednego albo więcej portów to powinien pojawiać się helper i tam napisać użytkownikowi czego nie dodał i jak ma uruchomić skrypt.
Zmieniłbym jeszcze żeby skrypt był 755 chmod +x
. Dodatkowo także ze względów eterycznych dodałem wcięcia w IFie, dodatkowo także dodałem apt install
bo testowałem skrypt na dockerowej instancji i bo brakowało paczek. Dodałem też do cd /opt/domoticz
jakby skrypt nie mogł utworzyć folderów etc.
diff --git a/scripts/chce_domoticz.sh b/scripts/chce_domoticz.sh
old mode 100644
new mode 100755
index cf8196d..e275ad4
--- a/scripts/chce_domoticz.sh
+++ b/scripts/chce_domoticz.sh
@@ -18,13 +18,18 @@ EOF
)
#===== Checker =====
if [ -z "$1" ]; then
-echo "Poprawna składnia: ./chce_domoticz.sh port_http port_https"
+ echo "Poprawna składnia: ./chce_domoticz.sh port_http port_https"
elif [ -z "$2" ]; then
-echo "Poprawna składnia: ./chce_domoticz.sh port_http port_https"
+ echo "Poprawna składnia: ./chce_domoticz.sh port_http port_https"
else
+
+#==== Pobranie potrzebnych paczek ====
+apt update
+apt install libusb-0.1-4 libcurl3-gnutls tar wget -y
+
#===== Script =====
mkdir /opt/domoticz
-cd /opt/domoticz
+cd /opt/domoticz || { echo 'Folder nie istnieje'; exit; }
wget --inet4-only https://releases.domoticz.com/releases/release/domoticz_linux_x86_64.tgz
tar -xzvf domoticz_linux_x86_64.tgz
rm domoticz_linux_x86_64.tgz
To skoro już paczki dorzucamy to jeszcze nie zapomnij dodać apt install lsb do której domoticz się też może doczepić :)
Oki doki można dodać bo ja tylko sprawdzałem czy ogolnie domoticz działa i czy wystartuje ;) Możesz zmiany dodać do PR bo ja nic nie mogłem dodać na tym PR
Akurat tego bitu wykonywalności na githubie nie potrafię jeszcze ustawiać więc nie dam rady z tym nic zrobić. Może ktoś mnie oświeci jak to się robi? :D
Wydaje mi się, że możesz to dodać w swoim commitice jako Ty! No chyba, że @unkn0w albo @loper mają lepszy pomysł :D
Wydaje mi się, że możesz to dodać w swoim commitice jako Ty! No chyba, że @unkn0w albo @loper mają lepszy pomysł :D
Mogę pobrać całe repo i nanieść zmiany od razu w mainie, także tym się nie przejmuj 👍
@loper no to rób merge bo potrzebuję tego pulla do Hacktoberfest :D