On 18.07.2012 14:05, Igor Chumak wrote: > 17.07.2012 18:34, "Артём Н." пишет: >> On 17.07.2012 11:24, Artem Chuprina wrote: >>> Артём Н. -> [email protected] @ Thu, 12 Jul 2012 20:31:32 >>> +0400: >>> >>> >> Дык, кто же будет это читать... Нужна мотивация. >>> АН> Вы прям убили мою светлую идею на корню. :-D >>> АН> Я понимаю, что читать и разбирать по строчке нафиг никому не нужно. >>> АН> Общее впечатление: читабельно, нечитабельно, >>> Вон там первая строчка квотинга дает ответ на этот вопрос. Нечитабельно. >> Тогда вопрос: почему? Комментарии есть. Код оформлен, как мне кажется, в >> читаемом стиле. "Трюков" не применялось. Из-за чего? > Дык 500 строк кода.. Если код в экран не влазит, и одним взглядом его не > окинешь > - значит это уже не совет, а платный суппорт ;) > А так , не спорю, код написан, код оформлен, код , возможно , работает ;) Не работал. ;-) Я забыл аргументы main передать и были некоторые косяки. В том, что не протестировано. Сейчас исправил. Теперь он успешно делает бэкапы. Даже первый полный бэкап того, что хотел вчера сделал. Но ещё минус: сейчас есть блокировка от запуска второй копии с тем же конфигом, но сигналы не ловлю. Поэтому, при нажатии Ctrl+C он завершается не убирая блокировку. В принципе-то не очень важно, потому что запускаться будет по anacron, но потом доделаю. Всего, кстати, получилось около 800 строк вместе с конфигами и инициализатором. Из 500 строк mkbackup 200 строк занимают тесты. Инициализатор просто запускает fusecompress и генерирует общий список для rdiff-backup (список для незашифрованных каталогов хранится в /etc, а список для шифрованного /home хранится в /home) в /tmp (у меня /tmp в памяти, так что, это нормально - генерировать его при каждой перезагрузке, но, в идеале, конечно, нужно пересоздавать только при изменении списков). Конфиги разделены: общий, для инициализатора и для оболочки.
В общем-то интересует, что плохо? И читабельно ли с тестами? И читабельно ли без тестов? -- To UNSUBSCRIBE, email to [email protected] with a subject of "unsubscribe". Trouble? Contact [email protected] Archive: http://lists.debian.org/[email protected]
