[make-initrd] [PATCH v1 12/41] fork pipeline: overlayfs step modified

[Leonid Krivoshein]

26.09.2021 16:56, Alexey Gladkov пишет:
> On Fri, Sep 24, 2021 at 06:56:41PM +0300, Leonid Krivoshein wrote:
>> [...]
>> diff --git
>> a/make-initrd/features/bootchain-core/data/lib/bootchain/overlayfs
>> b/make-initrd/features/bootchain-core/data/lib/bootchain/overlayfs
>> [...]
> Очередной "intermediate commit" (c) ? В описании одна строчка:
>
> overlayfs step modified
>
> То что он modified говорит сам патч, да я это вижу. Добавился
> bootchain-sh-functions, какие-то переименования ...
>
> Что это за изменения ?
> Зачем это делается ?
> Что это решает ?
>
> Леонид, такие патчи не возможно ревьювить. Это какая-то нарезка каких-то
> логически необоснованных изменений.

Базовые шаги mountfs, overlayfs, getimage и даже waitdev в нашей 
загрузочной цепочке вообще не используются, изменения в них 
непринципиальны и ни на что не повлияют. Если 38/41 (added symlinks) 
рабочий при обновлении с любой версии пакета, то первые три можно не 
трогать совсем. Более полезные изменения, отражённые в документации, 
затронули waitdev и rootfs. Первый раз я всё же старался побыстрее дать 
общую картину крупным планом, поэтому не уделил внимание commit message. 
Остальные переименования в коде выглядят вполне логично, если смотреть 
их после основного кода, а он в самом конце.


-- 
Best regards,
Leonid Krivoshein.