From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <gladkov.alexey@gmail.com>
Date: Mon, 27 Sep 2021 11:23:56 +0200
From: Alexey Gladkov <gladkov.alexey@gmail.com>
To: make-initrd@lists.altlinux.org
Message-ID: <20210927092356.7vvddgwbw3oxphqk@example.org>
References: <a9409f0a-207c-baa2-e6e8-8dbb850d5f87@gmail.com>
 <20210924190822.hkqkjmcsj3u3zazv@example.org>
 <a7bae7b2-6210-8044-bbef-33d3babd88b2@gmail.com>
 <20210926142921.qnmrkydotjtpq664@example.org>
 <4aa5ac9b-ae30-2e1d-171a-fb92591c5afb@gmail.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <4aa5ac9b-ae30-2e1d-171a-fb92591c5afb@gmail.com>
Subject: Re: [make-initrd] [PATCH v1 00/41] fork pipeline
X-BeenThere: make-initrd@lists.altlinux.org
X-Mailman-Version: 2.1.12
Precedence: list
Reply-To: make-initrd@lists.altlinux.org
List-Id: <make-initrd.lists.altlinux.org>
List-Unsubscribe: <https://lists.altlinux.org/mailman/options/make-initrd>,
 <mailto:make-initrd-request@lists.altlinux.org?subject=unsubscribe>
List-Archive: <http://lists.altlinux.org/pipermail/make-initrd>
List-Post: <mailto:make-initrd@lists.altlinux.org>
List-Help: <mailto:make-initrd-request@lists.altlinux.org?subject=help>
List-Subscribe: <https://lists.altlinux.org/mailman/listinfo/make-initrd>,
 <mailto:make-initrd-request@lists.altlinux.org?subject=subscribe>
X-List-Received-Date: Mon, 27 Sep 2021 09:23:57 -0000
Archived-At: <http://lore.altlinux.org/make-initrd/20210927092356.7vvddgwbw3oxphqk@example.org/>
List-Archive: <http://lore.altlinux.org/make-initrd/>

On Sun, Sep 26, 2021 at 11:09:39PM +0300, Leonid Krivoshein wrote:
> > Твои патчи не
> > разбиты логически, в них нет описания зачем делаются те или иные
> > изменения, в 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.
> 
> _______________________________________________
> Make-initrd mailing list
> Make-initrd@lists.altlinux.org
> https://lists.altlinux.org/mailman/listinfo/make-initrd

-- 
Rgrds, legion