From: Alexey Gladkov <gladkov.alexey@gmail.com>
To: make-initrd@lists.altlinux.org
Subject: Re: [make-initrd] [PATCH v1 12/41] fork pipeline: overlayfs step modified
Date: Mon, 27 Sep 2021 11:00:22 +0200
Message-ID: <20210927090022.czgjttvyyrjbqe67@example.org> (raw)
In-Reply-To: <bcd4ff25-eedd-26c9-cfad-a11fede6bad9@gmail.com>
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
prev parent reply other threads:[~2021-09-27 9:00 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-09-24 15:56 Leonid Krivoshein
2021-09-26 13:56 ` Alexey Gladkov
2021-09-26 19:40 ` Leonid Krivoshein
2021-09-27 9:00 ` Alexey Gladkov [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20210927090022.czgjttvyyrjbqe67@example.org \
--to=gladkov.alexey@gmail.com \
--cc=make-initrd@lists.altlinux.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Make-initrd development discussion
This inbox may be cloned and mirrored by anyone:
git clone --mirror http://lore.altlinux.org/make-initrd/0 make-initrd/git/0.git
# If you have public-inbox 1.1+ installed, you may
# initialize and index your mirror using the following commands:
public-inbox-init -V2 make-initrd make-initrd/ http://lore.altlinux.org/make-initrd \
make-initrd@lists.altlinux.org make-initrd@lists.altlinux.ru make-initrd@lists.altlinux.com
public-inbox-index make-initrd
Example config snippet for mirrors.
Newsgroup available over NNTP:
nntp://lore.altlinux.org/org.altlinux.lists.make-initrd
AGPL code for this site: git clone https://public-inbox.org/public-inbox.git