public inbox for isar-users@googlegroups.com
 help / color / mirror / Atom feed
* [PATCH] meta/classes/sdk: use rootfs_do_umounts to unmount the filesystem
@ 2025-01-17 14:25 srinuvasan.a via isar-users
  2025-01-27 13:28 ` Uladzimir Bely
  0 siblings, 1 reply; 2+ messages in thread
From: srinuvasan.a via isar-users @ 2025-01-17 14:25 UTC (permalink / raw)
  To: isar-users; +Cc: jan.kiszka, srinuvasan

From: srinuvasan <srinuvasan.a@siemens.com>

Use the rootfs_do_umounts function instead of duplicating the unmount
filesystem.

Signed-off-by: srinuvasan <srinuvasan.a@siemens.com>
---
 meta/classes/sdk.bbclass | 12 ++----------
 1 file changed, 2 insertions(+), 10 deletions(-)

diff --git a/meta/classes/sdk.bbclass b/meta/classes/sdk.bbclass
index c69f7096..46436d97 100644
--- a/meta/classes/sdk.bbclass
+++ b/meta/classes/sdk.bbclass
@@ -90,16 +90,8 @@ sdkchroot_configscript () {
 
 ROOTFS_POSTPROCESS_COMMAND:append:class-sdk = " sdkchroot_finalize"
 sdkchroot_finalize() {
-    mountpoint -q "${ROOTFSDIR}/dev/pts" && \
-        sudo umount "${ROOTFSDIR}/dev/pts"
-    mountpoint -q "${ROOTFSDIR}/dev/shm" && \
-        sudo umount "${ROOTFSDIR}/dev/shm"
-    mountpoint -q "${ROOTFSDIR}/dev" && \
-        sudo umount "${ROOTFSDIR}/dev"
-    mountpoint -q "${ROOTFSDIR}/proc" && \
-        sudo umount "${ROOTFSDIR}/proc"
-    mountpoint -q "${ROOTFSDIR}/sys" && \
-        sudo umount "${ROOTFSDIR}/sys"
+
+    rootfs_do_umounts
 
     # Remove setup scripts
     sudo rm -f ${ROOTFSDIR}/chroot-setup.sh ${ROOTFSDIR}/configscript.sh
-- 
2.39.5

-- 
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/20250117142513.264551-1-srinuvasan.a%40siemens.com.

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [PATCH] meta/classes/sdk: use rootfs_do_umounts to unmount the filesystem
  2025-01-17 14:25 [PATCH] meta/classes/sdk: use rootfs_do_umounts to unmount the filesystem srinuvasan.a via isar-users
@ 2025-01-27 13:28 ` Uladzimir Bely
  0 siblings, 0 replies; 2+ messages in thread
From: Uladzimir Bely @ 2025-01-27 13:28 UTC (permalink / raw)
  To: srinuvasan.a, isar-users

On Fri, 2025-01-17 at 19:55 +0530, srinuvasan.a via isar-users wrote:
> From: srinuvasan <srinuvasan.a@siemens.com>
> 
> Use the rootfs_do_umounts function instead of duplicating the unmount
> filesystem.
> 
> Signed-off-by: srinuvasan <srinuvasan.a@siemens.com>
> ---
>  meta/classes/sdk.bbclass | 12 ++----------
>  1 file changed, 2 insertions(+), 10 deletions(-)
> 
> diff --git a/meta/classes/sdk.bbclass b/meta/classes/sdk.bbclass
> index c69f7096..46436d97 100644
> --- a/meta/classes/sdk.bbclass
> +++ b/meta/classes/sdk.bbclass
> @@ -90,16 +90,8 @@ sdkchroot_configscript () {
>  
>  ROOTFS_POSTPROCESS_COMMAND:append:class-sdk = " sdkchroot_finalize"
>  sdkchroot_finalize() {
> -    mountpoint -q "${ROOTFSDIR}/dev/pts" && \
> -        sudo umount "${ROOTFSDIR}/dev/pts"
> -    mountpoint -q "${ROOTFSDIR}/dev/shm" && \
> -        sudo umount "${ROOTFSDIR}/dev/shm"
> -    mountpoint -q "${ROOTFSDIR}/dev" && \
> -        sudo umount "${ROOTFSDIR}/dev"
> -    mountpoint -q "${ROOTFSDIR}/proc" && \
> -        sudo umount "${ROOTFSDIR}/proc"
> -    mountpoint -q "${ROOTFSDIR}/sys" && \
> -        sudo umount "${ROOTFSDIR}/sys"
> +
> +    rootfs_do_umounts
>  
>      # Remove setup scripts
>      sudo rm -f ${ROOTFSDIR}/chroot-setup.sh
> ${ROOTFSDIR}/configscript.sh
> -- 
> 2.39.5
> 

Applied to next, thanks.

-- 
Best regards,
Uladzimir.



-- 
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/9d988fd8896a2b4744e474aedb2d838071bbc024.camel%40ilbers.de.

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2025-01-27 13:28 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2025-01-17 14:25 [PATCH] meta/classes/sdk: use rootfs_do_umounts to unmount the filesystem srinuvasan.a via isar-users
2025-01-27 13:28 ` Uladzimir Bely

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox