[make-initrd] [PATCH v4 0/4] split pipeline into 4 features

[make-initrd] [PATCH v4 0/4] split pipeline into 4 features

[Leonid Krivoshein]

Привет!


Пробный шар -- не чтобы апстримить, а чтобы оценить самую сложную часть. 
Если всё хорошо, за ней будут ещё коммиты, которые разделяются довольно 
просто. Но первые четыре коммита -- это разделение pipeline на четыре 
части, нет смысл такое апстримить как форк ради форка, в таком виде не 
добавляется дополнительная функциональность.



Leonid Krivoshein (4):
   Separate new feature of bootchain-core from pipeline
   kickstart: fix feature to work with bootchain-core
   bootchain-core: move bootchain-sh-function to /bin
   bootchain: separate getimage and waitdev features

  features/bootchain-core/README.md             | 81 +++++++++++++++++++
  features/bootchain-core/config.mk             |  5 ++
  .../data/bin/bootchain-sh-functions}          | 12 +--
  .../data/etc/initrd/cmdline.d/bootchain-core  |  4 +
  .../data/etc/rc.d/init.d/bootchain}           | 22 +----
  .../data/lib/bootchain}/mountfs               |  2 +-
  .../data/lib/bootchain}/overlayfs             |  2 +-
  .../data/lib/bootchain}/rootfs                |  2 +-
  .../data/lib/initrd/cmdline.d/bootchain       |  6 ++
  .../data/lib/initrd/cmdline.d/pipeline        |  2 +-
  .../data/sbin/bootchained}                    | 22 ++---
  features/bootchain-core/rules.mk              |  3 +
  features/bootchain-getimage/README.md         | 24 ++++++
  features/bootchain-getimage/config.mk         |  5 ++
  .../etc/initrd/cmdline.d/bootchain-getimage   |  1 +
  .../data/lib/bootchain}/getimage              |  2 +-
  features/bootchain-getimage/rules.mk          |  2 +
  features/bootchain-waitdev/README.md          | 22 +++++
  features/bootchain-waitdev/config.mk          |  6 ++
  .../etc/initrd/cmdline.d/bootchain-waitdev    |  1 +
  .../udev/rules.d/50-bootchain-waitdev.rules}  |  2 +-
  .../data/lib/bootchain}/waitdev               |  4 +-
  .../data/lib/initrd/pre/bootchain/300-waitdev | 12 +++
  .../lib/uevent/filters/bootchain-waitdev}     |  4 +-
  features/bootchain-waitdev/rules.mk           |  2 +
  .../lib/{pipeline => bootchain}/kickstart     |  2 +-
  features/pipeline/config.mk                   |  9 +--
  .../data/etc/initrd/cmdline.d/pipeline        |  5 --
  features/pipeline/rules.mk                    |  6 --
  29 files changed, 207 insertions(+), 65 deletions(-)
  create mode 100644 features/bootchain-core/README.md
  create mode 100644 features/bootchain-core/config.mk
  rename features/{pipeline/data/sbin/pipeline-sh-functions => 
bootchain-core/data/bin/bootchain-sh-functions} (70%)
  create mode 100644 
features/bootchain-core/data/etc/initrd/cmdline.d/bootchain-core
  rename features/{pipeline/data/etc/rc.d/init.d/pipeline => 
bootchain-core/data/etc/rc.d/init.d/bootchain} (63%)
  rename features/{pipeline/data/lib/pipeline => 
bootchain-core/data/lib/bootchain}/mountfs (91%)
  rename features/{pipeline/data/lib/pipeline => 
bootchain-core/data/lib/bootchain}/overlayfs (95%)
  rename features/{pipeline/data/lib/pipeline => 
bootchain-core/data/lib/bootchain}/rootfs (86%)
  create mode 100755 
features/bootchain-core/data/lib/initrd/cmdline.d/bootchain
  rename features/{pipeline => 
bootchain-core}/data/lib/initrd/cmdline.d/pipeline (64%)
  rename features/{pipeline/data/sbin/pipelined => 
bootchain-core/data/sbin/bootchained} (78%)
  create mode 100644 features/bootchain-core/rules.mk
  create mode 100644 features/bootchain-getimage/README.md
  create mode 100644 features/bootchain-getimage/config.mk
  create mode 100644 
features/bootchain-getimage/data/etc/initrd/cmdline.d/bootchain-getimage
  rename features/{pipeline/data/lib/pipeline => 
bootchain-getimage/data/lib/bootchain}/getimage (93%)
  create mode 100644 features/bootchain-getimage/rules.mk
  create mode 100644 features/bootchain-waitdev/README.md
  create mode 100644 features/bootchain-waitdev/config.mk
  create mode 100644 
features/bootchain-waitdev/data/etc/initrd/cmdline.d/bootchain-waitdev
  rename 
features/{pipeline/data/etc/udev/rules.d/50-pipeline-waitdev.rules => 
bootchain-waitdev/data/etc/udev/rules.d/50-bootchain-waitdev.rules} (77%)
  rename features/{pipeline/data/lib/pipeline => 
bootchain-waitdev/data/lib/bootchain}/waitdev (83%)
  create mode 100755 
features/bootchain-waitdev/data/lib/initrd/pre/bootchain/300-waitdev
  rename features/{pipeline/data/lib/uevent/filters/pipeline-waitdev => 
bootchain-waitdev/data/lib/uevent/filters/bootchain-waitdev} (77%)
  create mode 100644 features/bootchain-waitdev/rules.mk
  rename features/kickstart/data/lib/{pipeline => bootchain}/kickstart (93%)
  delete mode 100644 features/pipeline/data/etc/initrd/cmdline.d/pipeline

-- 
2.24.1

Re: [make-initrd] [PATCH v4 0/4] split pipeline into 4 features

[Alexey Gladkov]

On Wed, Oct 13, 2021 at 08:20:53PM +0300, Leonid Krivoshein wrote:
> Привет!
> 
> 
> Пробный шар -- не чтобы апстримить, а чтобы оценить самую сложную часть.
> Если всё хорошо, за ней будут ещё коммиты, которые разделяются довольно
> просто. Но первые четыре коммита -- это разделение pipeline на четыре части,
> нет смысл такое апстримить как форк ради форка, в таком виде не добавляется
> дополнительная функциональность.

Сейчас получилось гораздо понятнее. Я с этим патчсетом согласен.
Придираться к README не буду. Потом можно будет исправить.

