Почему не работает скрипт через cron (sh->php)?

Question

[Евгений Иванов]

Вот так скрипт работает нормально
crontab -e

*/1 * * * * /usr/bin/php  /var/www/site/get_data.php >/dev/null 2>&1

(Скрипт пишет данные в БД.)

Но потребовалось запускать скрипт чаще, чем раз в минуту.
Посоветовали делать через sh
crontab -e

*/3 * * * * /var/www/site/get_data.sh >/dev/null 2>&1

get_data.sh

#!/bin/bash
for i in {1..10}
do
/usr/bin/php  /var/www/site/get_data.php >/dev/null 2>&1
sleep 10
done

И скрипт не работает. Данные в БД не появляются.

Почему не работает скрипт?

Comments

[Михаил Коробков]

Какие права доступа на sh скрипте? Выполнение разрешено?
Если руками запустить скрипт, данные появляются?

[Евгений Иванов]

Михаил Коробков, права на get_data.sh 755.
При вызове get_data.sh через браузер скрипт скачивается.
При вызове get_data.php через браузер скрипт выполняется, данные в БД появляются.
(Ровно как и при вызове get_data.php через cron.)

[Михаил Коробков]

Доступ к шеллу есть?
Попробуйте запустить sh скрипт через шелл, если есть возможность.
bash get_data.sh
Возможно происходит какая-то ошибка при выполнении php скрипта, для теста уберите перенаправление выводов в /dev/null
Если в php.ini выключено отображение ошибок, в начале php скрипта пропишите:
ini_set('display_errors', '1');

Ваш sh скрипт я проверил, у меня работает(заменил запуск php скрипта на echo)

[Евгений Иванов]

Михаил Коробков, запустил

bash /var/www/site/get_data.sh
/var/www/site/get_data.sh: line 3: syntax error near unexpected token `$'do\r''
/var/www/site/get_data.sh: line 3: `d'

[hOtRush]

https://askubuntu.com/a/117989

Answer 1

[Михаил Коробков]

Судя по комментарию: Почему не работает скрипт через cron (sh->php)?

Проблема в том, что вы сохранили файл в Windows.
По-хорошему почитать про crlf и lf.
Решение проблемы:
Скрипт небольшой, можете руками переписать сразу на линуксе.(например в nano)

Либо попробуйте воспользоваться решением отсюда:
stackoverflow

awk '{ sub("\r$", ""); print }' get_data.sh > get_data2.sh
mv get_data2.sh get_data.sh

Comments

[Евгений Иванов]

Спасибо, заработало. Использовал apt-get install dos2unix
Хотя можно было и написать в линукс (ctrl+v).

[sash999]

оно конечно дело вкуса,но кмк на седе попроще будет, чем на авк:
sed 's/^M//g' get_data.sh > get_data2.sh
(^M в линуксовом терминале набирается с помощью CTRL-V CTRL-M)

Answer 2

[Radjah]

Вместо записи выхлопа в null сделай дозапись в файл. Еще cron может что-то в системные логи брякнуть.

Comments

[Евгений Иванов]

А по синтаксису ничего не сказать?
Может я где-то что то не дописал в задании крон для sh скрипта?
Или в самом sh скрипте?
Я всё верно написал?

[Radjah]

Евгений Иванов, да вроде всё правильно.
До кучи замени sleep на /bin/sleep или в crontab-файл пропиши PATH=/bin:/sbin:/usr/bin:/usr/sbin перед списком заданий, тогда не надо будет полные пути вводить.