[meta-networking][PATCH] autofs: fix do_install failure and remove unused /var/run directory

[meta-networking][PATCH] autofs: fix do_install failure and remove unused /var/run directory

[Qi.Chen]

From: Chen Qi <Qi.Chen@windriver.com>

If ${D}/run doesn't exist, the following statement will have non-zero
exit code, which leads to do_install failure.
     test -d ${D}/run && rmdir ${D}/run

Modify it to use the if statement to make the do_install task succeed
no matter ${D}/run exists or not.

Also, this patch removes the unused /var/run directory. Leaving that
directory there will result in /run directory created before splitting
package, which in turn lead to 'installed-vs-shipped' QA error or QA
warning.

Signed-off-by: Chen Qi <Qi.Chen@windriver.com>
---
 .../recipes-daemons/autofs/autofs_5.0.7.bb         |    7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/meta-networking/recipes-daemons/autofs/autofs_5.0.7.bb b/meta-networking/recipes-daemons/autofs/autofs_5.0.7.bb
index f8df0ef..c4eea31 100644
--- a/meta-networking/recipes-daemons/autofs/autofs_5.0.7.bb
+++ b/meta-networking/recipes-daemons/autofs/autofs_5.0.7.bb
@@ -96,7 +96,12 @@ do_configure_prepend () {
 }
 
 do_install_append () {
-    test -d ${D}/run && rmdir ${D}/run
+    if [ -d ${D}/run ]; then
+	rmdir ${D}/run
+    fi
+    if [ -d ${D}${localstatedir}/run ]; then
+	rmdir ${D}${localstatedir}/run
+    fi
 }
 
 INSANE_SKIP_${PN} = "dev-so"
-- 
1.7.9.5

Re: [meta-networking][PATCH] autofs: fix do_install failure and remove unused /var/run directory

[Joe MacDonald]

[-- Attachment #1: Type: text/plain, Size: 1572 bytes --]

Merged, thanks.
-J.

[[oe][meta-networking][PATCH] autofs: fix do_install failure and remove unused /var/run directory] On 13.10.24 (Thu 12:47) Qi.Chen@windriver.com wrote:

> From: Chen Qi <Qi.Chen@windriver.com>
> 
> If ${D}/run doesn't exist, the following statement will have non-zero
> exit code, which leads to do_install failure.
>      test -d ${D}/run && rmdir ${D}/run
> 
> Modify it to use the if statement to make the do_install task succeed
> no matter ${D}/run exists or not.
> 
> Also, this patch removes the unused /var/run directory. Leaving that
> directory there will result in /run directory created before splitting
> package, which in turn lead to 'installed-vs-shipped' QA error or QA
> warning.
> 
> Signed-off-by: Chen Qi <Qi.Chen@windriver.com>
> ---
>  .../recipes-daemons/autofs/autofs_5.0.7.bb         |    7 ++++++-
>  1 file changed, 6 insertions(+), 1 deletion(-)
> 
> diff --git a/meta-networking/recipes-daemons/autofs/autofs_5.0.7.bb b/meta-networking/recipes-daemons/autofs/autofs_5.0.7.bb
> index f8df0ef..c4eea31 100644
> --- a/meta-networking/recipes-daemons/autofs/autofs_5.0.7.bb
> +++ b/meta-networking/recipes-daemons/autofs/autofs_5.0.7.bb
> @@ -96,7 +96,12 @@ do_configure_prepend () {
>  }
>  
>  do_install_append () {
> -    test -d ${D}/run && rmdir ${D}/run
> +    if [ -d ${D}/run ]; then
> +	rmdir ${D}/run
> +    fi
> +    if [ -d ${D}${localstatedir}/run ]; then
> +	rmdir ${D}${localstatedir}/run
> +    fi
>  }
>  
>  INSANE_SKIP_${PN} = "dev-so"
-- 
-Joe MacDonald.
:wq

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 205 bytes --]