search

unkn0w / noobs

Kolekcja skryptów do szybkiego stawiania usług na serwerach Mikrusa
MIT License
251 stars 64 forks source link

Skrypt instalacji MongoDB #80

Closed kacioRabo closed 3 years ago

kacioRabo commented 3 years ago

Mój pierwszy skrypt, więc z chęcią przyjmę uwagi i wskazówki, jeśli są jakieś błędy. :)

loper commented 3 years ago

Hej,

Ta konstrukcja zawsze wywołuje pierwszy blok "{ ... }" - po co jest więc drugi? Albo co chciałeś osiągnąć w ten sposób? Jeśli sprawdzasz wynik osatniej komendy, no to najpierw "[ "$?" ] && ...", ew. w jednej linii z komendą. Zerknij proszę na pozostałe skrypty - tam jest ta konstrukcja.

kacioRabo commented 3 years ago

Cześć,

pisałem to czytając instrukcję z docs mongodb i w punkcie pierwszym jest, że pierwsza komenda może zwrócić błąd, że gnupg nie jest zainstalowany, więc chciałem, żeby nie przerywał tylko doinstalował gnupg i ponownie zaimportował klucz do repo mogodb. Czy lepiej zrobić tam ifa, że jeśli pierwsza komenda będzie miała error to wtedy druga?

loper commented 3 years ago

Czyli założenia jak wspomniałem. Można zrobić IFem albo skrótowo: jakas_komenda && { co jest ok; ] || { co jesli err; }

kacioRabo commented 3 years ago

Dzięki za podpowiedź. Dzisiaj poprawię i wrzucę.