Re: [make-initrd] [PATCH v1 00/41] fork pipeline

[Leonid Krivoshein <klark.devel@gmail.com>]

26.09.2021 17:29, Alexey Gladkov пишет:
> On Sat, Sep 25, 2021 at 12:59:27AM +0300, Leonid Krivoshein wrote:
>> 24.09.2021 22:08, Alexey Gladkov пишет:
>>>> На первой итерации, чтобы не флудить, сделал поменьше коммитов. Всё равно
>>>> вспомнить "что за чем менялось" уже невозможно. На следующих итерациях
>>>> постараюсь сделать это более понятным. Поэтому первые три коммита позволяют
>>>> понять порядок перемещения, переименования и что создано нового. Заранее
>>>> извиняюсь за порядок и оформление коммитов, готов исправиться.
>>> Я начал читать, но так как кода очень много не успею за один день всё
>>> осилить. Я буду ещё комментировать.
>> Да это понятно, смотри, по возможности. А я тут подумал, что bootchain не
>> просто же так форкался и переделывался, это не только демон и API --
>> основная часть его "клиентов" дойдёт до апстрима следом, но сейчас этого нет
>> в удобном тебе виде. Может помочь черновик документации, что я выслал. Также
>> прилагаю лог самой проблемной загрузки по FTP, сделанный в результате
>> сегодняшней проверки отправленного в апстрим. Могу выслать такой же с
>> локальной загрузкой, если надо. Где-то это может понять логику работы кода и
>> подход к отладочным сообщениям. Ну и на вопросы буду рад ответить!
> Я дальше смотреть пока не буду. Пожалуйста переделай патчи.

Конечно, переделаю.


> Твои патчи не
> разбиты логически, в них нет описания зачем делаются те или иные
> изменения, в README я не увидел описаний.

На первом этапе я хотел показать изменения pipeline, в целом, но сделать 
более читабельной историю и не флудить большим числом изменений. Мне и 
самому надо было понять, так как делались эти изменения очень давно и 
история нигде не сохранилась.

Необходимость писать правильные commit message я конечно понимаю. Сейчас 
их написание притормозит процесс. И ещё более сильно притормозит, если 
придётся писать README к каждой фиче на английском языке -- черновик 
документации на русском более 40 страниц.

Кстати, при разделении pipeline на 4 фичи должно получиться 4 README, 
включая исходный вариант, который я не трогал. Я вот не понял суть 
замечаний к тем двум README, что попали в bootchain-getimage и 
bootchain-waitdev. А про README из bootchain-core уже написал ранее.


> Форк + rename можно сделать за один коммит. После этого уже идут
> содержательные коммиты, которые приносят твой новый функционал или меняют
> поведение.
>
> Сейчас же есть пачка "fork pipeline: * modified", которые либо делают
> переименование, либо за одно что-то правят.

-- 
Best regards,
Leonid Krivoshein.