search

itdoginfo / domain-routing-openwrt

Automatic configuration of Openwrt router for routing by domains. Ansible role and shell script
https://t.me/itdoginfo
232 stars 60 forks source link

feat: Added package installation check #25

Closed HattabbI4 closed 3 weeks ago

HattabbI4 commented 3 weeks ago

Посчитал необходимым добавить, так как столкнулся с проблемой, что при не стабильном интернете curl не установился, и я поймал бесконечные попытки getdomains скачать список

itdoginfo commented 3 weeks ago

Хорошо, пару замечаний:

  1. Весь скрипт на английском, ваша вставка на русском.
  2. Nano не обязателен в скрипте, его не нужно так проверять, а значит цикл не нужен.

Поправьте эти моменты пожалуйста

HattabbI4 commented 3 weeks ago
  1. Весь скрипт на английском, ваша вставка на русском.

поправил

  1. Nano не обязателен в скрипте, его не нужно так проверять, а значит цикл не нужен.

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

itdoginfo commented 3 weeks ago

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

Дело в том, что везде настройку конфигов делаю через него. И когда он есть после установки, человеку будет проще с ним. Окей, пусть будет в цикле, меньше занимает места