On Wed, Jul 25, 2018 at 4:17 PM, Burton, Ross <[email protected]> wrote: > On 25 July 2018 at 21:05, Bruce Ashfield <[email protected]> wrote: >> My question about the change .. is why ? I really don't like cleanup >> patches on principle, and want things pinned to a bug, a performance >> issue or something else. > > See a post by RP to the oe-core list a few days ago: a general cleanup > of Python code that spawns is well overdue. In this case > oe.utils.getstatusoutpus is a wrapper that just calls a deprecated > function in subprocess, is best to pass arguments as an array instead > of string, best not to use a shell, etc.
ah ok. I still don't like it, but it is what it is. My comment stands, I'll figure out how to incorporate this and send it along with my next pull request. Bruce > > Ross -- "Thou shalt not follow the NULL pointer, for chaos and madness await thee at its end" -- _______________________________________________ Openembedded-core mailing list [email protected] http://lists.openembedded.org/mailman/listinfo/openembedded-core
