Решено, абы как: Не работает скрипт

[azsx]

Есть скрипт на баш

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

!/bin/bash
 #$stroka=$(ps aux | grep svyaz | wc -l)
 $stroka=$(ps aux | grep svyaz |wc -l)

 if [ "$stroka" > 1 ]
 then {
         echo "yes"
 }
 else {
         echo "no"
         bash ./svyaz.sh &
 }
 fi;

Суть скрипта в этой команде:

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

ps aux | grep svyaz |wc -l

Команда выводит число запущенных процессов и не дает им запускаться если уже один скрипт svyaz запущен. Проблема заключается в том, что на одном компьютере скрипт работает, а на втором компьютере не рабтает. То есть условие всегда не равно, всегда идет запуск скрипта. Отличием обоих компьютьеров смог найти только в одном, команда

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

sudo aptitude show coreutils
 Пакет: coreutils
 Пакеты первой необходимости: да
 Состояние: установлен
 Автоматически установлен: нет
 Мульти-архитектура: сторонняя
 Версия: 8.21-1ubuntu5.4

На компьютере где скрипт не работает выдает ubuntu5.4, а на котором работает выдает ubuntu5.3.
Что делать?

[Bizdelnick]

Ошибка здесь:

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

$stroka=$(ps aux | grep svyaz |wc -l)

надо

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

stroka=...

Ну и см. передать в kill id процесса

[SLEDopit]

if [ "$stroka" > 1 ]

это тоже работать не будет. if [ "$stroka" -gt 1 ] нужно.

и скобки фигурные не нужны

[Vascom]

А ещё неправильно использовать сравнение ">" внутри одинарных квадратных скобок. Во-первых, таким образом сравниваются не числа, а значения в соответствии с алфавитным порядком таблицы символов ASCII. Во-вторых, даже в этом случае знак ">" надо экранировать.

Числа сравниваются либо через "-gt" внутри одинарных квадратных скобок, либо через ">" внутри двойных круглых скобок. Вот http://www.bash-scripting.ru/abs/chunks/ch07s03.html

[yars]

А еще, если научите ваш svyaz.sh создавать lock-файлы и при запуске делать

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

 if [ -f $LOCKFILE ]; then
  echo "Another instance of $SCRIPTNAME is running, remove the lock and try again."
  exit 1
fi

то все вообще хорошо будет. man lockfile в помощь.

[azsx]

борюсь со скриптом. Поражает, что на более старой версии coreutils работает первый вариант скрипта, а вот на втором компе приходится велосипедить. Сейчас работает такой вариант:

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

#!/bin/bash
stroka=$(ps aux | grep svyaz | wc -l)
echo "Всего зафиксировано процессов $stroka"

if  [ $(ps aux | grep svyaz | wc -l) -le 4 ]
then {
        echo "yes"
        bash ./svyaz.sh &

}
fi;

Соотвественно переменная и echo используется просто для красоты, работает всё с условия. Непонятности...
В строке [ $(ps aux | grep svyaz | wc -l) -le 4 ] я в квадратных скобках отверяю условие, если оно истинно - то запускаю скрипт svyaz, если ложно - то не запускаю его. Верно? Совсем не понимаю, почему на этом компьютере отверяется именно с 4, чтобы определить наличие только одной копии скрипта svyaz командой ps. То есть если я ставлю 2 (как на компьютере со старой coreutils), то срабатывания на истину не происходит. Вот почему так?

[Vascom]

Введи ps aux | grep svyaz в консоли и увидишь.

[azsx]

ввожу

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

ps aux | grep svyaz | wc -l
2

и вижу два. Вполне логично, 1 процесс реальный скрипт, второй сам ps.
Но новый скрипт работает только при значении в 4 (четыре).

[Vascom]

Тогда сделай вывод этого значения из скрипта. Как называется твой скрипт? Может там тоже есть svyaz в названии и это значение равно 3?
И выводи без | wc -l, чтобы видеть конкретные строки, так понятнее будет.

[Bizdelnick]

Вот почему так?

см. передать в kill id процесса

Не надо никогда, ни при какой погоде, пытаться парсить вывод таких утилит, как ps или ls. Они на это не рассчитаны.

[azsx]

Действительно, я совсем забыл, что скрипт с которого запускаю проверку также имеет svyaz в имени. Как мог появится четвертая строчка не понятно, но, наверное, регистрируется svyaz запускаямая от cron. То есть если я вручную запускаю скрипт из консоли, то надо проверять 3. А если по крон - то 4.
Загадкой также остается почему на более старой версии coreutils работает первоначальный вариант.
Пофиг, решил оставить рабочее решение как есть.
зы
Bizdelnick спасибо, я читал вашу ссылку только не понял. Смысл в том, что ps не рабочий? Я вот как то слабо понимаю как под мой пример ваш способ приспособить.
У меня задача, надо сделать аналог радмин неубиваемый, если убьют процесс. В моем способе плохо только то, что может цифра в х11 сместиться, то есть:

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

x11vnc -usepw -display :1

один на конце, а не ноль как по умолчанию. Но процессы на самом деле будут убивать крайне редко.

[MiK13]

А не лучше ли вместо ps aux | grep svyaz использовать pgrep -lx svyaz ?
Правда, недавно обнаружил, что pgrep в jessie почему-то работает не так, как в предыдущих версиях Debian. А именно, если раньше для выдачи полной строки (с аргументами) надо было использовать -f, то в jessie надо использовать -a