unkn0w / noobs

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

Add script chce_go.sh. Update scripts without +x priviliges #145

Closed cloudziu closed 2 years ago

cloudziu commented 2 years ago

Dodałem skrypt instalujący Go. Podniosłem również uprawniena w skryptach które nie miały +x.

loper commented 2 years ago

Brakuje sprawdzania sudo, jak w innych skryptach. Taka definicja jest niebezpieczna: 'GO_INSTALL_PATH="/usr/local"', szczególnie jak potem używasz "rm -r". Ustawianie "GO_VERSION" dałoby się zautomatyzować, żeby nie zmieniać za każdym razem w kodzie (ew. chociaż pytać usera o pożądaną wersję). wget "https://dl.google.com/go/$(curl https://golang.org/VERSION?m=text).linux-amd64.tar.gz" https://stackoverflow.com/questions/51933636/url-for-golang-latest-stable-release

loper commented 2 years ago

Sypie błędem podczas instalacji jako root: grep: /home/root/.profile: No such file or directory scripts/chce_go.sh: line 38: /home/root/.profile: No such file or directory Dodaje Go do PATH w pliku /home/root/.profile

cloudziu commented 2 years ago

@loper Podesłałem poprawki

Taka definicja jest niebezpieczna: 'GO_INSTALL_PATH="/usr/local"', szczególnie jak potem używasz "rm -r".

Zrezygnowałem z tego i zhardcodowałem ścieżki instalacji.

Ustawianie "GO_VERSION" dałoby się zautomatyzować, żeby nie zmieniać za każdym razem w kodzie (ew. chociaż pytać usera o pożądaną wersję).

Przyzwyczajenie żeby nie ustawiać wersji dynamicznie 😅, ale faktycznie w tym przypadku to może być wskazane. Wersja pobierana jest przez curl jak sugerowałeś.

Sypie błędem podczas instalacji jako root:

Poprawione, obsłużone są następujące przypadki:

loper commented 2 years ago

Ale przekombinowane :D Patrz to: https://bencane.com/2013/09/16/understanding-a-little-more-about-etcprofile-and-etcbashrc/