On Wed, Jun 1, 2016 at 5:35 AM, Richard Purdie < richard.purdie@linuxfoundation.org> wrote: > diff --git a/meta/classes/base.bbclass b/meta/classes/base.bbclass > index 4be0a7e..1ba1222 100644 > --- a/meta/classes/base.bbclass > +++ b/meta/classes/base.bbclass > @@ -456,15 +456,15 @@ python () { > # If we're building a target package we need to use fakeroot (pseudo) > # in order to capture permissions, owners, groups and special files > if not bb.data.inherits_class('native', d) and not bb.data.inherits_class('cross', > d): > - d.setVarFlag('do_unpack', 'umask', '022') > - d.setVarFlag('do_configure', 'umask', '022') > - d.setVarFlag('do_compile', 'umask', '022') > + d.setVarFlag('do_unpack', 'umask', '0o022') > + d.setVarFlag('do_configure', 'umask', '0o022') > + d.setVarFlag('do_compile', 'umask', '0o022') > d.appendVarFlag('do_install', 'depends', ' > virtual/fakeroot-native:do_populate_sysroot') > d.setVarFlag('do_install', 'fakeroot', '1') > - d.setVarFlag('do_install', 'umask', '022') > + d.setVarFlag('do_install', 'umask', '0o022') > d.appendVarFlag('do_package', 'depends', ' > virtual/fakeroot-native:do_populate_sysroot') > d.setVarFlag('do_package', 'fakeroot', '1') > - d.setVarFlag('do_package', 'umask', '022') > + d.setVarFlag('do_package', 'umask', '0o022') > d.setVarFlag('do_package_setscene', 'fakeroot', '1') > d.appendVarFlag('do_package_setscene', 'depends', ' > virtual/fakeroot-native:do_populate_sysroot') > d.setVarFlag('do_devshell', 'fakeroot', '1') > Rather than modifying the string values in the metadata, shouldn't we retain compatibility by letting the umask *string* stored in the flag use the old format and if needed convert it when we convert the string to a number to pass to the appropriate os functions? Or was this just a sed gone wild, since these are string values, not python source octal syntax? -- Christopher Larson clarson at kergoth dot com Founder - BitBake, OpenEmbedded, OpenZaurus Maintainer - Tslib Senior Software Engineer, Mentor Graphics