Ошибка в задаче bash при изменении оболочек юзеров из файла. Как исправить?

Question

[nev3r17]

Решал задачку, в которой необходимо написать скрипт на bash, который бы создавал пользователей из файла users.txt, а позже менял их оболочку с /bin/sh на /bin/bash. Собственно с последним и возникли проблемы. Я написал вот такой цикл:

for line in $(cat users.txt | sed '/#[^ ]/d')
do
        sudo sed -i \"/^" $line ":/s/\\/bin\\/sh/\\/bin\\/bash/\" /etc/passwd
done

Ошибок никаких не выдает, просто не задает оболочку, ничего не меняется.

Comments

[Lynn «Кофеман»]

man chsh

[Lynn «Кофеман»]

А лучше man useradd и посмотреть на опцию -s

[nev3r17]

Lynn «Кофеман», у меня задача заставить работать именно при помощи sed (я на 1 курсе кибербеза)

[ValdikSS]

См. пункт 5.12 правил.
https://qna.habr.com/help/rules

[nev3r17]

ValdikSS, Разве я тут задал какую-то задачу? Это лишь 1/10 моей задачи с моей попыткой ее решения, в которой у меня ошибка. А почему это не удалили? Непонятно

[Lynn «Кофеман»]

В вашем sed как-то многовато бекслешей и стоят они в странных местах.
Я б что-то такое написал

for user in $(cat users.txt); do
    sed -i "/^$user:/s~:/bin/sh$~:/bin/bash~" /etc/passwd
done

[Saboteur]

Lynn «Кофеман», sudo не хватает, и комментарии в users.txt не обрабатываете

Answer 1

[Saboteur]

Если у вас такая задача, то она непонятно чему учит.
Слишком ненужные и длинные цепочки
Итак.

1. sed умеет сам открывать файлы. то есть вместо cat users.txt | sed можно просто sed users.txt
2. учитесь смотреть промежуточные значения, тот же ваш $line перед выполнением
3. Учитесь пользоваться sed без -i и смотреть промежуточный результат перед тем как он полезет в файл.
4. Что у вас с кавычками?
/^" $line ":/ это что?
вы пытаетесь пользоваться регулярками без понимания что это такое?
Вы вообще не понимаете как хотя бы одну строчку изменить через sed?

Тут даже чинить нечего, в строчке с sed все неверно.

for user in $(grep -vP "^(/s)*#" users.txt)
do
   sudo sed -ir $user "/^$user:/s/\\bin\\sh/\\bin\\bash/" /etc/passwd
done

Comments

[Lynn «Кофеман»]

Что-то у вас в строке с sed тоже ересь какая-то написана. Неожиданная переменная $line.

И «синдром зубочистки». В качестве разделителя в команде s можно использовать кучу разных символов, не обязательно слеш. И читаться будет проще.

$ echo /bin/sh | sed 's/\/bin\/sh/\/bin\/bash/'
/bin/bash
$ echo /bin/sh | sed 's~/bin/sh~/bin/bash~'
/bin/bash
$ echo /bin/sh | sed 's=/bin/sh=/bin/bash='
/bin/bash

мне кажется вторая и третья команды читаются гораздо проще.

И $user надо бы заякорить к началу строки "/^$user:/s...

[nev3r17]

Спасибо всем, разобрался :D