> 
> 
> 
> Leonid Krivoshein (4):
>   Separate new feature of bootchain-core from pipeline
>   kickstart: fix feature to work with bootchain-core
>   bootchain-core: move bootchain-sh-function to /bin
>   bootchain: separate getimage and waitdev features
> 
>  features/bootchain-core/README.md             | 81 +++++++++++++++++++
>  features/bootchain-core/config.mk             |  5 ++
>  .../data/bin/bootchain-sh-functions}          | 12 +--
>  .../data/etc/initrd/cmdline.d/bootchain-core  |  4 +
>  .../data/etc/rc.d/init.d/bootchain}           | 22 +----
>  .../data/lib/bootchain}/mountfs               |  2 +-
>  .../data/lib/bootchain}/overlayfs             |  2 +-
>  .../data/lib/bootchain}/rootfs                |  2 +-
>  .../data/lib/initrd/cmdline.d/bootchain       |  6 ++
>  .../data/lib/initrd/cmdline.d/pipeline        |  2 +-
>  .../data/sbin/bootchained}                    | 22 ++---
>  features/bootchain-core/rules.mk              |  3 +
>  features/bootchain-getimage/README.md         | 24 ++++++
>  features/bootchain-getimage/config.mk         |  5 ++
>  .../etc/initrd/cmdline.d/bootchain-getimage   |  1 +
>  .../data/lib/bootchain}/getimage              |  2 +-
>  features/bootchain-getimage/rules.mk          |  2 +
>  features/bootchain-waitdev/README.md          | 22 +++++
>  features/bootchain-waitdev/config.mk          |  6 ++
>  .../etc/initrd/cmdline.d/bootchain-waitdev    |  1 +
>  .../udev/rules.d/50-bootchain-waitdev.rules}  |  2 +-
>  .../data/lib/bootchain}/waitdev               |  4 +-
>  .../data/lib/initrd/pre/bootchain/300-waitdev | 12 +++
>  .../lib/uevent/filters/bootchain-waitdev}     |  4 +-
>  features/bootchain-waitdev/rules.mk           |  2 +
>  .../lib/{pipeline => bootchain}/kickstart     |  2 +-
>  features/pipeline/config.mk                   |  9 +--
>  .../data/etc/initrd/cmdline.d/pipeline        |  5 --
>  features/pipeline/rules.mk                    |  6 --
>  29 files changed, 207 insertions(+), 65 deletions(-)
>  create mode 100644 features/bootchain-core/README.md
>  create mode 100644 features/bootchain-core/config.mk
>  rename features/{pipeline/data/sbin/pipeline-sh-functions =>
> bootchain-core/data/bin/bootchain-sh-functions} (70%)
>  create mode 100644
> features/bootchain-core/data/etc/initrd/cmdline.d/bootchain-core
>  rename features/{pipeline/data/etc/rc.d/init.d/pipeline =>
> bootchain-core/data/etc/rc.d/init.d/bootchain} (63%)
>  rename features/{pipeline/data/lib/pipeline =>
> bootchain-core/data/lib/bootchain}/mountfs (91%)
>  rename features/{pipeline/data/lib/pipeline =>
> bootchain-core/data/lib/bootchain}/overlayfs (95%)
>  rename features/{pipeline/data/lib/pipeline =>
> bootchain-core/data/lib/bootchain}/rootfs (86%)
>  create mode 100755
> features/bootchain-core/data/lib/initrd/cmdline.d/bootchain
>  rename features/{pipeline =>
> bootchain-core}/data/lib/initrd/cmdline.d/pipeline (64%)
>  rename features/{pipeline/data/sbin/pipelined =>
> bootchain-core/data/sbin/bootchained} (78%)
>  create mode 100644 features/bootchain-core/rules.mk
>  create mode 100644 features/bootchain-getimage/README.md
>  create mode 100644 features/bootchain-getimage/config.mk
>  create mode 100644
> features/bootchain-getimage/data/etc/initrd/cmdline.d/bootchain-getimage
>  rename features/{pipeline/data/lib/pipeline =>
> bootchain-getimage/data/lib/bootchain}/getimage (93%)
>  create mode 100644 features/bootchain-getimage/rules.mk
>  create mode 100644 features/bootchain-waitdev/README.md
>  create mode 100644 features/bootchain-waitdev/config.mk
>  create mode 100644
> features/bootchain-waitdev/data/etc/initrd/cmdline.d/bootchain-waitdev
>  rename features/{pipeline/data/etc/udev/rules.d/50-pipeline-waitdev.rules
> => bootchain-waitdev/data/etc/udev/rules.d/50-bootchain-waitdev.rules} (77%)
>  rename features/{pipeline/data/lib/pipeline =>
> bootchain-waitdev/data/lib/bootchain}/waitdev (83%)
>  create mode 100755
> features/bootchain-waitdev/data/lib/initrd/pre/bootchain/300-waitdev
>  rename features/{pipeline/data/lib/uevent/filters/pipeline-waitdev =>
> bootchain-waitdev/data/lib/uevent/filters/bootchain-waitdev} (77%)
>  create mode 100644 features/bootchain-waitdev/rules.mk
>  rename features/kickstart/data/lib/{pipeline => bootchain}/kickstart (93%)
>  delete mode 100644 features/pipeline/data/etc/initrd/cmdline.d/pipeline
> 
> -- 
> 2.24.1
> 
> 
> _______________________________________________
> Make-initrd mailing list
> Make-initrd@lists.altlinux.org
> https://lists.altlinux.org/mailman/listinfo/make-initrd

-- 
Rgrds, legion

[make-initrd] broken patches? (Re: [PATCH v4 0/4] split pipeline into 4 features)

[Arseny Maslennikov]

[-- Attachment #1: Type: text/plain, Size: 11017 bytes --]

On Wed, Oct 13, 2021 at 08:20:53PM +0300, Leonid Krivoshein wrote:
> Привет!
> 
> 
> Пробный шар -- не чтобы апстримить, а чтобы оценить самую сложную часть.
> Если всё хорошо, за ней будут ещё коммиты, которые разделяются довольно
> просто. Но первые четыре коммита -- это разделение pipeline на четыре части,
> нет смысл такое апстримить как форк ради форка, в таком виде не добавляется
> дополнительная функциональность.
> 

Патчи почему-то получаются битые; не знаю, как они у Алексея
прикладываются.

  % git am < split-pipeline-into-4-features.mbox
  warning: Patch sent with format=flowed; space at the end of lines might be lost.
  Applying: Separate new feature of bootchain-core from pipeline
  error: corrupt patch at line 181
  Patch failed at 0001 Separate new feature of bootchain-core from pipeline
  hint: Use 'git am --show-current-patch=diff' to see the failed patch
  When you have resolved this problem, run "git am --continue".
  If you prefer to skip this patch, run "git am --skip" instead.
  To restore the original branch and stop patching, run "git am --abort".
  [128] %

Например:

  index 89d9de5..1e95e7d 100755
  --- a/features/pipeline/data/etc/rc.d/init.d/pipeline
  +++ b/features/bootchain-core/data/etc/rc.d/init.d/bootchain
  @@ -1,6 +1,6 @@
    #!/bin/bash
    ### BEGIN INIT INFO
  -# Provides:            pipeline
  +# Provides:            bootchain
    # Required-Start:      uevent udev
    # Should-Start:
    # Required-Stop:

Здесь написано, что строки контекста в файле `f/p/d/etc/rc.d/init.d/p`
начинаются с пробела и потом #, а подменяемая строка, где Provides,
начинается с самого #. В самом том инит-скрипте вся преамбула начинается
с первого символа #.

  @@ -13,14 +13,14 @@
    . /.initrd/initenv
    . /etc/init.d/template
    -NAME=pipelined
  +NAME=bootchained
    PIDFILE="/var/run/$NAME.pid"
    ARGS="--lockfile $LOCKFILE --pidfile $PIDFILE --name $NAME 
  --displayname $NAME"

Вот тут кто-то съел пробельные символы в начале строки, и
patch(1)/git-am(1) видят эту строчку как подлежащую удалению строчку с
содержанием `-displayname $NAME"`. Первый `-` оказывается служебным.

     prepare() {
    	local dir i n
    -	dir=/.initrd/pipeline/waitdev
  +	dir=/.initrd/bootchain/waitdev

Вот тут кто-то сдвинул минус с первой позиции на третью.
Чуть выше, где NAME=pipelined переименовывается в bootchained, тоже - и
+ не ровно стоят.

    	mkdir -p -- "$dir"
     	i=0
  @@ -32,7 +32,7 @@ prepare() {
     start() {
    	RETVAL=0
  -	if [ "${ROOT-}" = pipeline ]; then
  +	if [ "${ROOT-}" = bootchain ] || [ "${ROOT-}" = pipeline ]; then
    		prepare
    		start_daemon --background $ARGS "$NAME"
    		RETVAL=$?

Здесь идёт "кусок"/"фрагмент" (hunk, устоявшегося русского термина нет),
в заголовке которого заявлено, что 7 строк превращаются в 7 строк, но он
содержит всего 6: 2 строки контекста перед -+, одна строка подменяется,
3 строки контекста после. Кто-то украл одну строку контекста.
Если посмотреть в подвергаемый переименованию/запатчиванию файл,
окажется, что эта строка пуста, и её представление в файле с патчем —
это единственный пробел-маркер строки контекста и всё, перенос строки.

Таких повреждений патча там и в других коммитах больше.
Человек может напрячься, потратить время, провести расследование и
попробовать догадаться, что имелось в виду; программа patch этого делать
не будет (а git am — и подавно). Возможно, Thunderbird ломает патчи при
отправке.

Почта для переписки, почта для офиса/рекламы (аутлук-подобная) и почта
для патчей — это три разные почты, которые, как чудесным образом
получилось, используют некоторые общие сетевые протоколы для доступа в
ящик и для пересылки. На мой скромный взгляд, не стоит пытаться
инструменты для одной из этих почт использовать для работы с другими
двумя.
В очередной раз порекомендую пользоваться git-send-email, а если его
тяжело заставить посылать письма с GMail-ящика — можно попросить у
ldv@/glebfm@ доступ к серверу @altlinux.org по SMTP и посылать с адреса,
который в том же changelog пакетов используется.
Письма в гуглоящике тоже появятся, если их туда пришлёт рассылка или
если они будут проставлены в Cc, будь то явно или автоматически.

P.S. Очень приветствую это направление и благодарен за развитие
pipeline/bootchain! Очень пригодится. Надо догонять и обгонять дебиан
;).

> 
> 
> Leonid Krivoshein (4):
>   Separate new feature of bootchain-core from pipeline
>   kickstart: fix feature to work with bootchain-core
>   bootchain-core: move bootchain-sh-function to /bin
>   bootchain: separate getimage and waitdev features
> 
>  features/bootchain-core/README.md             | 81 +++++++++++++++++++
>  features/bootchain-core/config.mk             |  5 ++
>  .../data/bin/bootchain-sh-functions}          | 12 +--
>  .../data/etc/initrd/cmdline.d/bootchain-core  |  4 +
>  .../data/etc/rc.d/init.d/bootchain}           | 22 +----
>  .../data/lib/bootchain}/mountfs               |  2 +-
>  .../data/lib/bootchain}/overlayfs             |  2 +-
>  .../data/lib/bootchain}/rootfs                |  2 +-
>  .../data/lib/initrd/cmdline.d/bootchain       |  6 ++
>  .../data/lib/initrd/cmdline.d/pipeline        |  2 +-
>  .../data/sbin/bootchained}                    | 22 ++---
>  features/bootchain-core/rules.mk              |  3 +
>  features/bootchain-getimage/README.md         | 24 ++++++
>  features/bootchain-getimage/config.mk         |  5 ++
>  .../etc/initrd/cmdline.d/bootchain-getimage   |  1 +
>  .../data/lib/bootchain}/getimage              |  2 +-
>  features/bootchain-getimage/rules.mk          |  2 +
>  features/bootchain-waitdev/README.md          | 22 +++++
>  features/bootchain-waitdev/config.mk          |  6 ++
>  .../etc/initrd/cmdline.d/bootchain-waitdev    |  1 +
>  .../udev/rules.d/50-bootchain-waitdev.rules}  |  2 +-
>  .../data/lib/bootchain}/waitdev               |  4 +-
>  .../data/lib/initrd/pre/bootchain/300-waitdev | 12 +++
>  .../lib/uevent/filters/bootchain-waitdev}     |  4 +-
>  features/bootchain-waitdev/rules.mk           |  2 +
>  .../lib/{pipeline => bootchain}/kickstart     |  2 +-
>  features/pipeline/config.mk                   |  9 +--
>  .../data/etc/initrd/cmdline.d/pipeline        |  5 --
>  features/pipeline/rules.mk                    |  6 --
>  29 files changed, 207 insertions(+), 65 deletions(-)
>  create mode 100644 features/bootchain-core/README.md
>  create mode 100644 features/bootchain-core/config.mk
>  rename features/{pipeline/data/sbin/pipeline-sh-functions =>
> bootchain-core/data/bin/bootchain-sh-functions} (70%)
>  create mode 100644
> features/bootchain-core/data/etc/initrd/cmdline.d/bootchain-core
>  rename features/{pipeline/data/etc/rc.d/init.d/pipeline =>
> bootchain-core/data/etc/rc.d/init.d/bootchain} (63%)
>  rename features/{pipeline/data/lib/pipeline =>
> bootchain-core/data/lib/bootchain}/mountfs (91%)
>  rename features/{pipeline/data/lib/pipeline =>
> bootchain-core/data/lib/bootchain}/overlayfs (95%)
>  rename features/{pipeline/data/lib/pipeline =>
> bootchain-core/data/lib/bootchain}/rootfs (86%)
>  create mode 100755
> features/bootchain-core/data/lib/initrd/cmdline.d/bootchain
>  rename features/{pipeline =>
> bootchain-core}/data/lib/initrd/cmdline.d/pipeline (64%)
>  rename features/{pipeline/data/sbin/pipelined =>
> bootchain-core/data/sbin/bootchained} (78%)
>  create mode 100644 features/bootchain-core/rules.mk
>  create mode 100644 features/bootchain-getimage/README.md
>  create mode 100644 features/bootchain-getimage/config.mk
>  create mode 100644
> features/bootchain-getimage/data/etc/initrd/cmdline.d/bootchain-getimage
>  rename features/{pipeline/data/lib/pipeline =>
> bootchain-getimage/data/lib/bootchain}/getimage (93%)
>  create mode 100644 features/bootchain-getimage/rules.mk
>  create mode 100644 features/bootchain-waitdev/README.md
>  create mode 100644 features/bootchain-waitdev/config.mk
>  create mode 100644
> features/bootchain-waitdev/data/etc/initrd/cmdline.d/bootchain-waitdev
>  rename features/{pipeline/data/etc/udev/rules.d/50-pipeline-waitdev.rules
> => bootchain-waitdev/data/etc/udev/rules.d/50-bootchain-waitdev.rules} (77%)
>  rename features/{pipeline/data/lib/pipeline =>
> bootchain-waitdev/data/lib/bootchain}/waitdev (83%)
>  create mode 100755
> features/bootchain-waitdev/data/lib/initrd/pre/bootchain/300-waitdev
>  rename features/{pipeline/data/lib/uevent/filters/pipeline-waitdev =>
> bootchain-waitdev/data/lib/uevent/filters/bootchain-waitdev} (77%)
>  create mode 100644 features/bootchain-waitdev/rules.mk
>  rename features/kickstart/data/lib/{pipeline => bootchain}/kickstart (93%)
>  delete mode 100644 features/pipeline/data/etc/initrd/cmdline.d/pipeline
> 
> -- 
> 2.24.1
> 
> 
> _______________________________________________
> Make-initrd mailing list
> Make-initrd@lists.altlinux.org
> https://lists.altlinux.org/mailman/listinfo/make-initrd

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

