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

Question

[Вадим]

Здравствуйте,

Скрипт:

#!/bin/bash

filex="*.pdf"
folderp="/path/to/folder"

Когда я добавил команду
find "$folderp" -type f -name "$filex"

она ничего не находит, так как вторая переменная раскрывается с одинарными кавычками '.pdf' Естественно такого файла у меня нет. Не могу передать второй параметр без кавычек.

Конструкция:
find "$folderp" -type f -name "*.pdf"

Работает вполне ожидаемо

с уважением, Вадим

Comments

[Вадим]

folderpath="/pdffolder/"
declare -a fileext
fileext[0]="*.pdf"
mainpath=$folderpath${fileext[0]}

if [ ! -d "$folderpath" ]; then
     # critical error, no pdf folder
     echo "CRITICAL: PDF FOLDER NOT FOUND"
     echo "CRITICAL: PDF FOLDER NOT FOUND" >&2
     exit 2
fi

numberofpdfs=`ls -la $mainpath | wc -l`
if [ "$(($numberofpdfs))" -eq 0 ]
   then
     # critical error, no pdf files 
     echo "CRITICAL: NO PDF FILES FOUND"
     echo "CRITICAL: NO PDF FILES FOUND" >&2
     exit 2
fi

latestfilefullpath=`find $folderpath -type f -name "*.pdf" -printf "%T+\t%p\n" | sort -r| head -n 1`
latestfile=`echo $latestfilefullpath | rev | cut -d"/" -f1 | rev` #removes folders from the path

# the function below will try to find if this pdf file was  modifed less than 7 days and 8 hours ago
numberofnewpdfs=`find $mainpath -type f -name "$latestfile" -newermt '-10560 minutes'| wc -l`

if [ "$(($numberofnewpdfs))" -eq 0 ]
   then
     # not critical error, old pdf files 
     echo "WARNING: LATEST PDF IS MORE THAN 7 DAYS OLD"
     echo "WARNING: LATEST PDF IS MORE THAN 7 DAYS OLD" >&2
     exit 1
fi
echo "PDFs ARE UP-TO-DATE"
exit 0

[Saboteur]

Это ужас ужас...можно укоротить как минимум в два раза и уменьшить дублирование кода. Но это ладно.

Вижу одну логическую ошибку.

Вы везде используете find для поиска pdf, чтобы искать их в подпапках.
Но вот количество PDF файлов определяете через
numberofpdfs=`ls -la $folderpath/$fileext | wc -l`

То есть если у вас в основном каталоге pdf файлов нет, то оно в подкаталогах их не считает, и сразу выйдет с ошибкой.

[Вадим]

Согласен. Но дело в том, что подкаталогов там и нет!

Можно было везде использовать find, но решил что лист будет быстрее, а find только там, где нужно проверить дату модификации.

Answer 1

[Saboteur]

Вообще-то ваша конструкция должна работать и у меня работает вполне корректно.

Можете привести пример вывода и убедиться что хотя бы один pdf файл в нужном каталоге есть?
У вас точно есть *.pdf файлы по указанному пути? Может там file.PDF ?

Comments

[Вадим]

pdf точно есть, вторая же команда сработала и выдала список файлов

[Saboteur]

Так можете привести реальный скрипт, а не искусственный пример?

[Вадим]

привел

[Saboteur]

у вас /pdffolder прямо в корне?

[Saboteur]

Немного поправил ваш скрипт, попробуйте так

folderpath="/pdffolder"
fileext="*.pdf"

function errexit {
  echo "$1"
  echo "$1" >&2
  exit $2
}

if [[ ! -d "$folderpath" ]]; then
  errexit "CRITICAL: PDF FOLDER NOT FOUND" 2 # critical error, no pdf folder
fi

declare -i totalpdfs=$(find $folderpath -name "$fileext"|wc -l)
if [[ $todalpdfs -eq 0 ]]; then
  errexit "CRITICAL: NO PDF FILES FOUND" 2 
fi

LASTFILE=$(find . -type f -name "*.sh" -newermt '-10560 minutes' -printf "%T+ %f\n"|sort -r|head -n 1|cut -d" " -f 2)

if [[ -n "$LASTFILE" ]]; then
  errexit "WARNING: LATEST PDF IS MORE THAN 7 DAYS OLD" 1 # not critical error, old pdf files 
fi

echo "PDFs ARE UP-TO-DATE"

[Вадим]

Красиво! Что дает declare -i ?

[Вадим]

Saboteur, да в корне ! Это смонтированный folder

[Saboteur]

help declare.
объявляется числовая (integer), а не текстовая переменная.
Если работает, можете отметить ответ кнопочками

Answer 2

[jcmvbkbc]

она ничего не находит, так как вторая переменная раскрывается с одинарными кавычками '.pdf'

А у меня всё что надо находит твоим скриптом. Можешь добавить -x к #!/bin/bash чтобы шелл писал, что собирается выполнять, чтобы понять в чём дело. Подозреваю, что какие-нибудь смешные опции в .bashrc/.bash_profile

Comments

[Вадим]

Так и не разобрался, сделаю обход для разного типа файлов через if... fi или через case... isac еще проверю на другой машине !

[Saboteur]

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