Помогите отладить автоконвертер видео на bash плиз

Question

[Евгений Елизаров]

Господа, пытаюсь написать скриптик, который бы проверял наличие уже сконвертированного файла в заданной папке и если его нет — кодировал. НАверное по коду будет проще понять что я имею ввиду.
pastebin.com/C7aG2LEQ

Проблема в 12 строке, почему то проскакивает проверку существующий файл, в итоге пытается кодировать его повторно, что в общем то совершенно и не нужно мне делать, ибо файлов там навалом. Подскажите плиз в чём ошибка

Answer 1

[sledopit]

М, а как это у вас между строчкой if и then ещё какие-то команды затесались?

if [[ ! -e i.mp4 ]]
then  
echo "$i.mp4 not exist" 
echo "Converting "$i"." 
/usr/local/bin/ffmpeg -i ${video_folder}$i.mkv -acodec aac -strict experimental -ab 128k -ac 2 -vcodec libx264 -b 1200k -s 1280x720 -r 30 -threads 0                 ${films_folder}$i.mp4 ; echo "Convert "$i" to mp4 is done!"
fi

Так всё ок.

Ну и я ещё позанудствую чуток:

for i in `ls -a $video_folder/*$file_mask*.mkv`

Никогда так не делайте, слышите? Никогда! Вдруг у вас в имени пробелы и всякие непечатыемые символы?

for i in $video_folder/*$file_mask*.mkv

Вот так правильно.

i=$(echo $i | sed 's/.mkv//' | sed 's%^.*/%%')

омг. на фига тут sed?

echo ${i/.mkv}

прекрасно со всем справится.

Ну и else с shift'ом тут совершенно не нужны.

Comments

[sledopit]

А, ну и забыл совсем: "$i" "${i}" "${i/.mkv}" надо не забывать брать в кавычки. А то если пробел в имя файла запишется, то кирдык команде. Ошибку выплюнет.

Answer 2

[Petr0vich]

Зачем два раза квадратные скобки?

Answer 3

[egorinsk]

Bash-скрипт в 99% случаев должен начинаться с set -e. Странно, что никто не замечает таких вопиющих ошибок.

Answer 4

[BrainHacker]

Если у вас сообщение "${films_folder}$i.mp4 not exist" выводит правильное имя файла, то попробуйте перенести строки 13 и 14 внести под «then»:

if [[ ! -e "${films_folder}$i.mp4" ]]
then
    echo "${films_folder}$i.mp4 not exist"
    echo "Converting "$i"."

    /usr/local/bin/ffmpeg -i ${video_folder}$i.mkv -acodec aac -strict experimental -ab 128k -ac 2 -vcodec libx264 -b 1200k -s 1280x720 -r 30 -threads 0 ${films_folder}$i.mp4

    echo "Convert "$i" to mp4 is done!"
else
    shift
fi