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

Question

[Wellspring]

Добрый день.
Только открываю для себя мир скриптов.
Хочу автоматизировать получение информации с большого количества страниц.
Использую утилиту wget.
Все работает, если создаю пользовательскую переменную:
for ip in 10.0.0.1 10.0.0.2 и т.д.
Задача в том, чтобы взять IP адреса(записаные в столбик) из файла hosts.txt пропустить через команду

wget -qO- http://admin:password@$ip/System |grep model.

Если в качестве переменной задаю вывод ip адресов из файла, в котором они записаны по одному в столбик, то команда перестает работать.
Рабочий вариант:

#!/bin/bash
file="hosts.txt"
for ip in 10.0.0.1 10.0.0.2
do
echo "Device on ip address $ip"
wget -qO- http://admin:12345678q@$ip/System |grep model
done

Когда указываю, что переменную нужно брать из файла hosts.txt, скрипт передает работать.

#!/bin/bash
file="hosts.txt"
for ip in $(cat $file)
do
echo "Model for ip address $ip"
wget -qO- http://admin:password@$ip/System |grep model
done

Comments

[Модератор]

https://toster.ru/help/rules
3.8

Answer 1

[AUser0]

В echo $var поменяйте на $ip, а то как-то нелогичненько.
Перед grep поставьте pipe, то бишь вертикальную чёрточку: |
А в остальном - код вполне рабочий.

Comments

[Wellspring]

Немного запутался, открыто несколько вариантов)
Все работает, если переменная прописана в самом скрипте

#!/bin/bash
file="hosts.txt"
for ip in 10.0.0.1 10.0.0.2
do
echo "Device on ip address $ip"
wget -qO- http://admin:12345678q@$ip/System |grep model
done

как только сточку
for ip in 10.0.0.1 10.0.0.2
меняю на
for ip in $(cat $file)
команда wget перестает работать.
вариант с
for ip in `cat hosts.txt` тоже, к сожалению не работает.

[q2zoff]

Wellspring, покажите вывод. Не может оно не работать, если файл правильно отформатирован. Мне кажется, что причиной проблемы является символ возврата каретки во входящем файле.

[Wellspring]

q2zoff, большоае спасибо! Проблема была в формате файла.
Помогло
dos2unix $file

Answer 2

[DevMan]

https://ideone.com/DNkmvT

Comments

[Wellspring]

большое спасибо, вариант рабочий и пища для размышлений)

Answer 3

[xotkot]

добавте перед циклом, строку:
dos2unix $file

Comments

[Wellspring]

Огромное спасибо! Все заработало как нужно, два дня голову ломал)