Re: [make-initrd] broken patches? (Re: [PATCH v4 0/4] split pipeline into 4 features)

[Leonid Krivoshein]

13.10.2021 21:50, Arseny Maslennikov пишет:
> On Wed, Oct 13, 2021 at 08:20:53PM +0300, Leonid Krivoshein wrote:
>> Привет!
>>
>>
>> Пробный шар -- не чтобы апстримить, а чтобы оценить самую сложную часть.
>> Если всё хорошо, за ней будут ещё коммиты, которые разделяются довольно
>> просто. Но первые четыре коммита -- это разделение pipeline на четыре части,
>> нет смысл такое апстримить как форк ради форка, в таком виде не добавляется
>> дополнительная функциональность.
>>
> Патчи почему-то получаются битые; не знаю, как они у Алексея
> прикладываются.
>
>    % git am < split-pipeline-into-4-features.mbox
>    warning: Patch sent with format=flowed; space at the end of lines might be lost.
>    Applying: Separate new feature of bootchain-core from pipeline
>    error: corrupt patch at line 181
>    Patch failed at 0001 Separate new feature of bootchain-core from pipeline
>    hint: Use 'git am --show-current-patch=diff' to see the failed patch
>    When you have resolved this problem, run "git am --continue".
>    If you prefer to skip this patch, run "git am --skip" instead.
>    To restore the original branch and stop patching, run "git am --abort".
>    [128] %
>
> Например:
>
>    index 89d9de5..1e95e7d 100755
>    --- a/features/pipeline/data/etc/rc.d/init.d/pipeline
>    +++ b/features/bootchain-core/data/etc/rc.d/init.d/bootchain
>    @@ -1,6 +1,6 @@
>      #!/bin/bash
>      ### BEGIN INIT INFO
>    -# Provides:            pipeline
>    +# Provides:            bootchain
>      # Required-Start:      uevent udev
>      # Should-Start:
>      # Required-Stop:
>
> Здесь написано, что строки контекста в файле `f/p/d/etc/rc.d/init.d/p`
> начинаются с пробела и потом #, а подменяемая строка, где Provides,
> начинается с самого #. В самом том инит-скрипте вся преамбула начинается
> с первого символа #.
>
>    @@ -13,14 +13,14 @@
>      . /.initrd/initenv
>      . /etc/init.d/template
>      -NAME=pipelined
>    +NAME=bootchained
>      PIDFILE="/var/run/$NAME.pid"
>      ARGS="--lockfile $LOCKFILE --pidfile $PIDFILE --name $NAME
>    --displayname $NAME"
>
> Вот тут кто-то съел пробельные символы в начале строки, и
> patch(1)/git-am(1) видят эту строчку как подлежащую удалению строчку с
> содержанием `-displayname $NAME"`. Первый `-` оказывается служебным.
>
>       prepare() {
>      	local dir i n
>      -	dir=/.initrd/pipeline/waitdev
>    +	dir=/.initrd/bootchain/waitdev
>
> Вот тут кто-то сдвинул минус с первой позиции на третью.
> Чуть выше, где NAME=pipelined переименовывается в bootchained, тоже - и
> + не ровно стоят.
>
>      	mkdir -p -- "$dir"
>       	i=0
>    @@ -32,7 +32,7 @@ prepare() {
>       start() {
>      	RETVAL=0
>    -	if [ "${ROOT-}" = pipeline ]; then
>    +	if [ "${ROOT-}" = bootchain ] || [ "${ROOT-}" = pipeline ]; then
>      		prepare
>      		start_daemon --background $ARGS "$NAME"
>      		RETVAL=$?
>
> Здесь идёт "кусок"/"фрагмент" (hunk, устоявшегося русского термина нет),
> в заголовке которого заявлено, что 7 строк превращаются в 7 строк, но он
> содержит всего 6: 2 строки контекста перед -+, одна строка подменяется,
> 3 строки контекста после. Кто-то украл одну строку контекста.
> Если посмотреть в подвергаемый переименованию/запатчиванию файл,
> окажется, что эта строка пуста, и её представление в файле с патчем —
> это единственный пробел-маркер строки контекста и всё, перенос строки.
>
> Таких повреждений патча там и в других коммитах больше.
> Человек может напрячься, потратить время, провести расследование и
> попробовать догадаться, что имелось в виду; программа patch этого делать
> не будет (а git am — и подавно). Возможно, Thunderbird ломает патчи при
> отправке.

Да, я отправлял через Thunderbird готовые письма, сформированные через 
git format-patch. Не знаю, что тут сказать. Алексей, патчи действительно 
побились? Прикладывать надо к апстримному make-initrd, если что: 
git://github.com/osboot/make-initrd


> Почта для переписки, почта для офиса/рекламы (аутлук-подобная) и почта
> для патчей — это три разные почты, которые, как чудесным образом
> получилось, используют некоторые общие сетевые протоколы для доступа в
> ящик и для пересылки. На мой скромный взгляд, не стоит пытаться
> инструменты для одной из этих почт использовать для работы с другими
> двумя.
> В очередной раз порекомендую пользоваться git-send-email, а если его

До этого пока не дожил, надо перенастроить машину сначала, но имею ввиду.


> тяжело заставить посылать письма с GMail-ящика — можно попросить у
> ldv@/glebfm@ доступ к серверу @altlinux.org по SMTP и посылать с адреса,
> который в том же changelog пакетов используется.
> Письма в гуглоящике тоже появятся, если их туда пришлёт рассылка или
> если они будут проставлены в Cc, будь то явно или автоматически.
>
> P.S. Очень приветствую это направление и благодарен за развитие
> pipeline/bootchain! Очень пригодится. Надо догонять и обгонять дебиан
> ;).
>

А я благодарен за полезные ссылки на предыдущей итерации. Благодаря им 
набрёл на хорошую литературу по гиту. Хоть ребейзить немного научился, 
уже хлеб! :-)



>>
>> Leonid Krivoshein (4):
>>    Separate new feature of bootchain-core from pipeline
>>    kickstart: fix feature to work with bootchain-core
>>    bootchain-core: move bootchain-sh-function to /bin
>>    bootchain: separate getimage and waitdev features
>>
>>   features/bootchain-core/README.md             | 81 +++++++++++++++++++
>>   features/bootchain-core/config.mk             |  5 ++
>>   .../data/bin/bootchain-sh-functions}          | 12 +--
>>   .../data/etc/initrd/cmdline.d/bootchain-core  |  4 +
>>   .../data/etc/rc.d/init.d/bootchain}           | 22 +----
>>   .../data/lib/bootchain}/mountfs               |  2 +-
>>   .../data/lib/bootchain}/overlayfs             |  2 +-
>>   .../data/lib/bootchain}/rootfs                |  2 +-
>>   .../data/lib/initrd/cmdline.d/bootchain       |  6 ++
>>   .../data/lib/initrd/cmdline.d/pipeline        |  2 +-
>>   .../data/sbin/bootchained}                    | 22 ++---
>>   features/bootchain-core/rules.mk              |  3 +
>>   features/bootchain-getimage/README.md         | 24 ++++++
>>   features/bootchain-getimage/config.mk         |  5 ++
>>   .../etc/initrd/cmdline.d/bootchain-getimage   |  1 +
>>   .../data/lib/bootchain}/getimage              |  2 +-
>>   features/bootchain-getimage/rules.mk          |  2 +
>>   features/bootchain-waitdev/README.md          | 22 +++++
>>   features/bootchain-waitdev/config.mk          |  6 ++
>>   .../etc/initrd/cmdline.d/bootchain-waitdev    |  1 +
>>   .../udev/rules.d/50-bootchain-waitdev.rules}  |  2 +-
>>   .../data/lib/bootchain}/waitdev               |  4 +-
>>   .../data/lib/initrd/pre/bootchain/300-waitdev | 12 +++
>>   .../lib/uevent/filters/bootchain-waitdev}     |  4 +-
>>   features/bootchain-waitdev/rules.mk           |  2 +
>>   .../lib/{pipeline => bootchain}/kickstart     |  2 +-
>>   features/pipeline/config.mk                   |  9 +--
>>   .../data/etc/initrd/cmdline.d/pipeline        |  5 --
>>   features/pipeline/rules.mk                    |  6 --
>>   29 files changed, 207 insertions(+), 65 deletions(-)
>>   create mode 100644 features/bootchain-core/README.md
>>   create mode 100644 features/bootchain-core/config.mk
>>   rename features/{pipeline/data/sbin/pipeline-sh-functions =>
>> bootchain-core/data/bin/bootchain-sh-functions} (70%)
>>   create mode 100644
>> features/bootchain-core/data/etc/initrd/cmdline.d/bootchain-core
>>   rename features/{pipeline/data/etc/rc.d/init.d/pipeline =>
>> bootchain-core/data/etc/rc.d/init.d/bootchain} (63%)
>>   rename features/{pipeline/data/lib/pipeline =>
>> bootchain-core/data/lib/bootchain}/mountfs (91%)
>>   rename features/{pipeline/data/lib/pipeline =>
>> bootchain-core/data/lib/bootchain}/overlayfs (95%)
>>   rename features/{pipeline/data/lib/pipeline =>
>> bootchain-core/data/lib/bootchain}/rootfs (86%)
>>   create mode 100755
>> features/bootchain-core/data/lib/initrd/cmdline.d/bootchain
>>   rename features/{pipeline =>
>> bootchain-core}/data/lib/initrd/cmdline.d/pipeline (64%)
>>   rename features/{pipeline/data/sbin/pipelined =>
>> bootchain-core/data/sbin/bootchained} (78%)
>>   create mode 100644 features/bootchain-core/rules.mk
>>   create mode 100644 features/bootchain-getimage/README.md
>>   create mode 100644 features/bootchain-getimage/config.mk
>>   create mode 100644
>> features/bootchain-getimage/data/etc/initrd/cmdline.d/bootchain-getimage
>>   rename features/{pipeline/data/lib/pipeline =>
>> bootchain-getimage/data/lib/bootchain}/getimage (93%)
>>   create mode 100644 features/bootchain-getimage/rules.mk
>>   create mode 100644 features/bootchain-waitdev/README.md
>>   create mode 100644 features/bootchain-waitdev/config.mk
>>   create mode 100644
>> features/bootchain-waitdev/data/etc/initrd/cmdline.d/bootchain-waitdev
>>   rename features/{pipeline/data/etc/udev/rules.d/50-pipeline-waitdev.rules
>> => bootchain-waitdev/data/etc/udev/rules.d/50-bootchain-waitdev.rules} (77%)
>>   rename features/{pipeline/data/lib/pipeline =>
>> bootchain-waitdev/data/lib/bootchain}/waitdev (83%)
>>   create mode 100755
>> features/bootchain-waitdev/data/lib/initrd/pre/bootchain/300-waitdev
>>   rename features/{pipeline/data/lib/uevent/filters/pipeline-waitdev =>
>> bootchain-waitdev/data/lib/uevent/filters/bootchain-waitdev} (77%)
>>   create mode 100644 features/bootchain-waitdev/rules.mk
>>   rename features/kickstart/data/lib/{pipeline => bootchain}/kickstart (93%)
>>   delete mode 100644 features/pipeline/data/etc/initrd/cmdline.d/pipeline
>>
>> -- 
>> 2.24.1
>>
>>
>> _______________________________________________
>> Make-initrd mailing list
>> Make-initrd@lists.altlinux.org
>> https://lists.altlinux.org/mailman/listinfo/make-initrd
>>
>> _______________________________________________
>> Make-initrd mailing list
>> Make-initrd@lists.altlinux.org
>> https://lists.altlinux.org/mailman/listinfo/make-initrd

