From 91fdb8158136cef27c01906b3d85920c5dd5568a Mon Sep 17 00:00:00 2001 From: Richard Purdie Date: Wed, 25 Sep 2013 12:36:46 +0000 Subject: [PATCH] package.bbclass: Clear umask when using os.mkdir We switched to using os.mkdir with the file creation mode specified as the second parameter. Python masks this with umask behind the scenes which isn't what we want, we really want the permissions we specify. To avoid this we zero the umask beforehand and restore afterwards. Other solutions are possible but would not perform as well which is why we're using os.mkdir in the first place. Martin Jansa deserves the credit for debugging where the problem was. (From OE-Core rev: f91226553e39439bfd17ab2b06c56cb8bf41061b) Signed-off-by: Richard Purdie Signed-off-by: Saul Wold Signed-off-by: Richard Purdie --- meta/classes/package.bbclass | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/meta/classes/package.bbclass b/meta/classes/package.bbclass index fbb6839..c98c6ec 100644 --- a/meta/classes/package.bbclass +++ b/meta/classes/package.bbclass @@ -953,6 +953,9 @@ python populate_packages () { seen = [] + # os.mkdir masks the permissions with umask so we have to unset it first + oldumask = os.umask(0) + for pkg in package_list: root = os.path.join(pkgdest, pkg) bb.utils.mkdirhier(root) @@ -1025,6 +1028,7 @@ python populate_packages () { if ret is False or ret == 0: raise bb.build.FuncFailed("File population failed") + os.umask(oldumask) os.chdir(workdir) unshipped = [] -- 2.7.4