[poky] [PATCH 1/1] kernel.py: replace os.popen with subprocess.Popen

[Richard Purdie]

On Fri, 2012-06-08 at 08:39 -0700, Darren Hart wrote:
> 
> On 06/08/2012 08:14 AM, Martin Jansa wrote:
> > On Fri, Jun 08, 2012 at 08:08:59AM -0700, Darren Hart wrote:
> >> And here I think subprocess.check_output(...) is preferred.
> > 
> > That's only in python 2.7 and newer, are distros with python-2.6 
> > unsupported now?
> > 
> > I did the same mistake in opkg-utils (patch for it is on yocto
> > ML). 
> > https://lists.yoctoproject.org/pipermail/yocto/2012-May/009182.html
> 
> Ah,
> > 
> thanks for catching that. What is the minimum python requirement?
> 
> I thought we had something in place to allow for older distributions
> to work with more modern python syntax.

We have the option of the external python tarball but we only resort to
that for really old systems. Our minimum python version is currently 2.6
and whilst there are some benefits from 2.7 (such as the process stuff),
we've not seen enough to warrant switching at this point. We have
workarounds in place for the process pieces.

Cheers,

Richard