Две переменные подряд в bash

Хаха, вот это я тупанул. Нельзя было использовать $PATH в роли локальной переменной. Сменил на $MOUNTPATH и все заработало)

И кавычки не забывай.

И совсем правильно будет ${mountpath}${file}

И кавычки не забыл)

А кавычки?

В этом случае ненужно.

Не понял тебя

Тут {} вместо них, потому тут кавычки не нужны.

Почему не нужны?

А пробелы куда денутся?

учти, что это неатомарная процедура.

т.е. после того как ты проверил и перед тем как ты создашь файл, какой-то другой процесс (например, другая копия твоего же скрипта) может уже создать такой файл! это популярная проблема конкурентного программирования.

на шелле для атомарного «проверить и создать файл если его нет» я использую вот такую функцию:

function mkatomic
{
  file="$(mktemp)"
  mv -n "$file" "$1"
  if [ -e "$file" ]; then
    rm "$file"
    return 1
  fi
}

она вернет False если файл уже существует.

Скрипт можно сделать короче:

test -f «$FILE» || touch $PATH$FILE

${} - более строгая форма, не?

Хм, пойду man bash курить :)

Я имел в виду если в переменных будут пробелы, touch создаст несколько файлов.

https://stackoverflow.com/questions/13828544/atomic-create-file-if-not-exists...

Хм, да, это я упустил.

Напомнило

Это над удаленной и добавленной строкой 351 народ угорает? Не, ну бывает, что-то с толку сбивает и начинаешь бредить :) Я действительно как-то упустил сабж из виду, сорри.

В 351 посмотри на пробелы внимательно.

иии? вероятно из одних и тех же юзнетов приплыло, трюк-то стар как мир :)

А, усек :)
Ну, бывает же. Только компьютеры не ошибаются и работают круглосуточно.

Это вместо моей строки?

if [ ! -f ${MOUNTPATH}${FILE} ]

И совсем правильно будет ${mountpath}${file}

touch "${mountpath}/${file}"

Лучше добавить разделяющую косую черту. Во-первых, будет работать не зависимо от того, оканчивается косой чертой mountpath или нет. Во-вторых, нагляднее. В-третьих, не принято оканчивать mountpath косой чертой.

Ок, спасибо

Спасибо за сниппет. Возьму на заметку.

да, вместо всего этого if'а. у тебя тогда будет

mkatomic "$FPATH/$FILE"

mktemp может не сработать если tmp загажен весь лучше проверять