--- .../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
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
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.
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