From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <gladkov.alexey@gmail.com>
Date: Mon, 27 Sep 2021 10:52:10 +0200
From: Alexey Gladkov <gladkov.alexey@gmail.com>
To: make-initrd@lists.altlinux.org
Message-ID: <20210927085210.3gtayaiv6omkf5nk@example.org>
References: <1ee7d125-4779-fd54-f939-340ab8780d67@gmail.com>
 <20210926134923.v5zqena2e5baduzj@example.org>
 <d02678a4-6774-f9bc-6c9d-f7c0ddfc1418@gmail.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <d02678a4-6774-f9bc-6c9d-f7c0ddfc1418@gmail.com>
Subject: Re: [make-initrd] [PATCH v1 11/41] fork pipeline: mountfs 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 08:52:12 -0000
Archived-At: <http://lore.altlinux.org/make-initrd/20210927085210.3gtayaiv6omkf5nk@example.org/>
List-Archive: <http://lore.altlinux.org/make-initrd/>

On Sun, Sep 26, 2021 at 10:11:29PM +0300, Leonid Krivoshein wrote:
> 
> 26.09.2021 16:49, Alexey Gladkov пишет:
> > On Fri, Sep 24, 2021 at 06:56:33PM +0300, Leonid Krivoshein wrote:
> > > [...]
> > >   -. pipeline-sh-functions
> > > +. bootchain-sh-functions
> 
> О том, что pipeline при форке переименован в bootchain, это и так понятно.
> Неужели будет недостаточно описать причину переименования в одном месте и
> это нужно будет повторять в каждом патче?

Я в последующих письмах уже предлагал копирование и переименование сделать
одним коммитом. Тогда и описание будет одно на всех.

> 
> > > [...]
> > >   -run mount ${opts:+-o $opts} "$target" "$destdir"
> > > +run mount ${opts:+-o $opts} -- "$target" "$destdir"
> > Я не понял зачем этот патч ?
> 
> Второе изменение и так понятно, зачем. А если непонятно, то мне не так
> принципиально, чтобы ради описания "--" в этом месте придумывать целый
> текст.

Это увеличивает diff. Если честно на этой строчке я подумал, что у тебя
где-то появились target="-xxx" и ты защищаешься тут.

> 
> > Сделай пожалуйста описание.
> 
> А других изменений-то и нет. В итоге, действительно тут нужно описание чуть
> подробнее того, что сделано? В этом месте оно как раз соответствует
> содержимому.

"fork pipeline: * modified" на каждый файл отдельно делать не надо. Сделай
один коммит.

-- 
Rgrds, legion