-- 
Best regards,
Leonid Krivoshein.

Re: [make-initrd] broken patches? (Re: [PATCH v4 0/4] split pipeline into 4 features)

[Alexey Gladkov]

On Wed, Oct 13, 2021 at 10:04:52PM +0300, Leonid Krivoshein wrote:
> 
> 13.10.2021 21:50, Arseny Maslennikov пишет:
> > On Wed, Oct 13, 2021 at 08:20:53PM +0300, Leonid Krivoshein wrote:
> > > Привет!
> > > 
> > > 
> > > Пробный шар -- не чтобы апстримить, а чтобы оценить самую сложную часть.
> > > Если всё хорошо, за ней будут ещё коммиты, которые разделяются довольно
> > > просто. Но первые четыре коммита -- это разделение pipeline на четыре части,
> > > нет смысл такое апстримить как форк ради форка, в таком виде не добавляется
> > > дополнительная функциональность.
> > > 
> > Патчи почему-то получаются битые; не знаю, как они у Алексея
> > прикладываются.
> > 
> >    % git am < split-pipeline-into-4-features.mbox
> >    warning: Patch sent with format=flowed; space at the end of lines might be lost.
> >    Applying: Separate new feature of bootchain-core from pipeline
> >    error: corrupt patch at line 181
> >    Patch failed at 0001 Separate new feature of bootchain-core from pipeline
> >    hint: Use 'git am --show-current-patch=diff' to see the failed patch
> >    When you have resolved this problem, run "git am --continue".
> >    If you prefer to skip this patch, run "git am --skip" instead.
> >    To restore the original branch and stop patching, run "git am --abort".
> >    [128] %
> > 
> > Например:
> > 
> >    index 89d9de5..1e95e7d 100755
> >    --- a/features/pipeline/data/etc/rc.d/init.d/pipeline
> >    +++ b/features/bootchain-core/data/etc/rc.d/init.d/bootchain
> >    @@ -1,6 +1,6 @@
> >      #!/bin/bash
> >      ### BEGIN INIT INFO
> >    -# Provides:            pipeline
> >    +# Provides:            bootchain
> >      # Required-Start:      uevent udev
> >      # Should-Start:
> >      # Required-Stop:
> > 
> > Здесь написано, что строки контекста в файле `f/p/d/etc/rc.d/init.d/p`
> > начинаются с пробела и потом #, а подменяемая строка, где Provides,
> > начинается с самого #. В самом том инит-скрипте вся преамбула начинается
> > с первого символа #.
> > 
> >    @@ -13,14 +13,14 @@
> >      . /.initrd/initenv
> >      . /etc/init.d/template
> >      -NAME=pipelined
> >    +NAME=bootchained
> >      PIDFILE="/var/run/$NAME.pid"
> >      ARGS="--lockfile $LOCKFILE --pidfile $PIDFILE --name $NAME
> >    --displayname $NAME"
> > 
> > Вот тут кто-то съел пробельные символы в начале строки, и
> > patch(1)/git-am(1) видят эту строчку как подлежащую удалению строчку с
> > содержанием `-displayname $NAME"`. Первый `-` оказывается служебным.
> > 
> >       prepare() {
> >      	local dir i n
> >      -	dir=/.initrd/pipeline/waitdev
> >    +	dir=/.initrd/bootchain/waitdev
> > 
> > Вот тут кто-то сдвинул минус с первой позиции на третью.
> > Чуть выше, где NAME=pipelined переименовывается в bootchained, тоже - и
> > + не ровно стоят.
> > 
> >      	mkdir -p -- "$dir"
> >       	i=0
> >    @@ -32,7 +32,7 @@ prepare() {
> >       start() {
> >      	RETVAL=0
> >    -	if [ "${ROOT-}" = pipeline ]; then
> >    +	if [ "${ROOT-}" = bootchain ] || [ "${ROOT-}" = pipeline ]; then
> >      		prepare
> >      		start_daemon --background $ARGS "$NAME"
> >      		RETVAL=$?
> > 
> > Здесь идёт "кусок"/"фрагмент" (hunk, устоявшегося русского термина нет),
> > в заголовке которого заявлено, что 7 строк превращаются в 7 строк, но он
> > содержит всего 6: 2 строки контекста перед -+, одна строка подменяется,
> > 3 строки контекста после. Кто-то украл одну строку контекста.
> > Если посмотреть в подвергаемый переименованию/запатчиванию файл,
> > окажется, что эта строка пуста, и её представление в файле с патчем —
> > это единственный пробел-маркер строки контекста и всё, перенос строки.
> > 
> > Таких повреждений патча там и в других коммитах больше.
> > Человек может напрячься, потратить время, провести расследование и
> > попробовать догадаться, что имелось в виду; программа patch этого делать
> > не будет (а git am — и подавно). Возможно, Thunderbird ломает патчи при
> > отправке.
> 
> Да, я отправлял через Thunderbird готовые письма, сформированные через git
> format-patch. Не знаю, что тут сказать.

https://git-scm.com/docs/git-format-patch#_thunderbird

> Алексей, патчи действительно
> побились?

Да.

> Прикладывать надо к апстримному make-initrd, если что:
> git://github.com/osboot/make-initrd
> 
> 
> > Почта для переписки, почта для офиса/рекламы (аутлук-подобная) и почта
> > для патчей — это три разные почты, которые, как чудесным образом
> > получилось, используют некоторые общие сетевые протоколы для доступа в
> > ящик и для пересылки. На мой скромный взгляд, не стоит пытаться
> > инструменты для одной из этих почт использовать для работы с другими
> > двумя.
> > В очередной раз порекомендую пользоваться git-send-email, а если его
> 
> До этого пока не дожил, надо перенастроить машину сначала, но имею ввиду.
> 
> 
> > тяжело заставить посылать письма с GMail-ящика — можно попросить у
> > ldv@/glebfm@ доступ к серверу @altlinux.org по SMTP и посылать с адреса,
> > который в том же changelog пакетов используется.
> > Письма в гуглоящике тоже появятся, если их туда пришлёт рассылка или
> > если они будут проставлены в Cc, будь то явно или автоматически.
> > 

https://git-scm.com/docs/git-format-patch#_gmail

> > P.S. Очень приветствую это направление и благодарен за развитие
> > pipeline/bootchain! Очень пригодится. Надо догонять и обгонять дебиан
> > ;).
> > 
> 
> А я благодарен за полезные ссылки на предыдущей итерации. Благодаря им
> набрёл на хорошую литературу по гиту. Хоть ребейзить немного научился, уже
> хлеб! :-)
> 
> 
> 
> > > 
> > > Leonid Krivoshein (4):
> > >    Separate new feature of bootchain-core from pipeline
> > >    kickstart: fix feature to work with bootchain-core
> > >    bootchain-core: move bootchain-sh-function to /bin
> > >    bootchain: separate getimage and waitdev features
> > > 
> > >   features/bootchain-core/README.md             | 81 +++++++++++++++++++
> > >   features/bootchain-core/config.mk             |  5 ++
> > >   .../data/bin/bootchain-sh-functions}          | 12 +--
> > >   .../data/etc/initrd/cmdline.d/bootchain-core  |  4 +
> > >   .../data/etc/rc.d/init.d/bootchain}           | 22 +----
> > >   .../data/lib/bootchain}/mountfs               |  2 +-
> > >   .../data/lib/bootchain}/overlayfs             |  2 +-
> > >   .../data/lib/bootchain}/rootfs                |  2 +-
> > >   .../data/lib/initrd/cmdline.d/bootchain       |  6 ++
> > >   .../data/lib/initrd/cmdline.d/pipeline        |  2 +-
> > >   .../data/sbin/bootchained}                    | 22 ++---
> > >   features/bootchain-core/rules.mk              |  3 +
> > >   features/bootchain-getimage/README.md         | 24 ++++++
> > >   features/bootchain-getimage/config.mk         |  5 ++
> > >   .../etc/initrd/cmdline.d/bootchain-getimage   |  1 +
> > >   .../data/lib/bootchain}/getimage              |  2 +-
> > >   features/bootchain-getimage/rules.mk          |  2 +
> > >   features/bootchain-waitdev/README.md          | 22 +++++
> > >   features/bootchain-waitdev/config.mk          |  6 ++
> > >   .../etc/initrd/cmdline.d/bootchain-waitdev    |  1 +
> > >   .../udev/rules.d/50-bootchain-waitdev.rules}  |  2 +-
> > >   .../data/lib/bootchain}/waitdev               |  4 +-
> > >   .../data/lib/initrd/pre/bootchain/300-waitdev | 12 +++
> > >   .../lib/uevent/filters/bootchain-waitdev}     |  4 +-
> > >   features/bootchain-waitdev/rules.mk           |  2 +
> > >   .../lib/{pipeline => bootchain}/kickstart     |  2 +-
> > >   features/pipeline/config.mk                   |  9 +--
> > >   .../data/etc/initrd/cmdline.d/pipeline        |  5 --
> > >   features/pipeline/rules.mk                    |  6 --
> > >   29 files changed, 207 insertions(+), 65 deletions(-)
> > >   create mode 100644 features/bootchain-core/README.md
> > >   create mode 100644 features/bootchain-core/config.mk
> > >   rename features/{pipeline/data/sbin/pipeline-sh-functions =>
> > > bootchain-core/data/bin/bootchain-sh-functions} (70%)
> > >   create mode 100644
> > > features/bootchain-core/data/etc/initrd/cmdline.d/bootchain-core
> > >   rename features/{pipeline/data/etc/rc.d/init.d/pipeline =>
> > > bootchain-core/data/etc/rc.d/init.d/bootchain} (63%)
> > >   rename features/{pipeline/data/lib/pipeline =>
> > > bootchain-core/data/lib/bootchain}/mountfs (91%)
> > >   rename features/{pipeline/data/lib/pipeline =>
> > > bootchain-core/data/lib/bootchain}/overlayfs (95%)
> > >   rename features/{pipeline/data/lib/pipeline =>
> > > bootchain-core/data/lib/bootchain}/rootfs (86%)
> > >   create mode 100755
> > > features/bootchain-core/data/lib/initrd/cmdline.d/bootchain
> > >   rename features/{pipeline =>
> > > bootchain-core}/data/lib/initrd/cmdline.d/pipeline (64%)
> > >   rename features/{pipeline/data/sbin/pipelined =>
> > > bootchain-core/data/sbin/bootchained} (78%)
> > >   create mode 100644 features/bootchain-core/rules.mk
> > >   create mode 100644 features/bootchain-getimage/README.md
> > >   create mode 100644 features/bootchain-getimage/config.mk
> > >   create mode 100644
> > > features/bootchain-getimage/data/etc/initrd/cmdline.d/bootchain-getimage
> > >   rename features/{pipeline/data/lib/pipeline =>
> > > bootchain-getimage/data/lib/bootchain}/getimage (93%)
> > >   create mode 100644 features/bootchain-getimage/rules.mk
> > >   create mode 100644 features/bootchain-waitdev/README.md
> > >   create mode 100644 features/bootchain-waitdev/config.mk
> > >   create mode 100644
> > > features/bootchain-waitdev/data/etc/initrd/cmdline.d/bootchain-waitdev
> > >   rename features/{pipeline/data/etc/udev/rules.d/50-pipeline-waitdev.rules
> > > => bootchain-waitdev/data/etc/udev/rules.d/50-bootchain-waitdev.rules} (77%)
> > >   rename features/{pipeline/data/lib/pipeline =>
> > > bootchain-waitdev/data/lib/bootchain}/waitdev (83%)
> > >   create mode 100755
> > > features/bootchain-waitdev/data/lib/initrd/pre/bootchain/300-waitdev
> > >   rename features/{pipeline/data/lib/uevent/filters/pipeline-waitdev =>
> > > bootchain-waitdev/data/lib/uevent/filters/bootchain-waitdev} (77%)
> > >   create mode 100644 features/bootchain-waitdev/rules.mk
> > >   rename features/kickstart/data/lib/{pipeline => bootchain}/kickstart (93%)
> > >   delete mode 100644 features/pipeline/data/etc/initrd/cmdline.d/pipeline
> > > 
> > > -- 
> > > 2.24.1
> > > 
> > > 
> > > _______________________________________________
> > > Make-initrd mailing list
> > > Make-initrd@lists.altlinux.org
> > > https://lists.altlinux.org/mailman/listinfo/make-initrd
> > > 
> > > _______________________________________________
> > > Make-initrd mailing list
> > > Make-initrd@lists.altlinux.org
> > > https://lists.altlinux.org/mailman/listinfo/make-initrd
> 
> -- 
> Best regards,
> Leonid Krivoshein.
> 
> _______________________________________________
> Make-initrd mailing list
> Make-initrd@lists.altlinux.org
> https://lists.altlinux.org/mailman/listinfo/make-initrd

