summaryrefslogtreecommitdiff
path: root/meta
diff options
context:
space:
mode:
authorBjørn Forsman <bjorn.forsman@gmail.com>2016-04-24 18:10:35 -0700
committerRichard Purdie <richard.purdie@linuxfoundation.org>2016-05-09 14:37:19 +0100
commite49794b9fe3391073138cb6116a46b37dd5119e7 (patch)
tree58a1ca8c06549370433df03ad22e80df4097f87d /meta
parenta4808f800f856fb01761f4835f6a87e736349994 (diff)
downloadopenembedded-core-e49794b9fe3391073138cb6116a46b37dd5119e7.tar.gz
openembedded-core-e49794b9fe3391073138cb6116a46b37dd5119e7.tar.bz2
openembedded-core-e49794b9fe3391073138cb6116a46b37dd5119e7.zip
license.bbclass: fix warnings when run in unprivileged "container" env
An unprivileged "container" environment like this[1] doesn't have root account (uid 0) which causes tons of "Invalid argument" warnings: $ bitbake ... ... WARNING: Could not copy license file [src] to [dest]: [Errno 22] Invalid argument: '[src]' WARNING: Could not copy license file [src] to [dest]: [Errno 22] Invalid argument: '[src]' WARNING: Could not copy license file [src] to [dest]: [Errno 22] Invalid argument: '[src]' ... Fix it by handling EINVAL similar to existing handling of EPERM (which was added for when not running under pseudo). [1]: The real environemnt is buildFHSUserEnv from NixOS/nixpkgs, but a demonstration of the issue can be done like this: $ touch f $ unshare --user --mount chown 0:0 f chown: changing ownership of ‘f’: Invalid argument (From OE-Core master rev: d00b2250a6afebd7d1373c04b4006290f0cd4043) Signed-off-by: Bjørn Forsman <bjorn.forsman@gmail.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org> Signed-off-by: Robert Yang <liezhi.yang@windriver.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'meta')
-rw-r--r--meta/classes/license.bbclass8
1 files changed, 5 insertions, 3 deletions
diff --git a/meta/classes/license.bbclass b/meta/classes/license.bbclass
index c714da31f4..db0fc51fea 100644
--- a/meta/classes/license.bbclass
+++ b/meta/classes/license.bbclass
@@ -189,9 +189,11 @@ def copy_license_files(lic_files_paths, destdir):
os.chown(dst,0,0)
except OSError as err:
import errno
- if err.errno == errno.EPERM:
- # suppress "Operation not permitted" error, as
- # sometimes this function is not executed under pseudo
+ if err.errno in (errno.EPERM, errno.EINVAL):
+ # Suppress "Operation not permitted" error, as
+ # sometimes this function is not executed under pseudo.
+ # Also ignore "Invalid argument" errors that happen in
+ # some (unprivileged) container environments (no root).
pass
else:
raise