On Wed, Jul 25, 2018 at 4:20 PM, Bruce Ashfield <[email protected]> wrote: > 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.
... and I found the original email, I see the part about things being depreciated. Makes sense. I'll queue it up and run some tests on some intentionally bad BSPs. Bruce > > 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" -- "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
