проверка обновлений скриптом

[mykolaq]

Всем привет!
Может на форуме и есть, но почему-то не нашел.a =
Есть такой скрипт для проверки обновления по для mikrotik. Но он не работает, при этом отдельно команды

Код: Выделить всё

/system package update check-for-updates

,

Код: Выделить всё

/system package update download

и отсылка email работает. Направьте как мне отследить где косяк

Код: Выделить всё

:local email "emailhere"
#####
:if ( [/system package update get status] != "Downloaded, please reboot router to upgrade it" ) do={
/system package update check-for-updates
:delay 3
:if ( [/system package update get installed-version] != [/system package update get latest-version] ) do={
/system package update download
:delay 20
/tool e-mail send to=$email subject="Mikrotik updates"  body="Available update for RouterOS! New version - $[/system package update get latest-version]. Info - http://www.mikrotik.com/download"
 }

[DmNuts]

Может, проблема в отсутствии закрывающей скобки } в первом :if?
У меня отрабатывает.

[Dragon_Knight]

DmNuts, о какой скобки? все скобки на месте.
Скорее всего не работает из за комментариев, которые ведут себя очень неадекватно.

[DmNuts]

В смысле, вот так правильно:

Код: Выделить всё

:if ( [/system package update get status] != "Downloaded, please reboot router to upgrade it" ) do={
/system package update check-for-updates
}

[podarok66]

Значит так, по всей видимости вы начинаете проверку после перезагрузки. На этот момент

Код: Выделить всё

/system package update get status

не имеет статуса вообще. То есть вы получаете пустое значение. Для того, чтобы отрабатывало, надо команду

Код: Выделить всё

/system package update check-for-updates

дать в скрипте раньше, чем

Код: Выделить всё

/system package update get status

Понимаете, о чем я?
Самая простая проверка моих слов - запустите после перезагрузки в терминале такую связку

Код: Выделить всё

{ global PACUP [/system package update get status]; :put $PACUP}

А потом после еще одной перезагрузки такую

Код: Выделить всё

{/system package update check-for-updates ; global PACUP [/system package update get status]; :put $PACUP}

[mykolaq]

спасибо, сейчас все попробую, но думаю что все починю =)

[mykolaq]

да, все работает, всем спасибо за помощь. впредь буду внимательнее

[kreotoDr]

да, все работает, всем спасибо за помощь. впредь буду внимательнее

Добрый день!
Если не трудно сбросьте полный листинг скрипта...

[podarok66]

Вы что, издеваетесь? Ну всё же в теме прописано. Что ещё-то надо? Внимательно прочтите тему и поправьте скрипт из стартового поста в соответствии с рекомендациями.