Почему не срабатывает sh script?

Question

[Сергей Бурдужа]

Всем привет.
Написал скрипт, который найдет и опитизирует катинку, но почему-то команда для оптимизации не работает.

#! /bin/bash
args=("$@")

if [ ! $# -gt 0 ]; then
    echo "Set one ore more file name to optimize"
    exit 1
fi

for i in "${args[@]}"; do
  file_path=$(find -name "$1*" -type f);
  if [ $file_path ]; then
    echo $file_path;
    echo "file_size before: $(du -sh $file_path)";
    $(jpegoptim --strip-all --all-progressive -ptm 80 $file_path);
    echo "file_size after: $(du -sh $file_path)";
  fi
done

И вывод

$ jpg.sh big-image.jpg
./test/some/level3/big-image.jpg
file_size before: 44K	./test/some/level3/big-image.jpg
/home/serii/xubuntu/bin/jpg.sh: line 14: ./test/some/level3/big-image.jpg: Permission denied
file_size after: 44K	./test/some/level3/big-image.jpg

Причему тут permision denied?

Права к файлу jpg.sh уже существует

~/xubuntu/bin on  master! ⌚ 10:04:34
$ ll
Permissions Size User  Date Modified Name
.rwxrwxr-x   771 serii  3 Aug 12:36  dir.sh*
.rwxrwxr-x  1.1k serii  2 Aug 11:57  file.sh*
.rwxrwxr-x   282 serii  7 Aug  9:10  file_multiply.sh*
.rwxrwxr-x   595 serii  2 Aug 17:53  git.sh*
.rwxrwxr-x  1.3k serii  7 Aug  9:11  git_c.sh*
.rwxrwxr-x   408 serii  7 Aug 10:03  jpg.sh*
.rwxrwxr-x   545 serii 31 Jul 17:49  mg.sh*
.rwxrwxr-x  1.8k serii  7 Aug  9:10  pug.sh*
.rwxrwxr-x   302 serii  1 Aug 11:40  rename_fonts.sh*
.rwxrwxr-x   263 serii  2 Aug 10:43  t.sh*
.rwxrwxr-x  2.7k serii  7 Aug  9:32  wp.sh*

Заранее благодарен.

Comments

[Сергей Бурдужа]

Justa Gain,
permision denied у картинке.
А зачему картинке permision?
У меня есть функция jpg() которая делает тоже самое с одной картинкой.

jpg collection.jpg
collection.jpg 1920x3136 24bit P JFIF  [OK] 466200 --> 466200 bytes (0.00%), skipped.
Average compression (1 files): 0.00% (0k)

Я не написал в шапке, но использую оболочку zsh, а в скрипте прописано bash.

Answer 1

[Vitaly Karasik]

./test/some/level3/big-image.jpg: Permission denied

Речь о правах к файлу с картинкой.

Comments

[Сергей Бурдужа]

Можете пояснить а какие права нужны файлу и зачем?
В терминале, когда ввожу команду jpgeoptim никаких прав не нужно.

[Сергей Бурдужа]

Justa Gain,
Знаю про права, я вопрос задал, но так и не удостоился ответа.
Почему картинке права не нужны когда запускаю оптимизацию из терминала, а из скприпта нужны и какие и зачем?

Я не понимаю, что так сложно ответить?

[Сергей Бурдужа]

Justa Gain, Уже без функции jpg.

Код из jpg_optim.sh

for i in "${args[@]}"; do
  file_path=$(find -name "$1*" -type f);
  if [ $file_path ]; then
    echo $file_path;
    echo "file_size before: $(du -sh $file_path)";
    $(jpegoptim --strip-all --all-progressive -ptm 80 $file_path);
    echo "file_size after: $(du -sh $file_path)";
  fi
done

И в терминале:
- оптимизация картинки через jpgeoptim напрямую с парамертами
- ошибка при оптимизайии с помощью скрипта
- права на картинку

~/Downloads ⌚ 10:53:43
$ jpegoptim --strip-all --all-progressive -ptm 80 home.jpg
home.jpg 1920x720 24bit N JFIF  [OK] 671034 --> 133789 bytes (80.06%), optimized.
Average compression (1 files): 80.06% (525k)

~/Downloads ⌚ 10:54:09
$ jpg_optim.sh home.jpg
./home.jpg
file_size before: 132K	./home.jpg
/home/serii/xubuntu/bin/jpg_optim.sh: line 14: ./home.jpg: Permission denied
file_size after: 132K	./home.jpg

~/Downloads ⌚ 10:54:21
$ ll
Permissions Size User  Date Modified Name
.rw-rw-r--  133k serii  7 Aug 10:53  home.jpg

[AUser0]

Сергей Бурдужа, я бы посоветовал убрать $( ); вокруг запуска программы. И проверьте права доступа к файлу ./test/some/level3/big-image.jpg (и к директории с этой картинкой), программа в скрипте прямо об этом и сообщает.

[Сергей Бурдужа]

AUser0, Спасибо заработало

[Ипатьев]

AUser0, проверять не надо жи.
программа, как вы правильно заметили, пытается выполнить результат работы jpegoptim, отчего и возникает ошибка. то есть убрать $( ); было достаточно

[Сергей Бурдужа]

Ипатьев,
Спасибо, у меня первые шаги в написании скриптов на bash, и не знал, что бывают ситуации, когда нужно убрать $()

На курсах говорят, но именно и нужно ставить $() когда хочешь выполнить команду из терминал внутри скрипта, видать есть исключения.

[Vitaly Karasik]

Спасибо всем помогавшим (я заседал в Zoom, не мог ответить)!

Сергей Бурдужа - насчет $(). Это дает возможность получить вывод команды, например.

OUTPUT=$(echo "hello world")
echo $OUTPUT

- напечатает hello world.

То есть проблема, которую я не заметил, была что скрипт пытался запустить вывод команды jpegoptim.

[AUser0]

Ипатьев, вах, и правда - натурально исполняет! Блин!

[Сергей Бурдужа]

Vitaly Karasik,
Спасибо, теперь стало понятно.

[Saboteur]

На курсах говорят, но именно и нужно ставить $() когда хочешь выполнить команду из терминал внутри скрипта, видать есть исключения.

Капец, что у тебя за курсі.
$() это command substitution, то есть выполнить команду и результат подставить на это место.
Примеры, где это может быть полезно:

$ whoami
user1
$ echo "Hello, $(whoami)"
Hello, user1
$ myname=$(whoami)
$ echo $myname
user1

[Сергей Бурдужа]

Saboteur, Спасибо

[Ипатьев]

И, соответственно, если написать без echo, просто $myname (или как в вопросе, сразу $(whoami) без присваивания переменной) то оболочка воспримет user1 из $myname как команду. И напишет, что такой команды нет.
А в случае с картинкой файл есть, но у него нет разрешения на выполнение