ЧЯДНТ?

может ты хотел вместо таких кавычек (( )) где можно математику счиитать, поставить [[ ]] где различные условия проверять ?

скрипт завершает работу до строчки echo «end»

Почему ты решил, что не доходит?

Вот выхлоп твоего скрипта на файле, в конце строк которого нет пробелов:

└─> bash -x test.sh file 
+ file=file
++ grep -n ' $' -- file
+ spaces=
+ echo end
end
+ (( -n  ))

└─> bash -x test.sh file 
+ file=file
++ grep -n ' $' -- file
+ spaces='1:asd '
+ echo end
end
+ (( -n 1:asd  ))
test.sh: line 11: ((: -n 1:asd : синтаксическая ошибка в выражение (error token is "1:asd ")

не воспроизводится,
а ты пробовал отдельно запускать на боевых данных
grep -n ' $' — $file

может у тебя там фиксируется бинарные данные?

схерали? ищешь $ в конце строки?

таки что хочет ТС? в моем примере ищется «пробел доллар» внутри строки
ему нужно найти строку с пробелом в конце?

Почитай про регулярные выражения (man 7 regex). $ — это метасимвол, означающий конец строки.

ему нужно найти строку с пробелом в конце?

вдумчиво перечитываем ...

Как вариант проблемы: файл $file с виндовозными концами строк.

Да, и это (косвенно) указано в ОП.

Спасибо. Вроде заработало:

    spaces="$(grep -n ' $' -- $f)"
    if [ -n "$spaces" ]
    then 
        echo "error: $f has spaces in the end of line:"
        echo "$spaces"
        exit 1
    fi

echo «error: $f has spaces in the end of line:»

не косвенно, а явно указанно, ну так это ЛОР всегда найдутся ...

мдя ... разглядел

не понятно в чём разница? в круглых/квадратных скобках?

блин, так не работает. пришлось написать так:

    if [ -n "$(grep -n ' $' -- $f)" ]
    then 
        echo "error: $f has spaces in the end of line:"
        echo "$(grep -n ' $' -- $f)"
        exit 1
    fi

if [ -n "$(grep -n ' $' -- $f)" ]

пионеры…

if grep -n ' $' -- "$f"

man grep:

-q, --quiet, --silent Quiet; do not write anything to standard output. Exit immediately with zero status if any match is found, even if an error was detected. Also see the -s or --no-messages option. (-q is specified by POSIX.)

Не надо проверять, нашёл ли grep что-либо, строковыми операциями. Сравнение $? с нулём быстрее и надёжнее.

Сравнение $? с нулём быстрее и надёжнее.

я надеюсь ты не предлагаешь писать что-то типа grep...; if [ $? == 0 ]; then?

Не делай так

При большом $file выжрешь память.

grep -n ' $' "$file" \
| sed -e '1 i\
error: '"${file}"' has spaces in the end of line:\'

Нет, имею в виду if grep -q ...; then .

угу. Просто уточнил.

Кстати, я без -q делаю, удобно отлаживать.

это ты специально ТС путаешь, да? Твою хрень ему без поллитры будет не распарсить.

Одинарные кавычки разве не экранируют метасимволы?

Кстати какую роль здесь выполняют тире "--"?

какую роль здесь выполняют тире "--"?

конец ключей, представь себе имя файла --file

ты специально ТС путаешь?

Посмотри альтернативы - либо загрузить «весь» файл в переменную (кто сказал, что там пара строчек) либо обрабатывать в 2 прохода.

Впрочем, для слабых седом, можно попробовать что-то типа:

grep ... | while read line ; do
 echo WARNING
 echo -- "$line"
 cat
 break
done

Одинарные кавычки разве не экранируют метасимволы?

в grep экранируют. Только надо помнить, что в RE и ERE набор метасимволов разный.

Кстати какую роль здесь выполняют тире "--"?

что-бы можно обрабатывать файлы с именем "---", которые большинство команд воспринимают как ключи.

Посмотри альтернативы - либо загрузить «весь» файл в переменную (кто сказал, что там пара строчек) либо обрабатывать в 2 прохода.

а, ты ещё эту строку с пробелом добавил? Ладно, мой вариант:

sed -ne"/ $/{ierror: $file has spaces in the end of line:" -e"p}" "$file"

лучше даже так:

sed -ne"/ $/{ierror: $file has spaces in the end of line:" -e"p;q1}" "$file"

Этот вариант возвращает error code 1, если есть кривая строка.

Вот так легко и ненавязчиво программу на sh заменили программой на sed. :D

Вот так легко и ненавязчиво программу на sh заменили программой на sed. :D

дык там изначально grep была. Я её на sed поменял, и добавил пару деталей. В этой задаче shell не нужен.

Похоже, что у тебя bash запускает скрипт с ключом -e(остановка после первого ненулевого кода возврата)

echo -- "$line"

printf -- "$line"

printf '%s\n' "$line", дурачок. А то, что ты написал, будет криво выводить строки со знаками процента.

а где написано про '%s\n'? в мане нет.

это я видел. где ты взял '%s\n'? почему такая комбинация "лечит" %? что за s? почему \n в конце?

почему такая комбинация «лечит» %?

Потому что синтаксис printf — printf формат [аргументы], где «формат» — строка, в которой конструкции %что-то заменяются на аргументы (%s — вывести строку «как есть», %.5f — вывести дробное число до пятого знака после запятой, см. man 3 printf), по очереди, а \что-то — на спец. символы (\n — перевод строки, см. man 1 printf).

         ФОРМАТ             АРГУМЕНТ, соответствует "%s", будет выводиться как есть
         v                  v
$ printf 'Hello, %s!\n\n\n' 'world'
Hello, world!


$

и не лень столько inline на него тратить

лучше даже так:

Это выводит только одну проблемную строку(оригинал - заголовок и все - что и было причиной не засовывать в память), и номера строки нет(AFAIK односедово не не решается).

Да, кстати, я не претнендовал на наилучшее решение.