[PATCH] Change ownership of WORKDIR prior to unpacking

[PATCH] Change ownership of WORKDIR prior to unpacking

[Jan Kiszka]

From: Jan Kiszka <jan.kiszka@siemens.com>

This ensures we can delete old artifacts that may be root-owned after a
build run.

Signed-off-by: Jan Kiszka <jan.kiszka@siemens.com>
---
 meta/classes/base.bbclass | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/meta/classes/base.bbclass b/meta/classes/base.bbclass
index b0e174e..f83c4db 100644
--- a/meta/classes/base.bbclass
+++ b/meta/classes/base.bbclass
@@ -88,12 +88,17 @@ do_unpack[stamp-extra-info] = "${DISTRO}-${DISTRO_ARCH}"
 
 # Unpack package and put it into working directory
 python do_unpack() {
+    import subprocess
+
     src_uri = (d.getVar('SRC_URI', True) or "").split()
     if len(src_uri) == 0:
         return
 
     rootdir = d.getVar('WORKDIR', True)
 
+    uid = str(os.getuid())
+    subprocess.call('sudo chown -R ' + uid + ' ' + rootdir, shell=True)
+
     try:
         fetcher = bb.fetch2.Fetch(src_uri, d)
         fetcher.unpack(rootdir)

Re: [PATCH] Change ownership of WORKDIR prior to unpacking

[Alexander Smirnov]

On 02/18/2018 10:38 PM, Jan Kiszka wrote:
> From: Jan Kiszka <jan.kiszka@siemens.com>
> 
> This ensures we can delete old artifacts that may be root-owned after a
> build run.

Applied to next, thanks.

P.S: after this patch I believe there will be no more proposals to run 
whole bitbake under 'sudo'... :-)

Alex

> 
> Signed-off-by: Jan Kiszka <jan.kiszka@siemens.com>
> ---
>   meta/classes/base.bbclass | 5 +++++
>   1 file changed, 5 insertions(+)
> 
> diff --git a/meta/classes/base.bbclass b/meta/classes/base.bbclass
> index b0e174e..f83c4db 100644
> --- a/meta/classes/base.bbclass
> +++ b/meta/classes/base.bbclass
> @@ -88,12 +88,17 @@ do_unpack[stamp-extra-info] = "${DISTRO}-${DISTRO_ARCH}"
>   
>   # Unpack package and put it into working directory
>   python do_unpack() {
> +    import subprocess
> +
>       src_uri = (d.getVar('SRC_URI', True) or "").split()
>       if len(src_uri) == 0:
>           return
>   
>       rootdir = d.getVar('WORKDIR', True)
>   
> +    uid = str(os.getuid())
> +    subprocess.call('sudo chown -R ' + uid + ' ' + rootdir, shell=True)
> +
>       try:
>           fetcher = bb.fetch2.Fetch(src_uri, d)
>           fetcher.unpack(rootdir)
>