messaging.sh: line 29: [: missing `] '

Nie wiem, czy to jest zła rzecz, ani co to znaczy. Mój skrypt nadal działa poprawnie, ale czy powinienem to naprawić?

#!/bin/sh
#This script will send text and maybe images to other computers via ssh and scp.
#Configuration files in same folder

source /Users/jacobgarby/Desktop/messaging/messages.cfg
TIME=$(date +"%H:%M:%S")
CONNECTED[0]="mainmini@192.168.1.65"

if [ -d messages.log ]; then
    :
else
    touch messages.log
fi

read MSG

if [ "$MSG" == "!help" ]; then
    echo ; echo "!clear   Clear's your personal chat log."
    echo "!ban [usrname]    Prevents a user from entering this chat IN DEV."
else
    echo "$TIME | $USER | $MSG" >> messages.log; echo   >> messages.log; echo   >> messages.log
    tail messages.log
fi

for CONNECTION in CONNECTED; do
    echo "It works"
done

if [ "alerttype" == "notification"]; then
    osascript -e 'display notification "You have recieved a message!" with title "Message"'
else
    osascript -e 'display dialog "You have recieved a message!" with title "Message"'
fi

bash bash-scripting

— Jacob_
źródło

Należy do witryny Linux / Unix

— Michael Durrant

Ciąg alerttype nigdy nie będzie taki sam jak łańcuch notification. Być może masz na myśli [ "$alerttype" = "notification" ]?

— Charles Duffy

(i tak, to znaczy =, nie == - patrz specyfikacja POSIX pod adresem pubs.opengroup.org/onlinepubs/9699919799/utilities/test.html i zobaczysz, że obsługiwana składnia porównania łańcuchów to s1 = s2; == jest rozszerzeniem bash i nie wszystkie systemy operacyjne i powłoki będą go obsługiwać; jeśli chcesz użyć operatora porównania o rozszerzonym bashu zamiast standardowego operatora POSIX, [[ ]], nie [ ] ).

— Charles Duffy

Tak, odkąd to opublikowałem, zdałem sobie z tego sprawę

— Jacob_

Tego rodzaju błędy można łatwo znaleźć za pomocą: shellcheck.net

— SnakeDoc

Odpowiedzi:

messaging.sh: linia 29: [: brak ']'

Używasz następujących elementów:

if [ "alerttype" == "notification"]; then`

Jednak powyższemu poleceniu brakuje a space przed ], powinno być:

if [ "alerttype" == "notification" ]; then
                                  ^

Podstawowe zasady warunków

Kiedy zaczynasz pisać i używasz własnych warunków, są pewne zasady, które powinieneś znać, aby uniknąć błędów, które są trudne ślad. Oto trzy ważne:

Zawsze zachowuj odstępy między nawiasami a rzeczywistym sprawdzeniem / porównaniem. Następujące elementy nie będą działać:

if [$foo -ge 3]; then

Bash narzeka na „brakujące”] „”.

Źródło Warunki w skryptach bash (jeśli instrukcje)

— DavidPostill
źródło

Myślę, że Steven właśnie to powiedział, ale masz rację

— Jacob_

Tak, odpowiedzieliśmy mniej więcej w tym samym czasie ... Trwałem dłużej, ponieważ chciałem sprawdzić i zacytować odniesienie do poprawnej składni:

— DavidPostill

Wiem, że wybrałbym obu, ale nie mam wystarczającej reputacji

— Jacob_

Wydaje się, że jesteś w połowie drogi do przywileju „komentarz”;)

— DavidPostill

To jest bardziej złe niż to. Tak jak [ "alerttype" == "notification" ], porównuje dwie stałe, co oznacza nigdy Mów prawdę. Być może jeden koniec lub druga strona powinny być zmiennym odniesieniem? Również, == nie jest w specyfikacji POSIX test; aby być przenośnym, musi być =.

— Charles Duffy

Brakuje ci jednego miejsca.

#BEFORE
if [ "alerttype" == "notification"]; then
#AFTER
if [ "alerttype" == "notification" ]; then
#                                 ^

Inny przykład:

$ if [ "a" == "a"]; then echo "yes"; else echo "no"; fi
-bash: [: missing `]'
no

$ if [ "a" == "a" ]; then echo "yes"; else echo "no"; fi
yes

— Steven
źródło

To jest bardziej złe niż to. Tak jak [ "alerttype" == "notification" ], porównuje dwie stałe, co oznacza nigdy Mów prawdę. Również, == nie jest w specyfikacji POSIX test; aby być przenośnym, musi być =.

— Charles Duffy

Brakuje miejsca przed ] Inną opcją formatu jest:

$ [ "a" == "a" ] && echo "yes" || echo "no"

— Michael Durrant
źródło