From: Leonid Krivoshein <klark.devel@gmail.com>
To: make-initrd@lists.altlinux.org
Subject: Re: [make-initrd] [PATCH v6 10/22] bootchain-core: expanded addressing capabilities
Date: Sat, 30 Oct 2021 05:13:49 +0300
Message-ID: <2f91b485-6744-cd07-b1b8-46fcb250c713@gmail.com> (raw)
In-Reply-To: <YXyIu6AlXUaHj+gj@glebfm.cloud.tilaa.com>
30.10.2021 2:50, Gleb Fotengauer-Malinovskiy пишет:
> On Tue, Oct 26, 2021 at 03:58:34PM +0200, Alexey Gladkov wrote:
>> On Tue, Oct 26, 2021 at 02:28:53PM +0300, Leonid Krivoshein wrote:
>>> 26.10.2021 14:12, Alexey Gladkov пишет:
>>>> On Sun, Oct 24, 2021 at 08:21:28PM +0300, Leonid Krivoshein wrote:
>>>>> Adds support for reverse addressing relative to the current step,
>>>>> as well as calculating the path to the special device node obtained
>>>>> in the previous step.
>>>>>
>>>>> Signed-off-by: Leonid Krivoshein <klark.devel@gmail.com>
>>>>> ---
>>>>> features/bootchain-core/README.md | 6 +++
>>>>> .../data/bin/bootchain-sh-functions | 37 ++++++++++++++++---
>>>>> 2 files changed, 38 insertions(+), 5 deletions(-)
>>>>>
>>>>> diff --git a/features/bootchain-core/README.md b/features/bootchain-core/README.md
>>>>> index 8d341e0..ba66259 100644
>>>>> --- a/features/bootchain-core/README.md
>>>>> +++ b/features/bootchain-core/README.md
>>>>> @@ -49,6 +49,12 @@ us to optimize fill in `initramfs` only which we are need.
>>>>> - Modularity: loading methods are initially separated from the common
>>>>> code and daemon.
>>>>> +- Via resolve_target() supports not only forward, but also reverse addressing,
>>>>> + relative to the current step. For example, a record like `step-3/dir1/dev`
>>>>> + will process the result of `dir1/dev`, made in the third step from the current
>>>>> + one. Together with the overload of the chain of steps, direct addressing is safe
>>>>> + only when storing the numbers of the completed steps in files, whereas reverse
>>>>> + relative addressing it is safe in any case and can often be more convenient.
>>>> На мой взгляд очень непонятный синтаксис. step1 и step-1 кажется
>>>> одинаковым. Хотя бы current-1
>>> OK. Давай переделаю так:
>>>
>>> step<N> = pipe<N> -- прямая адресация, так первоначально и было.
>>> curr-<N> -- обратная относительная адресация.
>>>
>>> Так будет лучше?
>> Мне будет понятнее. Но я хотел бы спросить помощи зала.
>>
>> Глеб, Антон, как вы думаете, как будет понятнее ?
> Если из предложенного выбирать, то curr-<N> однозначно лучше.
>
> Но мне кажется, что тут - не читается как минус. Я бы скорее предложил
> что-то вроде prev<N> или back<N>.
Отлично, тогда переделаю на back<N> !
Как я понял, по остальным моментам Алексей пока взял паузу...
--
Best regards,
Leonid Krivoshein.
next prev parent reply other threads:[~2021-10-30 2:13 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-10-26 11:12 ` Alexey Gladkov
2021-10-26 11:28 ` Leonid Krivoshein
2021-10-26 13:58 ` Alexey Gladkov
2021-10-29 23:50 ` Gleb Fotengauer-Malinovskiy
2021-10-30 2:13 ` Leonid Krivoshein [this message]
2021-11-06 12:47 ` Alexey Gladkov
2021-11-06 14:26 ` Leonid Krivoshein
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=2f91b485-6744-cd07-b1b8-46fcb250c713@gmail.com \
--to=klark.devel@gmail.com \
--cc=make-initrd@lists.altlinux.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Make-initrd development discussion
This inbox may be cloned and mirrored by anyone:
git clone --mirror http://lore.altlinux.org/make-initrd/0 make-initrd/git/0.git
# If you have public-inbox 1.1+ installed, you may
# initialize and index your mirror using the following commands:
public-inbox-init -V2 make-initrd make-initrd/ http://lore.altlinux.org/make-initrd \
make-initrd@lists.altlinux.org make-initrd@lists.altlinux.ru make-initrd@lists.altlinux.com
public-inbox-index make-initrd
Example config snippet for mirrors.
Newsgroup available over NNTP:
nntp://lore.altlinux.org/org.altlinux.lists.make-initrd
AGPL code for this site: git clone https://public-inbox.org/public-inbox.git