-- 
Rgrds, legion

Re: [make-initrd] broken patches? (Re: [PATCH v4 0/4] split pipeline into 4 features)

[Leonid Krivoshein]

13.10.2021 22:17, Alexey Gladkov пишет:
>> Да, я отправлял через Thunderbird готовые письма, сформированные через git
>> format-patch. Не знаю, что тут сказать.
> https://git-scm.com/docs/git-format-patch#_thunderbird

OK, на следующей итерации исправлюсь!


>> Алексей, патчи действительно
>> побились?
> Да.

Понял, это печально. Как же они у Антона прикладывались?...
Если кому надо, могу пока упаковать в тарбол и выслать.


>> Прикладывать надо к апстримному make-initrd, если что:
>> git://github.com/osboot/make-initrd

-- 
Best regards,
Leonid Krivoshein.

Re: [make-initrd] broken patches? (Re: [PATCH v4 0/4] split pipeline into 4 features)

[Антон Мидюков]

14.10.2021 02:41, Leonid Krivoshein пишет:
> 
> 13.10.2021 22:17, Alexey Gladkov пишет:
>>> Да, я отправлял через Thunderbird готовые письма, сформированные через git
>>> format-patch. Не знаю, что тут сказать.
>> https://git-scm.com/docs/git-format-patch#_thunderbird
> 
> OK, на следующей итерации исправлюсь!
> 
> 
>>> Алексей, патчи действительно
>>> побились?
>> Да.
> 
> Понял, это печально. Как же они у Антона прикладывались?...

