On Wed, 2016-06-01 at 14:57 -0700, Christopher Larson wrote: > > On Wed, Jun 1, 2016 at 5:35 AM, Richard Purdie < > richard.pur...@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?
I'm not entirely sure why I did this but its not needed, the string octal values continue to work just fine. This change doesn't break anything but also doesn't help. I'll take this piece out... Cheers, Richard -- _______________________________________________ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core