Как передать все файлы из папки для команды?

Question

[historydev]

Нужно чтобы команда отработала для всех файлов в папке назначения с расширением etc/nginx/locations/*.locations

./replace_location_port.sh -n test -v v2.2.2 -p 2222 -f etc/nginx/locations/*.locations

while getopts n:v:p:f: flag
do
    case "${flag}" in
        n) name=${OPTARG};;
        v) version=${OPTARG};;
        p) port=${OPTARG};;
        f) file=${OPTARG};;
    esac
done

if [[ -z $name || -z $version || -z $port || -z $file ]]; then
  echo "Required params not passed: -n: $name -v: $version -p: $port -f: $file"
  exit
fi

echo "$file" # Здесь выводит первый файл из папки

location="$name\/$version"

awk -i inplace '/location \/projects\/'"$location"'/,/}/ {if(/proxy_pass/) sub(/:[0-9]+.;/,":'"$port"';")} {print}' "$file" # Команда применяется только к нему

P.S: можно так, если варианта проще нет:

search_dir=./etc/nginx/locations
for entry in "$search_dir"/*.locations
do
  echo "$entry"
done

Answer 1

[Rsa97]

find etc/nginx/locations/*.locations -exec ./replace_location_port.sh -n test -v v2.2.2 -p 2222 -f {} \;

Comments

[historydev]

Это проще выглядит, меньше букв:

for entry in $file
do
  awk -i inplace '/location \/projects\/'"$location"'/,/}/ {if(/proxy_pass/) sub(/:[0-9]+.;/,":'"$port"';")} {print}' "$entry"
done

./replace_location_port.sh -n test -v v2.2.2 -p 2222 -f etc/nginx/locations/\*.locations

Почему не работает так-же как с ls *.locations?
ls etc/nginx/locations/test.*.locations

[Rsa97]

historydev, А почему должно работать так же, если ls - это команда вывода списка файлов, в которой заложен перебор всего, что попадает под маску?

[historydev]

Rsa97,

Если передать без экранирования, передаётся первый найденный файл, а если экранировать \*.locations то получаю awk ошибку.

Подумал что должен уметь такое, он ведь работает с stdin, по идее если вызвать что-то вроде $(ls etc/nginx/locations/*.locations) вместо "$file" для awk сработает, сейчас проверю)

Answer 2

[historydev]

- awk -i inplace '/location \/projects\/'"$location"'/,/}/ {if(/proxy_pass/) sub(/:[0-9]+.;/,":'"$port"';")} {print}' "$file"
+ awk -i inplace '/location \/projects\/'"$location"'/,/}/ {if(/proxy_pass/) sub(/:[0-9]+.;/,":'"$port"';")} {print}' $(ls $file)

./replace_location_port.sh -n who-am-i -v v2.50.6 -p 222 -f etc/nginx/locations/\*.locations

Comments

[AUser0]

Забыли только упомянуть, что при вызове команды нужно экранировать etc/nginx/locations/\*.locations

Ну или правильно обрабатывать множественные аргументы (man glob), которые появляются из-за * в командной строке:

while getopts "n:v:p:f:" flag
do
    case "${flag}" in
        n) name=${OPTARG};;
        v) version=${OPTARG};;
        p) port=${OPTARG};;
        f) file=${OPTARG}
            until [[ $(eval "echo \${$OPTIND}") =~ ^-.* ]] || [ -z $(eval "echo \${$OPTIND}") ]; do
                file+=" $(eval "echo \${$OPTIND}")"
                OPTIND=$((OPTIND + 1))
            done
        ;;
    esac
done

[historydev]

AUser0, да, вспомнил про это буквально через минуту, потом отвлёкся и забыл, сейчас поправлю)

[historydev]

AUser0,

Ну или правильно обрабатывать

Слишком сложно выглядит, костыльнул вот так:

ls $filePath || exit 1

awk -i inplace '/location \/projects\/'"$location"'/,/}/ {if(/proxy_pass/) sub(/:[0-9]+/,":'"$port"'")} {print}' "$(ls $filePath)"

[AUser0]

historydev, тогда хотя-бы не мусорите в stdout
ls $filePath >/dev/null 2>&1 || exit 1

[AUser0]

historydev, хотя, что тут сложного? getopts же не сложно использовать?

[historydev]

AUser0,

Не мусорить да, хотелось бы, не понял как это сделать, спасибо!

getopts я с гугла взял, пока не хочу в скрипты углубляться, не уверен что мне это нужно, изучил всё что было нужно для реализации и всё.

Они мне нужны для ci/cd конвейера, неделю пишу уже это всё, устал порядком:

[AUser0]

historydev, ну, ваше право право костылить на каждом повороте, ваше решение.
И примите мои соболезнования, на костылях да по граблям, не легко наверное так жить...

[historydev]

AUser0,

Не на каждом, а на лишних.
Сейчас мне нужен работоспособный прототип, остальное не важно.

[AUser0]

historydev, скрипт с особенным написанием аргументов - это как раз НЕ лишний поворот!
Для нормально работающего скрипта, со стандартным написанием аргументов, достаточно добавить 3 (три!) строчки в скрипт - и будет работать. Будет работать с двумя и более именами файлов, будет работать с любым количеством glob-ов... Интересно, как вы это накостылите?

Упс, что это я? Пытаюсь перевербовать уверенного костыльщика... Ваше право, ваш путь.

[historydev]

AUser0,

Я никуда не вербовался, не понимаю с чего столько эмоций)

Я не работаю на постоянной основе с баш скриптами, чаще всего упаковываю длинные команды.

Сейчас исключение и мне нужен был быстрый результат, я его получил.

Изучать как они устроены у меня нет желания и времени.

Костыльщик - это человек который работает с этим постоянно и осознанно юзает всякий калл, ему плевать на поддержку.

А я мимо проходил, собрал из кусков нужный функционал и использую.

Поддерживать это только мне и если будет нужно, я сделаю как нужно попросив помощи например у вас.

Отвечать не нужно, бредом занимаемся, я всё сказал - вас услышал.