Re: [PATCH 0/4] replace os.system/popen with subprocess module

[Richard Purdie <richard.purdie@linuxfoundation.org>]

On Tue, 2012-05-29 at 22:53 +0800, Robert Yang wrote:
> These patches are based on oe-core.
> 
> * Testinfo
>   1) $ bitbake core-image-sato core-image-sato-sdk metatoolchain-sdk with
>         MACHINE=qemuarm, PACKAGE_CLASSES = "package_rpm"
> 
>   2) $ bitbake core-image-sato core-image-sato-sdk metatoolchain-sdk with
>         MACHINE=qemuarm, PACKAGE_CLASSES = "package_ipk"
> 
>   3) $ bitbake core-image-sato with MACHINE=qemux86, PACKAGE_CLASSES = "package_deb"
> 
>   4) $ bitbake world with MACHINE=qemux86
> 
> // Robert
> 
> The following changes since commit a7532d6b2870a51079c39366def9ae55faeba626:
> 
>   rootfs_rpm.bbclass: save rpmlib rather than remove it (2012-05-29 22:36:39 +0800)
> 
> are available in the git repository at:
>   git://git.pokylinux.org/poky-contrib robert/meta_subprocess
>   http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=robert/meta_subprocess
> 
> Robert Yang (4):
>   meta: replace os.system with subprocess.call
>   scripts: replace os.system with subprocess.call
>   meta: replace os.popen with subprocess.Popen

I took the first three of these, thanks.

>   scripts: replace os.popen with subprocess.Popen

As Chris says, we may as well use an unlink call for this case.

Cheers,

Richard