помогите с циклом в баш.

sdio

- скрипт исполняемый?
- вставь в начало #!/bin/bash -x и посмотри что не так

# bash -xx whilecappor.sh 
+ lease=/var/lib/dhcpd/dhcpd.leases
+ read line
++ grep -E lease 'lease 10.46.12.151 {'
++ sed 's/lease\ //'
++ sed 's/.$//'
grep: lease 10.46.12.151 {: No such file or directory

Не могу понять почему он этот наркоманский смайлик как второй файл хочет взять.

Не смайлик, а всю строку

В $line у тебя строчка из файла, а не имя файла.

я понял. то есть он хочет всё что после do применить ко всем строчкам???
а как мне ему сказать что 1-ый греп выполни для 1-ой строки, 2-ой греп для 4-ой строки и т.д ?

awk'ом с ключевыми словами, например

И вообще, покажи образец данных и то, что нужно получить

добавь echo перед grep. Xoтя зачем там вообще греп?

ipaddr=$(echo $line | grep -E "lease" | sed 's/lease\ //' | sed s/.$//)

eщё sed можно сократить: sed -e выражение1 -е выражение2

ipaddr=$(echo $line | grep -E "lease" | sed 's/lease\ //' | sed s/.$//)

у грепа второй аргумент - имя файла

у грепа второй аргумент - имя файла

опциональный

а по существу ОП есть, что сказать?

хм? Я вроде написал по существу.

глазки раскрой ;)

у ТСа второй аргумент есть, но он явно не файл в контексте скрипта.

так что, твой камент про опциональность не привнёс сюда ни капли конструктива ;)

Я так понимаю, для каждого такого блока надо получить находящиеся внутри него данные?

оп, читает файл построчно и вторым аргументом у него идёт строка, а не файл.

о чем и речь. маны открывай быстро и решительно.

Я так понимаю, для каждого такого блока надо получить находящиеся внутри него данные?

да. Моя проблема в том, что он каждый греп выполняет для каждой строчки.
После ip начинаются проблемы.

# bash -xx whilecappor.sh 
+ lease=/var/lib/dhcpd/dhcpd.leases
+ read line
++ echo lease 10.46.12.151 '{'
++ sed 's/lease\ //'
++ grep -E lease
++ sed 's/.$//'
+ ipaddr='10.46.12.151 '
++ echo lease 10.46.12.151 '{'
++ sed 's/\ hardware\ ethernet\ //'
++ sed 's/.$//'
++ grep -E 'hardware ethernet'
+ macaddr=

в общем, я понял предмет нашего с тобой «недопонимания» :-D
твой ответ ТСу мне кажут только если включить отображение каментов от игнорируемых :-D
хотя, ни тебя, ни ТСа я не игнорю. кругом баги!

..и вообще, мутная тема - «Показано 8 сообщений из 18. Показать все.» ;)

rain@elitebook:/tmp$ awk '/^lease/{ip=$2; mac=""; hst=""}; /ethernet/{mac=$3; gsub(/;/, "", mac)}; /client-hostname/{hst=$2; gsub(/;|\"/, "", hst)}; /}/{print ip, mac, hst}' bla
10.46.12.154 40:XX:XX:XX:XX::af android-15e
10.46.12.155 41:XX:XX:XX:XX::af andro123id-15e
10.46.12.156 43:XX:XX:XX:XX::af andwwroid-15e
10.46.12.157 40:YY:XX:XX:XX::af androisdsd-15e
10.46.12.158 40:XX:XX:XX:ZZ::af android-666e
10.46.12.159 40:XX:XX:XX:XX:ЫЫ Windows

Как я понимаю, каждый блок lease закрывается }? Форматирование в последнем блоке добавить по вкусу

Спасибо тебе.
Только можешь указать на ошибку в моём скрипте ?

Выше ведь уже показали. Не надо передавать грепу сразу 2 переменных. Ты не грепаешь файл, ты грепаешь данные, считываемые построчно из файла. Поэтому «echo данные | grep выражение | обработка»

У него вообще задума неверна: после _каждой_ строки печаталось бы

        echo "IP Address is:    "$ipaddr""
        echo "MAC Address is:   "$macaddr""
        echo "Hostname is:      "$hostname""

Хм, ещё и что-то с кавычками.

echo данные

А если «данные» — "-n", "-e", "-eEen" и т.п.?

после _каждой_ строки печаталось бы

Кстати, да. У меня это учтено.

А если «данные» — "-n", "-e", "-eEen" и т.п.?

Да, тогда неудобно.

rain@elitebook:/tmp$ data='-e'
rain@elitebook:/tmp$ echo "$data"

rain@elitebook:/tmp$ echo " $data"
 -e
rain@elitebook:/tmp$ grep e <<< "$data"
-e

echo " $data"

Это не то, что нужно.

grep e <<< «$data»

Я знаю. Зачем тогда советовать echo?

Это не то, что нужно.

Да. Но echo не понимает --, а пробел в данном случае не мешает работе. Опять же, я просто подстраиваюсь под предоставленные данные. В leases нет -e и тому подобного.

Зачем тогда советовать echo?

Если уж на то пошло, то я советовал awk. А «<<<» - башизм, «echo bla | grep something» - более общепринятый вариант

echo bla | grep something

printf '%s\n' bla | grep something :)

Ok, ok =)