Make-initrd development discussion
 help / color / mirror / Atom feed
From: Leonid Krivoshein <klark.devel@gmail.com>
To: make-initrd@lists.altlinux.org
Subject: Re: [make-initrd] [PATCH v1 21/41] fork pipeline: bootchain sysvinit script now use external hooks
Date: Mon, 27 Sep 2021 20:22:05 +0300
Message-ID: <245f1b4c-b417-11b7-57c3-6650ade2ef75@gmail.com> (raw)
In-Reply-To: <20210927135745.35ejs7bg67mdjxbx@example.org>


27.09.2021 16:57, Alexey Gladkov пишет:
> On Mon, Sep 27, 2021 at 04:11:43PM +0300, Leonid Krivoshein wrote:
>> [...]
>>>> Другим компонентам bootchain при расширении (а теперь это уже более
>>>> десятка фич и подпакетов) тоже может потребоваться подобная инициализация.
>>>> Здесь в одном коммите видно, куда переехал код функции prepare(). Хуки нужны
>>>> для того, чтобы вызывать подобную инициализацию из определённого места, так
>>>> как базовый пакет bootchain-core может не знать, с какими компонентами
>>>> bootchain собран initramfs. Соответственно, включаемые скрипты кладутся в
>>>> /lib/bootchain-prepare.d. Если потребуется, по аналогии можно добавить в
>>>> stop() хуки для деинициализации.
>>> https://github.com/osboot/make-initrd/blob/master/data/etc/rc.d/rc#L32-L38
>>> https://github.com/osboot/make-initrd/blob/master/data/etc/rc.d/rc#L61
>>> https://github.com/osboot/make-initrd/blob/master/data/etc/rc.d/rc#L109
>>> https://github.com/osboot/make-initrd/blob/master/data/etc/rc.d/rc#L114
>>>
>>> Ты имеешь в виду вот такие хуки ?
>> Ну очень похожие, хотя use_hooks() явно проще -- без блокировки консоли, без
>> бита выполнения, просто соурсим исходники:
> Эти хуки уже давно есть и можно использовать. Не нужно дублировать этот же
> функционал.

Дойдя до этого места не понял, как их можно использовать...

Делаю скрипт выполняемым, а класть-то его куда? В 
/lib/initrd/pre/bootchain-core/ ?


>>>>>> +use_hooks()
>>>>>> +{
>>>>>> +	local hook hdir="$1"
>>>>>> +
>>>>>> +	if [ -d "$hdir" ]; then
>>>>>> +		# shellcheck disable=SC2012
>>>>>> +		for hook in $(ls -1 -- "$hdir"/* |sort) _; do
>>>>>> +			[ -s "$hook" ] ||
>>>>>> +				continue
>>>>>> +			. "$hook"
>>>>>> +		done
>>>>>> +	fi
>>>>>>     }
>> и:
>>>>>> --- /dev/null
>>>>>> +++ b/make-initrd/features/bootchain-waitdev/data/lib/bootchain-prepare.d/300-waitdev
>>>>>> @@ -0,0 +1,12 @@
>>>>>> +#!/bin/bash -efu
>>>>>> +
>>>>>> +dir=/.initrd/bootchain/waitdev
>>>>>> +mkdir -p -- "$dir"
>>>>>> +
>>>>>> +[ -z "${WAITDEV_TIMEOUT-}" ] ||
>>>>>> +	printf '%s\n' "$WAITDEV_TIMEOUT" >"$dir/TIMECNT"
>>>>>> +i=0
>>>>>> +while [ "$i" -lt "${WAITDEV:-0}" ]; do
>>>>>> +	touch "$dir/$i"
>>>>>> +	i=$((1 + $i))
>>>>>> +done
>>

-- 
Best regards,
Leonid Krivoshein.



  reply	other threads:[~2021-09-27 17:22 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-09-24 15:57 Leonid Krivoshein
2021-09-26 14:09 ` Alexey Gladkov
2021-09-26 19:53   ` Leonid Krivoshein
2021-09-27  9:09     ` Alexey Gladkov
2021-09-27 13:11       ` Leonid Krivoshein
2021-09-27 13:57         ` Alexey Gladkov
2021-09-27 17:22           ` Leonid Krivoshein [this message]
2021-09-28  0:40             ` Leonid Krivoshein
2021-09-28  7:11             ` 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=245f1b4c-b417-11b7-57c3-6650ade2ef75@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