public inbox for isar-users@googlegroups.com
 help / color / mirror / Atom feed
From: Zhihang Wei <wzh@ilbers.de>
To: Clara Kowalsky <clara.kowalsky@siemens.com>, isar-users@googlegroups.com
Cc: quirin.gylstorff@siemens.com
Subject: Re: [PATCH] docs: fix typos in initramfs part
Date: Mon, 8 Dec 2025 14:20:08 +0100	[thread overview]
Message-ID: <eee0cd2d-ed4c-452c-80e9-d17cf02f62ce@ilbers.de> (raw)
In-Reply-To: <20251203141036.952294-1-clara.kowalsky@siemens.com>

Applied to next, thanks.

Zhihang

On 12/3/25 15:10, 'Clara Kowalsky' via isar-users wrote:
> Signed-off-by: Clara Kowalsky <clara.kowalsky@siemens.com>
> ---
>   doc/user_manual.md             | 29 +++++++++++++++--------------
>   meta/classes/initramfs.bbclass |  4 ++--
>   2 files changed, 17 insertions(+), 16 deletions(-)
>
> diff --git a/doc/user_manual.md b/doc/user_manual.md
> index 4df122a8..5a95fdc3 100644
> --- a/doc/user_manual.md
> +++ b/doc/user_manual.md
> @@ -1687,16 +1687,16 @@ SRC_URI += "docker://debian;digest=sha256:f528891ab1aa484bf7233dbcc84f3c806c3e42
>   
>   ## Switch from initramfs-tools to dracut
>   
> -To build a Isar image with dracut as the initramfs generator instead
> -of initramfs-tools in Debian 13(trixie) or previous versions add dracut
> +To build an Isar image with dracut as the initramfs generator instead
> +of initramfs-tools in Debian 13 (trixie) or previous versions, add dracut
>   as a package to the image:
>   
>   ```
>   IMAGE_PREINSTALL +=  "dracut"
>   ```
>   
> -An dracut based initrd contains the file `/usr/lib/initrd-release`. In
> -case of trixie the file has the following content:
> +A dracut-based initrd contains the file `/usr/lib/initrd-release`. In
> +case of trixie, the file has the following content:
>   
>   ```bash
>   NAME=dracut
> @@ -1737,7 +1737,7 @@ For running a custom script during boot-up, provide a bottom-half file in
>   process: `init-top`, `init-premount`, `local-top`, `nfs-top`, `local-block`,
>   `local-premount`, `nfs-premount`, `local-bottom`, `nfs-bottom`, `init-bottom`.
>   If you do not benefit from the script header with its static `SCRIPT_PREREQ`,
> -you may instead provide `init-top-complete`, `init-premount-complete` etc. to
> +you may instead provide `init-top-complete`, `init-premount-complete`, etc., to
>   still use automatic installation while defining the boot script completely
>   yourself.
>   
> @@ -1759,17 +1759,17 @@ It is controlled by following variables:
>   - `DRACUT_REQUIRED_BINARIES` defines the binaries required by the module.
>   - `DRACUT_MODULE_DEPENDENCIES` defines dependencies to other dracut modules.
>   - `DRACUT_MODULE_NO` defines the module number which prefixes the module name
> -to define the execution order.The default is `50`.
> -- `DRACUT_MODULE_NAME` the name of the module which is used to install the
> +to determine the execution order. The default is `50`.
> +- `DRACUT_MODULE_NAME` defines the name of the module which is used to install the
>   module in the initrd or as a dependency to other modules. It defaults to
>   `${PN}` without the prefix `dracut-`.
>   - `DRACUT_MODULE_PATH` contains the path to the installed module. It is set
> -to `${D}/usr/lib/dracut/modules.d/${DRACUT_MODULE_NO}${DRACUT_MODULE_NAME}/`
> +to `${D}/usr/lib/dracut/modules.d/${DRACUT_MODULE_NO}${DRACUT_MODULE_NAME}/`.
>   
>   The `install()` function is added by storing the file `install.sh` in the
>   files directory of the dracut module.
>   
> -Other files can by added to the module by coping them to the Module folder
> +Other files can by added to the module by copying them to the module folder
>   with:
>   ```bash
>   install -m 666 ${WORKDIR}/lighttpd.service ${DRACUT_MODULE_PATH}
> @@ -1794,15 +1794,16 @@ See `isar-initramfs` or `isar-dracut` for an example recipes.
>   
>   #### dracut config
>   
> -A dracut initramfs can be configured by the command line or a configuration file.
> -The use configuration files is preferred:
> +A dracut initramfs can be configured via the command line or a configuration file.
> +The use of configuration files is preferred:
>    - Debian provides dracut-config-* packages
>    - It is easier to upstream and to maintain.
>   
>   The configuration file can be chosen with the variable `DRACUT_CONFIG_PATH`. This variable
> -contains the absolut path to the used configuration in the root file system.
> +contains the absolute path to the used configuration in the root file system.
> +The configuration file should be placed in `/etc/dracut.conf.d` to follow the dracut convention.
>   
> -Still there are some use cases like debugging to add modules via the command line.
> -For this the recipe meta/classes/initrd-dracut.bbclass provides the following options:
> +There are still some use cases, like debugging, where modules can be added via the command line.
> +For this, the recipe meta/classes/initrd-dracut.bbclass provides the following options:
>    - `DRACUT_EXTRA_DRIVERS` add extra drivers to the dracut initrd
>    - `DRACUT_EXTRA_MODULES` add extra modules to the dracut initrd
> diff --git a/meta/classes/initramfs.bbclass b/meta/classes/initramfs.bbclass
> index 862bd873..85b07f28 100644
> --- a/meta/classes/initramfs.bbclass
> +++ b/meta/classes/initramfs.bbclass
> @@ -29,9 +29,9 @@ ROOTFSDIR = "${INITRAMFS_ROOTFS}"
>   ROOTFS_FEATURES = "generate-manifest"
>   ROOTFS_PACKAGES = "${INITRAMFS_GENERATOR_PKG} ${INITRAMFS_PREINSTALL} ${INITRAMFS_INSTALL}"
>   
> -# validate if have incompatible packages in the installation list
> +# validate whether there are incompatible packages in the installation list
>   python do_validate_rootfs_packages () {
> -    # in Debian initramfs-tools specific packages should end or star
> +    # in Debian, initramfs-tools specific packages should end or start
>       # with initramfs
>       # dracut specific packages end with dracut
>       incompatible_initrd_packages = { 'initramfs-tools':['dracut'],

-- 
You received this message because you are subscribed to the Google Groups "isar-users" group.
To unsubscribe from this group and stop receiving emails from it, send an email to isar-users+unsubscribe@googlegroups.com.
To view this discussion visit https://groups.google.com/d/msgid/isar-users/eee0cd2d-ed4c-452c-80e9-d17cf02f62ce%40ilbers.de.

      reply	other threads:[~2025-12-08 13:20 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-12-03 14:10 'Clara Kowalsky' via isar-users
2025-12-08 13:20 ` Zhihang Wei [this message]

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=eee0cd2d-ed4c-452c-80e9-d17cf02f62ce@ilbers.de \
    --to=wzh@ilbers.de \
    --cc=clara.kowalsky@siemens.com \
    --cc=isar-users@googlegroups.com \
    --cc=quirin.gylstorff@siemens.com \
    /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
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox