Skip to contentSkip to conversation

Tornado 4.3 is missing python2 RDEPENDS

PATCH Superseded S. Lockwood-Childs 3 participants 10 messages Jun 2, 2016 – Jun 3, 2016

v3 of 4 Superseded Latest is v4 →

lore
Share & tools
lore
#0 lore raw

v3 cover Tornado 4.3 is missing python2 RDEPENDS

Tornado 4.3 needs a couple more backport modules to prevent tornado/gen.py from bombing on import errors when running on python2 rather than python3.4

ImportError: No module named singledispatch

ImportError: No module named backports_abc

S. Lockwood-Childs (3):
python-backports-abc: initial recipe for version 0.4
python-singledispatch: initial recipe for version 3.4.0.3
python-tornado: add missing python2 runtime deps

 .../python/python-backports-abc_0.4.bb                 | 18 ++++++++++++++++++
 .../python/python-singledispatch_3.4.0.3.bb            | 13 +++++++++++++
 .../recipes-devtools/python/python-tornado_4.3.bb      |  2 +-
 3 files changed, 32 insertions(+), 1 deletion(-)
 create mode 100644 meta-python/recipes-devtools/python/python-backports-abc_0.4.bb
 create mode 100644 meta-python/recipes-devtools/python/python-singledispatch_3.4.0.3.bb
Signature · 1 line
1.9.1
#1 lore raw

PATCH v3 1/3 python-backports-abc: initial recipe for version 0.4 RDEPENDS

Tornado 4.3 needs a couple more backport modules to prevent tornado/gen.py from bombing on import errors when running on python2 rather than python3.4

ImportError: No module named backports_abc

---
 .../python/python-backports-abc_0.4.bb                 | 18 ++++++++++++++++++
 1 file changed, 18 insertions(+)
 create mode 100644 meta-python/recipes-devtools/python/python-backports-abc_0.4.bb
meta-python/recipes-devtools/python/python-backports-abc_0.4.bb new file +18 -0
new file mode 100644
index 0000000..6a4d267
@@ -0,0 +1,18 @@
+SUMMARY = "collections.abc from Python 3.4"
+DESCRIPTION = "A backport of recent additions to the 'collections.abc' module"
+
+LICENSE = "PSF"
+LIC_FILES_CHKSUM = "file://${WORKDIR}/LICENSE;md5=dd98d01d471fac8d8dbdd975229dba03"
+
+# pypi package doesn't have valid entry for license
+SRC_URI += "https://raw.githubusercontent.com/cython/backports_abc/master/LICENSE;name=license"
+
+SRC_URI[md5sum] = "0b65a216ce9dc9c1a7e20a729dd7c05b"
+SRC_URI[sha256sum] = "8b3e4092ba3d541c7a2f9b7d0d9c0275b21c6a01c53a61c731eba6686939d0a5"
+SRC_URI[license.md5sum] = "dd98d01d471fac8d8dbdd975229dba03"
+SRC_URI[license.sha256sum] = "0a4f3b38055f50f047a42521568fa6ddb9a5976c2884f6ae138796d0f71150ca"
+
+
+
+PYPI_PACKAGE = "backports_abc"
+inherit pypi setuptools
Signature · 1 line
1.9.1
#2 lore raw

Re: [meta-python][PATCHv3 1/3] python-backports-abc: initial recipe for version 0.4 RDEPENDS

Quoted 6 lines · S. Lockwood-Childs

Tornado 4.3 needs a couple more backport modules to prevent tornado/gen.py from bombing on import errors when running on python2 rather than python3.4

ImportError: No module named backports_abc

In my mind, this patch (and its sibling) don't really have anything to to with Tornado, they are just new packages being added to make that package happy. I'd prefer to see a commit message that reflects what this package is as well as why it is being added.

Quoted 30 lines
---
  .../python/python-backports-abc_0.4.bb                 | 18 ++++++++++++++++++
  1 file changed, 18 insertions(+)

create mode 100644 meta-python/recipes-devtools/python/python-backports-abc_0.4.bb

meta-python/recipes-devtools/python/python-backports-abc_0.4.bb new file +18 -0
new file mode 100644
index 0000000..6a4d267
@@ -0,0 +1,18 @@
+SUMMARY = "collections.abc from Python 3.4"
+DESCRIPTION = "A backport of recent additions to the 'collections.abc' module"
+
+LICENSE = "PSF"
+LIC_FILES_CHKSUM = "file://${WORKDIR}/LICENSE;md5=dd98d01d471fac8d8dbdd975229dba03"
+
+# pypi package doesn't have valid entry for license
+SRC_URI += "https://raw.githubusercontent.com/cython/backports_abc/master/LICENSE;name=license"
+
+SRC_URI[md5sum] = "0b65a216ce9dc9c1a7e20a729dd7c05b"
+SRC_URI[sha256sum] = "8b3e4092ba3d541c7a2f9b7d0d9c0275b21c6a01c53a61c731eba6686939d0a5"
+SRC_URI[license.md5sum] = "dd98d01d471fac8d8dbdd975229dba03"
+SRC_URI[license.sha256sum] = "0a4f3b38055f50f047a42521568fa6ddb9a5976c2884f6ae138796d0f71150ca"
+
+
+
+PYPI_PACKAGE = "backports_abc"
+inherit pypi setuptools
Signature · 4 lines
------------------------------------------------------------
Gary Thomas                 |  Consulting for the
MLB Associates              |    Embedded world
------------------------------------------------------------
#3 lore raw

Re: [meta-python][PATCHv3 1/3] python-backports-abc: initial recipe for version 0.4 RDEPENDS

Quoted 8 lines · Gary Thomas
Quoted 6 lines · S. Lockwood-Childs

Tornado 4.3 needs a couple more backport modules to prevent tornado/gen.py from bombing on import errors when running on python2 rather than python3.4

ImportError: No module named backports_abc

Quoted 7 lines

In my mind, this patch (and its sibling) don't really have anything to to with Tornado, they are just new packages being added to make that package happy. I'd prefer to see a commit message that reflects what this package is as well as why it is being added.

meta-python/recipes-devtools/python/python-backports-abc_0.4.bb line 2

Quoted 2 lines
meta-python/recipes-devtools/python/python-backports-abc_0.4.bb new file +2 -0
+SUMMARY = "collections.abc from Python 3.4"
+DESCRIPTION = "A backport of recent additions to the 'collections.abc' module"

What would you like to see, a repeat of the SUMMARY or DESCRIPTION lines added to the corresponding commit messages (if so, which)?

Other than that, I don't know what else to say about them; if their absence wasn't breaking Tornado, I would not have picked either of these particular modules to submit. Neither one has much use yet outside of Tornado (found no other users of abc, while singledispatch is also used by rpy (python interface to R).

#4 lore raw

Re: [meta-python][PATCHv3 1/3] python-backports-abc: initial recipe for version 0.4 RDEPENDS

Quoted 20 lines · S. Lockwood-Childs
Quoted 8 lines · Gary Thomas
Quoted 6 lines · S. Lockwood-Childs

Tornado 4.3 needs a couple more backport modules to prevent tornado/gen.py from bombing on import errors when running on python2 rather than python3.4

ImportError: No module named backports_abc

Quoted 7 lines

In my mind, this patch (and its sibling) don't really have anything to to with Tornado, they are just new packages being added to make that package happy. I'd prefer to see a commit message that reflects what this package is as well as why it is being added.

meta-python/recipes-devtools/python/python-backports-abc_0.4.bb line 2

Quoted 2 lines
meta-python/recipes-devtools/python/python-backports-abc_0.4.bb new file +2 -0
+SUMMARY = "collections.abc from Python 3.4"
+DESCRIPTION = "A backport of recent additions to the 'collections.abc' module"

What would you like to see, a repeat of the SUMMARY or DESCRIPTION lines added to the corresponding commit messages (if so, which)?

Quoted 6 lines

Other than that, I don't know what else to say about them; if their absence wasn't breaking Tornado, I would not have picked either of these particular modules to submit. Neither one has much use yet outside of Tornado (found no other users of abc, while singledispatch is also used by rpy (python interface to R).

I would think the SUMMARY would be OK, along with a note that the package was introduced to make Tornado happy. (I know this is just "eye candy", but it does help when searching through git logs...)

Thanks for doing this - I was about to make similar changes myself.

Signature · 4 lines
------------------------------------------------------------
Gary Thomas                 |  Consulting for the
MLB Associates              |    Embedded world
------------------------------------------------------------
#5 lore raw

Re: [meta-python][PATCHv3 1/3] python-backports-abc: initial recipe for version 0.4 RDEPENDS

Quoted 28 lines · Gary Thomas
Quoted 20 lines · S. Lockwood-Childs
Quoted 8 lines · Gary Thomas
Quoted 6 lines · S. Lockwood-Childs

Tornado 4.3 needs a couple more backport modules to prevent tornado/gen.py from bombing on import errors when running on python2 rather than python3.4

ImportError: No module named backports_abc

Quoted 7 lines

In my mind, this patch (and its sibling) don't really have anything to to with Tornado, they are just new packages being added to make that package happy. I'd prefer to see a commit message that reflects what this package is as well as why it is being added.

meta-python/recipes-devtools/python/python-backports-abc_0.4.bb line 2

Quoted 2 lines
meta-python/recipes-devtools/python/python-backports-abc_0.4.bb new file +2 -0
+SUMMARY = "collections.abc from Python 3.4"
+DESCRIPTION = "A backport of recent additions to the 'collections.abc' module"

What would you like to see, a repeat of the SUMMARY or DESCRIPTION lines added to the corresponding commit messages (if so, which)?

Quoted 6 lines

Other than that, I don't know what else to say about them; if their absence wasn't breaking Tornado, I would not have picked either of these particular modules to submit. Neither one has much use yet outside of Tornado (found no other users of abc, while singledispatch is also used by rpy (python interface to R).

Quoted 5 lines

I would think the SUMMARY would be OK, along with a note that the package was introduced to make Tornado happy. (I know this is just "eye candy", but it does help when searching through git logs...)

Thanks for doing this - I was about to make similar changes myself.

I am all for having useful git logs.

And at least it gives me a reason to resubmit the patch series without the typo that came back (grrr) though Martin was nice enough not to make me.

#6 lore raw

PATCH v3 2/3 python-singledispatch: initial recipe for version 3.4.0.3

In reply to S. Lockwood-Childs

Tornado 4.3 needs a couple more backport modules to prevent tornado/gen.py from bombing on import errors when running on python2 rather than python3.4

ImportError: No module named singledispatch

---
 .../python/python-singledispatch_3.4.0.3.bb                 | 13 +++++++++++++
 1 file changed, 13 insertions(+)
 create mode 100644 meta-python/recipes-devtools/python/python-singledispatch_3.4.0.3.bb
meta-python/recipes-devtools/python/python-singledispatch_3.4.0.3.bb new file +13 -0
new file mode 100644
index 0000000..87f46e5
@@ -0,0 +1,13 @@
+SUMMARY = "functools.singledispatch from Python 3.4"
+DESCRIPTION = "PEP 443 proposed to expose a mechanism in the functools standard library module \
+in Python 3.4 that provides a simple form of generic programming known as single-dispatch \
+generic functions.  This library is a backport of this functionality to Python 2.6 - 3.3"
+
+LICENSE = "Apache-2.0"
+LIC_FILES_CHKSUM = "file://README.rst;md5=ee3cd67264adc7eb07981f3644dc17dc"
+
+SRC_URI[md5sum] = "af2fc6a3d6cc5a02d0bf54d909785fcb"
+SRC_URI[sha256sum] = "5b06af87df13818d14f08a028e42f566640aef80805c3b50c5056b086e3c2b9c"
+
+PYPI_PACKAGE = "singledispatch"
+inherit pypi setuptools
Signature · 1 line
1.9.1
#7 lore raw

PATCH v3 3/3 python-tornado: add missing python2 runtime deps

In reply to S. Lockwood-Childs

Tornado 4.3 needs backports_abc and singledispatch backport modules at runtime to prevent tornado/gen.py from bombing on import errors when running on python2 rather than python3.4

---
 meta-python/recipes-devtools/python/python-tornado_4.3.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
meta-python/recipes-devtools/python/python-tornado_4.3.bb +1 -1
index 55a48b8..d30f88f 100644
@@ -1,3 +1,3 @@
 inherit pypi setuptools
 require python-tornado.inc
-RDEPENDS_${PN} += "${PYTHON_PN}-backports-ssl"
+RDEPENDS_${PN} += "${PYTHON_PN}-backports-ssl ${PYTHON_P}-backports-abc ${PYTHON_PN}-singledispatch"
Signature · 1 line
1.9.1
#8 lore raw

Re: [meta-python][PATCHv3 3/3] python-tornado: add missing python2 runtime deps

Quoted 1 line · S. Lockwood-Childs

+RDEPENDS_${PN} += "${PYTHON_PN}-backports-ssl ${PYTHON_P}-backports-abc ${PYTHON_PN}-singledispatch"

Well that's really fun -- I managed to pick the version of the patch with the typo when splitting it up. Do I have to send the whole series again to add one letter (PYTHON_P -> PYTHON_PN) ?

#9 lore raw

Re: [meta-python][PATCHv3 3/3] python-tornado: add missing python2 runtime deps

Quoted 6 lines · S. Lockwood-Childs
Quoted 1 line · S. Lockwood-Childs

+RDEPENDS_${PN} += "${PYTHON_PN}-backports-ssl ${PYTHON_P}-backports-abc ${PYTHON_PN}-singledispatch"

Well that's really fun -- I managed to pick the version of the patch with the typo when splitting it up. Do I have to send the whole series again to add one letter (PYTHON_P -> PYTHON_PN) ?

I'll try not to forget to change it when cherry-picking to master-next.

Signature · 1 line
Martin 'JaMa' Jansa     jabber: [email protected]