[PATCH] isar-bootstrap-helper: don't use USER environment variable

[PATCH] isar-bootstrap-helper: don't use USER environment variable

[Maxim Yu. Osipov]

In some build environments USER variable is not set.
Use `whoami` instead.

Suggested-by: Henning Schild <henning.schild@siemens.com>
Signed-off-by: Maxim Yu. Osipov <mosipov@ilbers.de>
---
 meta/classes/isar-bootstrap-helper.bbclass | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/classes/isar-bootstrap-helper.bbclass b/meta/classes/isar-bootstrap-helper.bbclass
index a316fe8..1ac39db 100644
--- a/meta/classes/isar-bootstrap-helper.bbclass
+++ b/meta/classes/isar-bootstrap-helper.bbclass
@@ -140,7 +140,7 @@ setup_root_file_system() {
         if [ ${KEEP_APT_CACHE} -eq 1 ]; then
             mkdir -p ${WORKDIR}/apt_cache
             sudo mv $(find $ROOTFSDIR/var/cache/apt -name '*.deb') ${WORKDIR}/apt_cache
-            sudo chown $USER ${WORKDIR}/apt_cache/*
+            sudo chown `whoami` ${WORKDIR}/apt_cache/*
         fi
         sudo -E chroot "$ROOTFSDIR" \
             /usr/bin/apt-get autoremove --purge --yes
-- 
2.11.0

Re: [PATCH] isar-bootstrap-helper: don't use USER environment variable

[chombourger]

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


> diff --git a/meta/classes/isar-bootstrap-helper.bbclass b/meta/classes/isar-bootstrap-helper.bbclass
> index a316fe8..1ac39db 100644
> --- a/meta/classes/isar-bootstrap-helper.bbclass
> +++ b/meta/classes/isar-bootstrap-helper.bbclass
> @@ -140,7 +140,7 @@ setup_root_file_system() {
>          if [ ${KEEP_APT_CACHE} -eq 1 ]; then
>              mkdir -p ${WORKDIR}/apt_cache
>              sudo mv $(find $ROOTFSDIR/var/cache/apt -name '*.deb') ${WORKDIR}/apt_cache
> -            sudo chown $USER ${WORKDIR}/apt_cache/*
> +            sudo chown `whoami` ${WORKDIR}/apt_cache/*

Consider using the new-style syntax $(whoami) instead

>          fi
>          sudo -E chroot "$ROOTFSDIR" \
>              /usr/bin/apt-get autoremove --purge --yes
> -- 
> 2.11.0