Я не прикладывал твои письма-патчи, так как ты мне предварительно их не высылал.
До этого мы с тобой вложениями перекидывались.

-- 
С уважением, Антон Мидюков <antohami@basealt.ru>

Re: [make-initrd] broken patches? (Re: [PATCH v4 0/4] split pipeline into 4 features)

[Alexey Gladkov]

On Wed, Oct 13, 2021 at 09:50:26PM +0300, Arseny Maslennikov wrote:
> On Wed, Oct 13, 2021 at 08:20:53PM +0300, Leonid Krivoshein wrote:
> > Привет!
> > 
> > 
> > Пробный шар -- не чтобы апстримить, а чтобы оценить самую сложную часть.
> > Если всё хорошо, за ней будут ещё коммиты, которые разделяются довольно
> > просто. Но первые четыре коммита -- это разделение pipeline на четыре части,
> > нет смысл такое апстримить как форк ради форка, в таком виде не добавляется
> > дополнительная функциональность.
> > 
> 
> Патчи почему-то получаются битые; не знаю, как они у Алексея
> прикладываются.

Я не пробовал их прикладывать потому что "не чтобы апстримить". Но патчи
битые, да. Такое ощущение, что MTA linewrap сделал. Это хорошо видно на
README.

>   % git am < split-pipeline-into-4-features.mbox
>   warning: Patch sent with format=flowed; space at the end of lines might be lost.
>   Applying: Separate new feature of bootchain-core from pipeline
>   error: corrupt patch at line 181
>   Patch failed at 0001 Separate new feature of bootchain-core from pipeline
>   hint: Use 'git am --show-current-patch=diff' to see the failed patch
>   When you have resolved this problem, run "git am --continue".
>   If you prefer to skip this patch, run "git am --skip" instead.
>   To restore the original branch and stop patching, run "git am --abort".
>   [128] %
> 
> Например:
> 
>   index 89d9de5..1e95e7d 100755
>   --- a/features/pipeline/data/etc/rc.d/init.d/pipeline
>   +++ b/features/bootchain-core/data/etc/rc.d/init.d/bootchain
>   @@ -1,6 +1,6 @@
>     #!/bin/bash
>     ### BEGIN INIT INFO
>   -# Provides:            pipeline
>   +# Provides:            bootchain
>     # Required-Start:      uevent udev
>     # Should-Start:
>     # Required-Stop:
> 
> Здесь написано, что строки контекста в файле `f/p/d/etc/rc.d/init.d/p`
> начинаются с пробела и потом #, а подменяемая строка, где Provides,
> начинается с самого #. В самом том инит-скрипте вся преамбула начинается
> с первого символа #.
> 
>   @@ -13,14 +13,14 @@
>     . /.initrd/initenv
>     . /etc/init.d/template
>     -NAME=pipelined
>   +NAME=bootchained
>     PIDFILE="/var/run/$NAME.pid"
>     ARGS="--lockfile $LOCKFILE --pidfile $PIDFILE --name $NAME 
>   --displayname $NAME"
> 
> Вот тут кто-то съел пробельные символы в начале строки, и
> patch(1)/git-am(1) видят эту строчку как подлежащую удалению строчку с
> содержанием `-displayname $NAME"`. Первый `-` оказывается служебным.
> 
>      prepare() {
>     	local dir i n
>     -	dir=/.initrd/pipeline/waitdev
>   +	dir=/.initrd/bootchain/waitdev
> 
> Вот тут кто-то сдвинул минус с первой позиции на третью.
> Чуть выше, где NAME=pipelined переименовывается в bootchained, тоже - и
> + не ровно стоят.
> 
>     	mkdir -p -- "$dir"
>      	i=0
>   @@ -32,7 +32,7 @@ prepare() {
>      start() {
>     	RETVAL=0
>   -	if [ "${ROOT-}" = pipeline ]; then
>   +	if [ "${ROOT-}" = bootchain ] || [ "${ROOT-}" = pipeline ]; then
>     		prepare
>     		start_daemon --background $ARGS "$NAME"
>     		RETVAL=$?
> 
> Здесь идёт "кусок"/"фрагмент" (hunk, устоявшегося русского термина нет),
> в заголовке которого заявлено, что 7 строк превращаются в 7 строк, но он
> содержит всего 6: 2 строки контекста перед -+, одна строка подменяется,
> 3 строки контекста после. Кто-то украл одну строку контекста.
> Если посмотреть в подвергаемый переименованию/запатчиванию файл,
> окажется, что эта строка пуста, и её представление в файле с патчем —
> это единственный пробел-маркер строки контекста и всё, перенос строки.
> 
> Таких повреждений патча там и в других коммитах больше.
> Человек может напрячься, потратить время, провести расследование и
> попробовать догадаться, что имелось в виду; программа patch этого делать
> не будет (а git am — и подавно). Возможно, Thunderbird ломает патчи при
> отправке.
> 
> Почта для переписки, почта для офиса/рекламы (аутлук-подобная) и почта
> для патчей — это три разные почты, которые, как чудесным образом
> получилось, используют некоторые общие сетевые протоколы для доступа в
> ящик и для пересылки. На мой скромный взгляд, не стоит пытаться
> инструменты для одной из этих почт использовать для работы с другими
> двумя.
> В очередной раз порекомендую пользоваться git-send-email, а если его
> тяжело заставить посылать письма с GMail-ящика — можно попросить у
> ldv@/glebfm@ доступ к серверу @altlinux.org по SMTP и посылать с адреса,
> который в том же changelog пакетов используется.
> Письма в гуглоящике тоже появятся, если их туда пришлёт рассылка или
> если они будут проставлены в Cc, будь то явно или автоматически.
> 
> P.S. Очень приветствую это направление и благодарен за развитие
> pipeline/bootchain! Очень пригодится. Надо догонять и обгонять дебиан
> ;).
> 
> > 
> > 
> > Leonid Krivoshein (4):
> >   Separate new feature of bootchain-core from pipeline
> >   kickstart: fix feature to work with bootchain-core
> >   bootchain-core: move bootchain-sh-function to /bin
> >   bootchain: separate getimage and waitdev features
> > 
> >  features/bootchain-core/README.md             | 81 +++++++++++++++++++
> >  features/bootchain-core/config.mk             |  5 ++
> >  .../data/bin/bootchain-sh-functions}          | 12 +--
> >  .../data/etc/initrd/cmdline.d/bootchain-core  |  4 +
> >  .../data/etc/rc.d/init.d/bootchain}           | 22 +----
> >  .../data/lib/bootchain}/mountfs               |  2 +-
> >  .../data/lib/bootchain}/overlayfs             |  2 +-
> >  .../data/lib/bootchain}/rootfs                |  2 +-
> >  .../data/lib/initrd/cmdline.d/bootchain       |  6 ++
> >  .../data/lib/initrd/cmdline.d/pipeline        |  2 +-
> >  .../data/sbin/bootchained}                    | 22 ++---
> >  features/bootchain-core/rules.mk              |  3 +
> >  features/bootchain-getimage/README.md         | 24 ++++++
> >  features/bootchain-getimage/config.mk         |  5 ++
> >  .../etc/initrd/cmdline.d/bootchain-getimage   |  1 +
> >  .../data/lib/bootchain}/getimage              |  2 +-
> >  features/bootchain-getimage/rules.mk          |  2 +
> >  features/bootchain-waitdev/README.md          | 22 +++++
> >  features/bootchain-waitdev/config.mk          |  6 ++
> >  .../etc/initrd/cmdline.d/bootchain-waitdev    |  1 +
> >  .../udev/rules.d/50-bootchain-waitdev.rules}  |  2 +-
> >  .../data/lib/bootchain}/waitdev               |  4 +-
> >  .../data/lib/initrd/pre/bootchain/300-waitdev | 12 +++
> >  .../lib/uevent/filters/bootchain-waitdev}     |  4 +-
> >  features/bootchain-waitdev/rules.mk           |  2 +
> >  .../lib/{pipeline => bootchain}/kickstart     |  2 +-
> >  features/pipeline/config.mk                   |  9 +--
> >  .../data/etc/initrd/cmdline.d/pipeline        |  5 --
> >  features/pipeline/rules.mk                    |  6 --
> >  29 files changed, 207 insertions(+), 65 deletions(-)
> >  create mode 100644 features/bootchain-core/README.md
> >  create mode 100644 features/bootchain-core/config.mk
> >  rename features/{pipeline/data/sbin/pipeline-sh-functions =>
> > bootchain-core/data/bin/bootchain-sh-functions} (70%)
> >  create mode 100644
> > features/bootchain-core/data/etc/initrd/cmdline.d/bootchain-core
> >  rename features/{pipeline/data/etc/rc.d/init.d/pipeline =>
> > bootchain-core/data/etc/rc.d/init.d/bootchain} (63%)
> >  rename features/{pipeline/data/lib/pipeline =>
> > bootchain-core/data/lib/bootchain}/mountfs (91%)
> >  rename features/{pipeline/data/lib/pipeline =>
> > bootchain-core/data/lib/bootchain}/overlayfs (95%)
> >  rename features/{pipeline/data/lib/pipeline =>
> > bootchain-core/data/lib/bootchain}/rootfs (86%)
> >  create mode 100755
> > features/bootchain-core/data/lib/initrd/cmdline.d/bootchain
> >  rename features/{pipeline =>
> > bootchain-core}/data/lib/initrd/cmdline.d/pipeline (64%)
> >  rename features/{pipeline/data/sbin/pipelined =>
> > bootchain-core/data/sbin/bootchained} (78%)
> >  create mode 100644 features/bootchain-core/rules.mk
> >  create mode 100644 features/bootchain-getimage/README.md
> >  create mode 100644 features/bootchain-getimage/config.mk
> >  create mode 100644
> > features/bootchain-getimage/data/etc/initrd/cmdline.d/bootchain-getimage
> >  rename features/{pipeline/data/lib/pipeline =>
> > bootchain-getimage/data/lib/bootchain}/getimage (93%)
> >  create mode 100644 features/bootchain-getimage/rules.mk
> >  create mode 100644 features/bootchain-waitdev/README.md
> >  create mode 100644 features/bootchain-waitdev/config.mk
> >  create mode 100644
> > features/bootchain-waitdev/data/etc/initrd/cmdline.d/bootchain-waitdev
> >  rename features/{pipeline/data/etc/udev/rules.d/50-pipeline-waitdev.rules
> > => bootchain-waitdev/data/etc/udev/rules.d/50-bootchain-waitdev.rules} (77%)
> >  rename features/{pipeline/data/lib/pipeline =>
> > bootchain-waitdev/data/lib/bootchain}/waitdev (83%)
> >  create mode 100755
> > features/bootchain-waitdev/data/lib/initrd/pre/bootchain/300-waitdev
> >  rename features/{pipeline/data/lib/uevent/filters/pipeline-waitdev =>
> > bootchain-waitdev/data/lib/uevent/filters/bootchain-waitdev} (77%)
> >  create mode 100644 features/bootchain-waitdev/rules.mk
> >  rename features/kickstart/data/lib/{pipeline => bootchain}/kickstart (93%)
> >  delete mode 100644 features/pipeline/data/etc/initrd/cmdline.d/pipeline
> > 
> > -- 
> > 2.24.1
> > 
> > 
> > _______________________________________________
> > Make-initrd mailing list
> > Make-initrd@lists.altlinux.org
> > https://lists.altlinux.org/mailman/listinfo/make-initrd



