Запустить/убить процесс одним скриптом

[winhex]

Для запуска клиента openvpn используется такая строчка:

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

sudo openvpn --daemon --config config.ovpn --askpass passfile

Нужно, чтобы один скрипт при запуске проверял, запущен ли openvpn, и в случае если нет, запускал его, а если запущен, убивал.
Заранее спасибо

[SLEDopit]

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

[ -n "$(pidof openvpn)" ] && kill -9 $(pidof openvpn) || sudo openvpn --daemon --config config.ovpn --askpass passfile

[winhex]

SLEDopit ,спасибо, но я сделал как-то так:

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

if [ "$(pidof openvpn)" ]
    then
    killall openvpn
    else
    openvpn --daemon --config config.ovpn  --askpass passfile
fi

Какой вариант более правильный?

[SLEDopit]

Какой вариант более правильный?

Оба.
Просто вы полностью расписали конструкцию, а я - нет.
По сути if [ smth ] then comm else comm fi то же самое, что и [ smth ] && comm || comm. Просто разные записи.

[watashiwa_daredeska]

По сути if [ smth ] then comm else comm fi то же самое, что и [ smth ] && comm || comm. Просто разные записи.

Нет, не то же самое.

Shell

$ if true; then false; else echo fail; fi $ true && false || echo fail fail

Собственно, конструкцию для перезапуска можно записать так:

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

kill $(pidof openvpn) || openvpn --daemon --config config.ovpn --askpass passfile

[t.t]

Какой вариант более правильный?

Оба.
Просто вы полностью расписали конструкцию, а я - нет.
По сути if [ smth ] then comm else comm fi то же самое, что и [ smth ] && comm || comm. Просто разные записи.

Не совсем то же самое. В Вашем варианте третья команда выполнится, если _любая_ из первых двух завершится с ошибкой:

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

t $ if true; then false; else echo 'catch you!'; fi
t $ true && false || echo 'catch you!'
catch you!

[t.t]

Нет, не то же самое.

Shell

$ if true; then false; else echo fail; fi $ true && false || echo fail fail

Синхронно мыслим. (:

[KiWi]

По сути if [ smth ] then comm else comm fi то же самое, что и [ smth ] && comm || comm. Просто разные записи.

Нет, не то же самое.

Shell

$ if true; then false; else echo fail; fi $ true && false || echo fail fail

Собственно, конструкцию для перезапуска можно записать так:

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

kill $(pidof openvpn) || openvpn --daemon --config config.ovpn --askpass passfile

pkill openvpn

P.S.: если совсем точно -- pkill -x openvpn.

[Ленивая Бестолочь]

pkill openvpn

типа не проверяет, вдруг нет такого. типа не круто :-)

[watashiwa_daredeska]

типа не проверяет, вдруг нет такого. типа не круто :-)

Проверяет. Если такого нет или нет прав на убиение, то возвращает ненулевой статус.

[Ленивая Бестолочь]

ах ты черт, а я думал он ругнётся.