Re: [make-initrd] [PATCH v1 11/41] fork pipeline: mountfs step modified

[Alexey Gladkov <gladkov.alexey@gmail.com>]

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