[PATCH 1/1] rootfs: remove temporary sstate deploy directory after task execution

[PATCH 1/1] rootfs: remove temporary sstate deploy directory after task execution

['Felix Moessbauer' via isar-users]

The rootfs_install_sstate_prepare currently leaves an empty directory
skeleton behind. As this does not have any value for debugging, we
remove it.

Signed-off-by: Felix Moessbauer <felix.moessbauer@siemens.com>
---
 meta/classes-recipe/rootfs.bbclass | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/meta/classes-recipe/rootfs.bbclass b/meta/classes-recipe/rootfs.bbclass
index 73ffa6b2..9044bf93 100644
--- a/meta/classes-recipe/rootfs.bbclass
+++ b/meta/classes-recipe/rootfs.bbclass
@@ -662,6 +662,8 @@ rootfs_install_sstate_prepare() {
     # tar --one-file-system will cross bind-mounts to the same filesystem,
     # so we use some mount magic to prevent that
     mkdir -p ${WORKDIR}/mnt/rootfs
+    trap 'rmdir ${WORKDIR}/mnt/rootfs ${WORKDIR}/mnt' EXIT
+
     run_privileged_heredoc <<'EOF' 3> rootfs.tar
         mount -o bind,private '${ROOTFSDIR}' '${WORKDIR}/mnt/rootfs' -o ro
         lopts="--one-file-system --exclude=var/cache/apt/archives"
-- 
2.51.0

-- 
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/20260223133255.1025809-1-felix.moessbauer%40siemens.com.

Re: [PATCH 1/1] rootfs: remove temporary sstate deploy directory after task execution

['MOESSBAUER, Felix' via isar-users]

On Mon, 2026-02-23 at 14:32 +0100, Felix Moessbauer wrote:
> The rootfs_install_sstate_prepare currently leaves an empty directory
> skeleton behind. As this does not have any value for debugging, we
> remove it.
> 
> Signed-off-by: Felix Moessbauer <felix.moessbauer@siemens.com>
> ---
>  meta/classes-recipe/rootfs.bbclass | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/meta/classes-recipe/rootfs.bbclass b/meta/classes-recipe/rootfs.bbclass
> index 73ffa6b2..9044bf93 100644
> --- a/meta/classes-recipe/rootfs.bbclass
> +++ b/meta/classes-recipe/rootfs.bbclass
> @@ -662,6 +662,8 @@ rootfs_install_sstate_prepare() {
>      # tar --one-file-system will cross bind-mounts to the same filesystem,
>      # so we use some mount magic to prevent that
>      mkdir -p ${WORKDIR}/mnt/rootfs
> +    trap 'rmdir ${WORKDIR}/mnt/rootfs ${WORKDIR}/mnt' EXIT
> +

PS: I found this while working on the rootless series. I just noticed,
that this patch does not apply as-is on isar next, but the idea behind
it should be clear.

Felix

>      run_privileged_heredoc <<'EOF' 3> rootfs.tar
>          mount -o bind,private '${ROOTFSDIR}' '${WORKDIR}/mnt/rootfs' -o ro
>          lopts="--one-file-system --exclude=var/cache/apt/archives"
> -- 
> 2.51.0

-- 
Siemens AG
Linux Expert Center
Friedrich-Ludwig-Bauer-Str. 3
85748 Garching, Germany

-- 
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/fd505a40fb9fda75a301d652fa2e468ceb0056f5.camel%40siemens.com.

Re: [PATCH 1/1] rootfs: remove temporary sstate deploy directory after task execution

[Zhihang Wei]

Rebased onto next and applied. Thanks.

Zhihang

On 2/23/26 14:32, 'Felix Moessbauer' via isar-users wrote:
> The rootfs_install_sstate_prepare currently leaves an empty directory
> skeleton behind. As this does not have any value for debugging, we
> remove it.
>
> Signed-off-by: Felix Moessbauer <felix.moessbauer@siemens.com>
> ---
>   meta/classes-recipe/rootfs.bbclass | 2 ++
>   1 file changed, 2 insertions(+)
>
> diff --git a/meta/classes-recipe/rootfs.bbclass b/meta/classes-recipe/rootfs.bbclass
> index 73ffa6b2..9044bf93 100644
> --- a/meta/classes-recipe/rootfs.bbclass
> +++ b/meta/classes-recipe/rootfs.bbclass
> @@ -662,6 +662,8 @@ rootfs_install_sstate_prepare() {
>       # tar --one-file-system will cross bind-mounts to the same filesystem,
>       # so we use some mount magic to prevent that
>       mkdir -p ${WORKDIR}/mnt/rootfs
> +    trap 'rmdir ${WORKDIR}/mnt/rootfs ${WORKDIR}/mnt' EXIT
> +
>       run_privileged_heredoc <<'EOF' 3> rootfs.tar
>           mount -o bind,private '${ROOTFSDIR}' '${WORKDIR}/mnt/rootfs' -o ro
>           lopts="--one-file-system --exclude=var/cache/apt/archives"

-- 
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/24a30178-dc88-4cc8-af02-72f909bc8270%40ilbers.de.