On Tue, Feb 18, 2014 at 5:27 AM, Martin Jansa <[email protected]>wrote:
> Maybe we should change meta/classes/package.bbclass to > fail when some filedeprunner call fails like this and fix > filedeprunner to escape '()' and other possibly dangerous chars > it's called like this: > processed = list(pool.imap(oe.package.filedeprunner, pkglist)) > We'd likely be better off just avoiding os.popen entirely in favor of subprocess based operations, which generally support specifying the command arguments as a list, and can bypass the shell entirely, avoiding the need to escape anything. Alternatively, use os.popen2(), which can accept a list to bypass the shell as well. -- Christopher Larson clarson at kergoth dot com Founder - BitBake, OpenEmbedded, OpenZaurus Maintainer - Tslib Senior Software Engineer, Mentor Graphics
_______________________________________________ Openembedded-core mailing list [email protected] http://lists.openembedded.org/mailman/listinfo/openembedded-core