> _______________________________________________
> Make-initrd mailing list
> Make-initrd@lists.altlinux.org
> https://lists.altlinux.org/mailman/listinfo/make-initrd


-- 
Rgrds, legion

Re: [make-initrd] broken patches? (Re: [PATCH v4 0/4] split pipeline into 4 features)

[Leonid Krivoshein]

13.10.2021 21:50, Arseny Maslennikov пишет:
> В очередной раз порекомендую пользоваться git-send-email, а если его
> тяжело заставить посылать письма с GMail-ящика — можно попросить у
> ldv@/glebfm@ доступ к серверу @altlinux.org по SMTP и посылать с адреса,
> который в том же changelog пакетов используется.

Единственное, что мне удалось заставить отправлять нормальные письма -- 
ssmtp, но в эту рассылку они почему-то не пропускаются:

$ /usr/sbin/ssmtp -v -t < v6-0000-cover-letter.patch
[<-] 220 air.basealt.ru ESMTP Postfix
[->] HELO bmt.localdomain
[<-] 250 air.basealt.ru
[->] MAIL FROM:<klark@altlinux.org>
[<-] 250 2.1.0 Ok
[->] RCPT TO:<make-initrd@lists.altlinux.org>
[<-] 554 5.7.1 <make-initrd@lists.altlinux.org>: Relay access denied
ssmtp: RCPT TO:<make-initrd@lists.altlinux.org> (554 5.7.1 
<make-initrd@lists.altlinux.org>: Relay access denied)

Обратные адреса пробовал разные, не помогает.


-- 
Best regards,
Leonid Krivoshein.

Re: [make-initrd] broken patches? (Re: [PATCH v4 0/4] split pipeline into 4 features)

[Vladimir D. Seleznev]

On Sun, Oct 24, 2021 at 02:13:56PM +0300, Leonid Krivoshein wrote:
> 
> 13.10.2021 21:50, Arseny Maslennikov пишет:
> > В очередной раз порекомендую пользоваться git-send-email, а если его
> > тяжело заставить посылать письма с GMail-ящика — можно попросить у
> > ldv@/glebfm@ доступ к серверу @altlinux.org по SMTP и посылать с адреса,
> > который в том же changelog пакетов используется.
> 
> Единственное, что мне удалось заставить отправлять нормальные письма -- 
> ssmtp, но в эту рассылку они почему-то не пропускаются:
> 
> $ /usr/sbin/ssmtp -v -t < v6-0000-cover-letter.patch
> [<-] 220 air.basealt.ru ESMTP Postfix
> [->] HELO bmt.localdomain
> [<-] 250 air.basealt.ru
> [->] MAIL FROM:<klark@altlinux.org>
> [<-] 250 2.1.0 Ok
> [->] RCPT TO:<make-initrd@lists.altlinux.org>
> [<-] 554 5.7.1 <make-initrd@lists.altlinux.org>: Relay access denied
> ssmtp: RCPT TO:<make-initrd@lists.altlinux.org> (554 5.7.1 
> <make-initrd@lists.altlinux.org>: Relay access denied)
> 
> Обратные адреса пробовал разные, не помогает.

Ничего не могу сказать про ssmtp, но мб ты не предоставил данные для
авторизации посылки писем с air?

-- 
   WBR,
   Vladimir D. Seleznev

Re: [make-initrd] broken patches? (Re: [PATCH v4 0/4] split pipeline into 4 features)

