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

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

[Leonid Krivoshein]

---
  .../features/bootchain-core/data/lib/bootchain/mountfs        | 4 ++--
  1 file changed, 2 insertions(+), 2 deletions(-)

diff --git 
a/make-initrd/features/bootchain-core/data/lib/bootchain/mountfs 
b/make-initrd/features/bootchain-core/data/lib/bootchain/mountfs
index d15ae91..a9f515e 100755
--- a/make-initrd/features/bootchain-core/data/lib/bootchain/mountfs
+++ b/make-initrd/features/bootchain-core/data/lib/bootchain/mountfs
@@ -1,6 +1,6 @@
  #!/bin/bash -efu
  -. pipeline-sh-functions
+. bootchain-sh-functions
   check_parameter MOUNTFS
  param="$(get_parameter MOUNTFS)"
@@ -13,4 +13,4 @@ opts=
  [ -c "$target" ] || [ -b "$target" ] ||
  	opts='ro,loop'
  -run mount ${opts:+-o $opts} "$target" "$destdir"
+run mount ${opts:+-o $opts} -- "$target" "$destdir"
-- 
2.21.0

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

[Alexey Gladkov]

On Fri, Sep 24, 2021 at 06:56:33PM +0300, Leonid Krivoshein wrote:
> ---
>  .../features/bootchain-core/data/lib/bootchain/mountfs        | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/make-initrd/features/bootchain-core/data/lib/bootchain/mountfs
> b/make-initrd/features/bootchain-core/data/lib/bootchain/mountfs
> index d15ae91..a9f515e 100755
> --- a/make-initrd/features/bootchain-core/data/lib/bootchain/mountfs
> +++ b/make-initrd/features/bootchain-core/data/lib/bootchain/mountfs
> @@ -1,6 +1,6 @@
>  #!/bin/bash -efu
>  -. pipeline-sh-functions
> +. bootchain-sh-functions
>   check_parameter MOUNTFS
>  param="$(get_parameter MOUNTFS)"
> @@ -13,4 +13,4 @@ opts=
>  [ -c "$target" ] || [ -b "$target" ] ||
>  	opts='ro,loop'
>  -run mount ${opts:+-o $opts} "$target" "$destdir"
> +run mount ${opts:+-o $opts} -- "$target" "$destdir"

Я не понял зачем этот патч ?

Сделай пожалуйста описание.

-- 
Rgrds, legion

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

[Leonid Krivoshein]

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"
> Я не понял зачем этот патч ?

Второе изменение и так понятно, зачем. А если непонятно, то мне не так 
принципиально, чтобы ради описания "--" в этом месте придумывать целый 
текст.


> Сделай пожалуйста описание.

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


-- 
Best regards,
Leonid Krivoshein.

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

[Alexey Gladkov]

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