[PATCH] rm_work: Allow pseudo directory to be in the excludes

[PATCH] rm_work: Allow pseudo directory to be in the excludes

[Jan-Simon Möller]

From: Jan-Simon Möller <dl9pf@gmx.de>

While debugging the pseudo issues, rm_work was always removing the pseudo folder
even when it was in the RM_WORK_EXCLUDE_ITEMS variable.

This patch puts the pseudo case within the conditional.

Signed-off-by: Jan-Simon Möller <dl9pf@gmx.de>
---
 meta/classes/rm_work.bbclass | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/meta/classes/rm_work.bbclass b/meta/classes/rm_work.bbclass
index 01c2ab1c78..0e52fb6ae6 100644
--- a/meta/classes/rm_work.bbclass
+++ b/meta/classes/rm_work.bbclass
@@ -99,10 +99,12 @@ do_rm_work () {
     do
         # Retain only logs and other files in temp, safely ignore
         # failures of removing pseudo folers on NFS2/3 server.
-        if [ $dir = 'pseudo' ]; then
-            rm -rf $dir 2> /dev/null || true
-        elif ! echo "$excludes" | grep -q -w "$dir"; then
-            rm -rf $dir
+        if ! echo "$excludes" | grep -q -w "$dir"; then
+            if [ $dir = 'pseudo' ]; then
+                rm -rf $dir 2> /dev/null || true
+            else
+                rm -rf $dir
+            fi
         fi
     done
 }
--
2.26.2

Re: [OE-core] [PATCH] rm_work: Allow pseudo directory to be in the excludes

[Richard Purdie]

On Sat, 2020-10-03 at 18:34 +0200, Jan-Simon Möller wrote:
> From: Jan-Simon Möller <dl9pf@gmx.de>
> 
> While debugging the pseudo issues, rm_work was always removing the
> pseudo folder
> even when it was in the RM_WORK_EXCLUDE_ITEMS variable.
> 
> This patch puts the pseudo case within the conditional.

I've mixed feelings on this. Given the things rm_work removes, it has
to remove the pseudo database as well or you will get path mismatch
errors. I suspect we intentionally don't let the user do this.

Have you had this running without issues?

Cheers,

Richard