Как поправить shell скрипт?

Question

[bychok300]

пытаюсь создать лог с текущей датой и запустить питон скрипт в этот лог
не могу понять правильно ли я вызываю переменную _file?

#!/bin/sh

_now=$(date +"%d-%m-%Y-%H-%m")
_file="/home/folder/logs/log_$_now.txt"
sudo touch $_file

cd /home/folder/logs
sudo python3 handleMouseEvent.py > ${_file}

Comments

[sim3x]

ошибка в чем?

[bychok300]

cannot create /home/folder/logs/log_14-03-2017-15-03.txt: Permission denied и питоновский скрипт не запускается, если стираю вызов питона, все нормально работает

[Дмитрий]

sudo в скриптах использовать - сомнительная полезность, зачем он тут?

[sim3x]

ls /home/folder/logs/

[bychok300]

Дмитрий: лог который создается, почему то с правами рута, и по другому в него не запишешь

[Дмитрий]

[me@fedora ~]$ ls -l logs
total 4
-rw-rw-r--. 1 me me 12 Mar  3 10:58 log_03-03-2017-10-03.txt

у меня все ок.

[Дмитрий]

bychok300: потому что ты touch с sudo вызываешь. т.е. создаешь файл пользователем root. если создашь директорию рядовым пользователем, то не будет проблем ни с созданием файла, ни с его модификацией.

[bychok300]

Дмитрий: пробовал без судо
я хочу что бы файл создавался во время запуска компа и сразу начал туда писать
добавил в /etc/rc.local перед exit 0
файл создается от рута, хотя все судо убрал и из-за того что этот лог создался в от рута, пайтон туда stdout перенаправить не может

[Дмитрий]

что у Вас в rc.local? выложите листинг, плз.

[bychok300]

Дмитрий:

#!/bin/sh -e
#
# rc.local
#
# This script is executed at the end of each multiuser runlevel.
# Make sure that the script will "exit 0" on success or any other
# value on error.
#
# In order to enable or disable this script just change the execution
# bits.
#
# By default this script does nothing.
cd /home/me
./myscrip
exit 0

[Дмитрий]

Почему именно rc.local используется? Нужно сценарий выполнить обязательно до загрузки X11? Какой используется дистрибутив?

[bychok300]

Ubuntu 16.04 LTS
Я читал что загрузка из rc.local происходит после запуска системы, мне это показалось оптимальным,
я думал запускать из /etc/init.d/, но тут загрузка происходит на этапе инициализации системы, это мне показалось не столь необходимым

Answer 1

[Дмитрий]

$ ls -ld /home/folder/logs
вернет скорее всего что-то навроде:
drw-r--r-- 2 root root ...
дайте права на запись в папку всем пользователям:
$ sudo chmod 777 /home/folder/logs
и выкиньте sudo из скрипта:

#!/bin/sh

NOW=$(date +"%d-%m-%Y-%H-%m")
PYFILE="/home/folder/logs/handleMouseEvent.py"
LOGFILE="/home/folder/logs/log_$NOW.txt"

python3 $PYFILE > $LOGFILE

должно работать.

P.S. changedir тоже лучше не использовать

Answer 2

[Saboteur]

Зачем вам нужен sudo для создания файла и запуска питона?
Если нужен sudo - запускайте весь скрипт, то есть:

#!/bin/sh

OUTFILE="/home/folder/logs/log_$(date +"%d-%m-%Y-%H-%m").txt"
cd /home/folder/logs
python3 handleMouseEvent.py > ${OUTFILE}

и затем
sudo myscript.sh

Во-вторых, вы делаете перенаправление > а перед этим делаете touch - зачем? перенаправление и так создаст нужный вам файл. Главное, чтобы у юзера, от которого вы запускаете (при sudo - у рута) были права записи в указанной папке.

P.S. В шелл принято именовать переменные в верхнем регистре.