[Leonid Krivoshein]

24.10.2021 19:38, Vladimir D. Seleznev пишет:
> On Sun, Oct 24, 2021 at 02:13:56PM +0300, Leonid Krivoshein wrote:
>> 13.10.2021 21:50, Arseny Maslennikov пишет:
>>> В очередной раз порекомендую пользоваться git-send-email, а если его
>>> тяжело заставить посылать письма с GMail-ящика — можно попросить у
>>> ldv@/glebfm@ доступ к серверу @altlinux.org по SMTP и посылать с адреса,
>>> который в том же changelog пакетов используется.
>> Единственное, что мне удалось заставить отправлять нормальные письма --
>> ssmtp, но в эту рассылку они почему-то не пропускаются:
>>
>> $ /usr/sbin/ssmtp -v -t < v6-0000-cover-letter.patch
>> [<-] 220 air.basealt.ru ESMTP Postfix
>> [->] HELO bmt.localdomain
>> [<-] 250 air.basealt.ru
>> [->] MAIL FROM:<klark@altlinux.org>
>> [<-] 250 2.1.0 Ok
>> [->] RCPT TO:<make-initrd@lists.altlinux.org>
>> [<-] 554 5.7.1 <make-initrd@lists.altlinux.org>: Relay access denied
>> ssmtp: RCPT TO:<make-initrd@lists.altlinux.org> (554 5.7.1
>> <make-initrd@lists.altlinux.org>: Relay access denied)
>>
>> Обратные адреса пробовал разные, не помогает.
> Ничего не могу сказать про ssmtp, но мб ты не предоставил данные для
> авторизации посылки писем с air?

Предоставлял, пишет, что сервер не поддерживает аутентификацию. Похоже 
что air позволяет отправлять почту только в наш домен. Хотя, через него 
каким-то чудом же я в эту рассылку пишу из Thunderbird. Пробился два дня 
и, наконец, нашёл рабочий способ -- настроил msmtp на гугловый ящик, с 
ним, вроде, должно всё получиться...


-- 
Best regards,
Leonid Krivoshein.

Re: [make-initrd] broken patches? (Re: [PATCH v4 0/4] split pipeline into 4 features)

[Leonid Krivoshein]

24.10.2021 19:56, Leonid Krivoshein пишет:
> Предоставлял, пишет, что сервер не поддерживает аутентификацию. Похоже 
> что air позволяет отправлять почту только в наш домен. Хотя, через 
> него каким-то чудом же я в эту рассылку пишу из Thunderbird. Пробился 
> два дня и, наконец, нашёл рабочий способ -- настроил msmtp на гугловый 
> ящик, с ним, вроде, должно всё получиться... 

Увы, такая же фигня. (( На адрес make-initrd@lists.altlinux.org почту не 
пропускают:

msmtp: recipient address make-initrd@lists.altlinux.org not accepted by 
the server
msmtp: server message: 554 5.7.1 <make-initrd@lists.altlinux.org>: Relay 
access denied
msmtp: could not send mail (account default from /home/klark/.msmtprc)

Попробую ещё заслать Алексею напрямую. Но дальше не знаю, если только в 
тарбол паковать или отправлять сюда для обсуждения битыми, а скачать в 
нормальном виде их же можно из задания #288096.


-- 
Best regards,
Leonid Krivoshein.

Re: [make-initrd] [PATCH v4 0/4] split pipeline into 4 features

[Vladimir D. Seleznev]

On Wed, Oct 13, 2021 at 08:20:53PM +0300, Leonid Krivoshein wrote:
> Привет!
> 
> 
> Пробный шар -- не чтобы апстримить, а чтобы оценить самую сложную часть. 
> Если всё хорошо, за ней будут ещё коммиты, которые разделяются довольно 
> просто. Но первые четыре коммита -- это разделение pipeline на четыре 
> части, нет смысл такое апстримить как форк ради форка, в таком виде не 
> добавляется дополнительная функциональность.

У меня большая просьба: присылать патчи одним тредом. Сейчас в рассылке
куча разрозненных тредов, очень сложно стало ориентироваться.

-- 
   WBR,
   Vladimir D. Seleznev

Re: [make-initrd] [PATCH v4 0/4] split pipeline into 4 features

[Leonid Krivoshein]

27.10.2021 0:13, Vladimir D. Seleznev пишет:
> On Wed, Oct 13, 2021 at 08:20:53PM +0300, Leonid Krivoshein wrote:
>> Привет!
>>
>>
>> Пробный шар -- не чтобы апстримить, а чтобы оценить самую сложную часть.
>> Если всё хорошо, за ней будут ещё коммиты, которые разделяются довольно
>> просто. Но первые четыре коммита -- это разделение pipeline на четыре
>> части, нет смысл такое апстримить как форк ради форка, в таком виде не
>> добавляется дополнительная функциональность.
> У меня большая просьба: присылать патчи одним тредом. Сейчас в рассылке
> куча разрозненных тредов, очень сложно стало ориентироваться.

А у меня есть подозрение, что даже одним письмом не всех подписчиков 
этой рассылки интересует вопрос апстрима bootchain/altboot, так-то и мне 
было бы удобней вложить патчи в одно письмо, но сказали делать разными. 
Поэтому надеюсь следующую итерацию реализовать через pull request, благо 
блох в свитере вчера удалось завести. :-)


-- 
Best regards,
Leonid Krivoshein.

Re: [make-initrd] [PATCH v4 0/4] split pipeline into 4 features

[Vladimir D. Seleznev]

On Wed, Oct 27, 2021 at 01:09:59AM +0300, Leonid Krivoshein wrote:
> 
> 27.10.2021 0:13, Vladimir D. Seleznev пишет:
> > On Wed, Oct 13, 2021 at 08:20:53PM +0300, Leonid Krivoshein wrote:
> >> Привет!
> >>
> >>
> >> Пробный шар -- не чтобы апстримить, а чтобы оценить самую сложную часть.
> >> Если всё хорошо, за ней будут ещё коммиты, которые разделяются довольно
> >> просто. Но первые четыре коммита -- это разделение pipeline на четыре
> >> части, нет смысл такое апстримить как форк ради форка, в таком виде не
> >> добавляется дополнительная функциональность.
> > У меня большая просьба: присылать патчи одним тредом. Сейчас в рассылке
> > куча разрозненных тредов, очень сложно стало ориентироваться.
> 
> А у меня есть подозрение, что даже одним письмом не всех подписчиков 
> этой рассылки интересует вопрос апстрима bootchain/altboot, так-то и мне 
> было бы удобней вложить патчи в одно письмо, но сказали делать разными. 
> Поэтому надеюсь следующую итерацию реализовать через pull request, благо 
> блох в свитере вчера удалось завести. :-)

Не, не одним письмом, а одним тредом.

-- 
   WBR,
   Vladimir D. Seleznev

Re: [make-initrd] [PATCH v4 0/4] split pipeline into 4 features

[Alexey Gladkov]

On Wed, Oct 27, 2021 at 01:09:59AM +0300, Leonid Krivoshein wrote:
> 27.10.2021 0:13, Vladimir D. Seleznev пишет:
> > У меня большая просьба: присылать патчи одним тредом. Сейчас в рассылке
> > куча разрозненных тредов, очень сложно стало ориентироваться.

+1

> А у меня есть подозрение, что даже одним письмом не всех подписчиков этой
> рассылки интересует вопрос апстрима bootchain/altboot, так-то и мне было бы
> удобней вложить патчи в одно письмо, но сказали делать разными. Поэтому
> надеюсь следующую итерацию реализовать через pull request, благо блох в
> свитере вчера удалось завести. :-)

Володя просит делать git format-patch --thread. Тогда утилита добавит
In-Reply-To в патчи и получится тред, если патчи отослать как они есть.

-- 
Rgrds, legion

Re: [make-initrd] [PATCH v4 0/4] split pipeline into 4 features

[Leonid Krivoshein]

Привет!


06.11.2021 16:28, Alexey Gladkov пишет:
> On Wed, Oct 27, 2021 at 01:09:59AM +0300, Leonid Krivoshein wrote:
>> 27.10.2021 0:13, Vladimir D. Seleznev пишет:
>>> У меня большая просьба: присылать патчи одним тредом. Сейчас в рассылке
>>> куча разрозненных тредов, очень сложно стало ориентироваться.
> +1
>
>> А у меня есть подозрение, что даже одним письмом не всех подписчиков этой
>> рассылки интересует вопрос апстрима bootchain/altboot, так-то и мне было бы
>> удобней вложить патчи в одно письмо, но сказали делать разными. Поэтому
>> надеюсь следующую итерацию реализовать через pull request, благо блох в
>> свитере вчера удалось завести. :-)
> Володя просит делать git format-patch --thread. Тогда утилита добавит
> In-Reply-To в патчи и получится тред, если патчи отослать как они есть.

Понял, попробую. Не получалось отправить "как есть", но раз разобрался с 
настройками smtp, может теперь и получится.


-- 
Best regards,
Leonid Krivoshein.