From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <gladkov.alexey@gmail.com>
Date: Mon, 27 Sep 2021 11:00:22 +0200
From: Alexey Gladkov <gladkov.alexey@gmail.com>
To: make-initrd@lists.altlinux.org
Message-ID: <20210927090022.czgjttvyyrjbqe67@example.org>
References: <146dd20e-866c-0e26-8071-8ba3a643a1f7@gmail.com>
 <20210926135615.gtoxoddn3sev4gj6@example.org>
 <bcd4ff25-eedd-26c9-cfad-a11fede6bad9@gmail.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <bcd4ff25-eedd-26c9-cfad-a11fede6bad9@gmail.com>
Subject: Re: [make-initrd] [PATCH v1 12/41] fork pipeline: overlayfs step
 modified
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:00:23 -0000
Archived-At: <http://lore.altlinux.org/make-initrd/20210927090022.czgjttvyyrjbqe67@example.org/>
List-Archive: <http://lore.altlinux.org/make-initrd/>

On Sun, Sep 26, 2021 at 10:40:33PM +0300, Leonid Krivoshein wrote:
> 
> 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. Остальные переименования в коде выглядят
> вполне логично, если смотреть их после основного кода, а он в самом конце.

Я надеюсь в следующей версии патчей последовательность будет логичной ))

Я почему так на этом настаиваю: эти коммиты потом будут читать в том числе
и твои коллеги, когда добавлять что-то будут и у них не будет тебя,
который скажет "переименования в коде выглядят логично, если смотреть их
после основного кода". Они и я будут делать git-log/git-blame и читать или
вспоминать последовательно.

-- 
Rgrds, legion