Re: [PATCH] python-3.5: Include _posixsubprocess in package

Re: [PATCH] python-3.5: Include _posixsubprocess in package

[Alexander Kanavin]

On 05/15/2018 12:45 PM, Joe Rutledge wrote:
> Change the python-3.5-manifest.inc file to also include:
> lib-dynload/_posixsubprocess.*.so
> 
> This is needed for Python subprocess support but was not included.

You need to include [krogoth] in the subject to indicate which release 
this is for.

> -    "lib-dynload/xreadlines.*.so types.* platform.* ${bindir}/python* "  +
> +    "lib-dynload/xreadlines.*.so types.* platform.* ${bindir}/python* "  +

What has changed here?

Alex

[PATCH] python-3.5: Include _posixsubprocess in package

[Joe Rutledge]

Change the python-3.5-manifest.inc file to also include:
lib-dynload/_posixsubprocess.*.so

This is needed for Python subprocess support but was not included.

Signed-off-by: Joe Rutledge <joe.rutledge@phabrix.com>
---
 meta/recipes-devtools/python/python-3.5-manifest.inc | 2 +-
 scripts/contrib/python/generate-manifest-3.5.py      | 6 +++---
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/meta/recipes-devtools/python/python-3.5-manifest.inc b/meta/recipes-devtools/python/python-3.5-manifest.inc
index 0ad9186..7b642c3 100644
--- a/meta/recipes-devtools/python/python-3.5-manifest.inc
+++ b/meta/recipes-devtools/python/python-3.5-manifest.inc
@@ -231,7 +231,7 @@ FILES_${PN}-stringold="${libdir}/python3.5/lib-dynload/strop.*.so ${libdir}/pyth
 
 SUMMARY_${PN}-subprocess="Python subprocess support"
 RDEPENDS_${PN}-subprocess="${PN}-core ${PN}-io ${PN}-re ${PN}-fcntl ${PN}-pickle ${PN}-signal ${PN}-selectors"
-FILES_${PN}-subprocess="${libdir}/python3.5/subprocess.* "
+FILES_${PN}-subprocess="${libdir}/python3.5/subprocess.* ${libdir}/python3.5/lib-dynload/_posixsubprocess.*.so "
 
 SUMMARY_${PN}-syslog="Python syslog interface"
 RDEPENDS_${PN}-syslog="${PN}-core"
diff --git a/scripts/contrib/python/generate-manifest-3.5.py b/scripts/contrib/python/generate-manifest-3.5.py
index eac493a..0db8540 100755
--- a/scripts/contrib/python/generate-manifest-3.5.py
+++ b/scripts/contrib/python/generate-manifest-3.5.py
@@ -17,7 +17,7 @@ import os
 import sys
 import time
 
-VERSION = "3.5.0"
+VERSION = "3.5.1"
 
 __author__ = "Michael 'Mickey' Lauer <mlauer@vanille-media.de>"
 __version__ = "20140131"
@@ -173,7 +173,7 @@ if __name__ == "__main__":
     "warnings.* site.* stat.* " +
     "UserDict.* UserList.* UserString.* " +
     "lib-dynload/binascii.*.so lib-dynload/_struct.*.so lib-dynload/time.*.so " +
-    "lib-dynload/xreadlines.*.so types.* platform.* ${bindir}/python* "  + 
+    "lib-dynload/xreadlines.*.so types.* platform.* ${bindir}/python* "  +
     "_weakrefset.* sysconfig.* _sysconfigdata.* config/Makefile " +
     "${includedir}/python${PYTHON_BINABI}/pyconfig*.h " +
     "${libdir}/python${PYTHON_MAJMIN}/collections " +
@@ -349,7 +349,7 @@ if __name__ == "__main__":
     "cmd.* commands.* dircache.* fnmatch.* glob.* popen2.* shlex.* shutil.*" )
 
     m.addPackage( "${PN}-subprocess", "Python subprocess support", "${PN}-core ${PN}-io ${PN}-re ${PN}-fcntl ${PN}-pickle ${PN}-signal ${PN}-selectors",
-    "subprocess.*" )
+    "subprocess.* lib-dynload/_posixsubprocess.*.so" )
 
     m.addPackage( "${PN}-signal", "Python set handlers for asynchronous events support", "${PN}-core",
     "signal.*" )
-- 
2.7.4

Re: [PATCH] python-3.5: Include _posixsubprocess in package

[Peter Kjellerstedt]

> -----Original Message-----
> From: openembedded-core-bounces@lists.openembedded.org
> [mailto:openembedded-core-bounces@lists.openembedded.org] On Behalf Of
> Alexander Kanavin
> Sent: den 15 maj 2018 11:40
> To: Joe Rutledge <joe.rutledge@phabrix.com>; openembedded-
> core@lists.openembedded.org
> Subject: Re: [OE-core] [PATCH] python-3.5: Include _posixsubprocess in
> package
> 
> On 05/15/2018 12:45 PM, Joe Rutledge wrote:
> > Change the python-3.5-manifest.inc file to also include:
> > lib-dynload/_posixsubprocess.*.so
> >
> > This is needed for Python subprocess support but was not included.
> 
> You need to include [krogoth] in the subject to indicate which release
> this is for.
> 
> > -    "lib-dynload/xreadlines.*.so types.* platform.* ${bindir}/python* "  +
> > +    "lib-dynload/xreadlines.*.so types.* platform.* ${bindir}/python* "  +
> 
> What has changed here?

A trailing space has been removed (not seen above, but it can be seen 
in the original mail).

> Alex

//Peter