Skip to contentSkip to conversation

python-libconfig: fix python-3.7 support

PATCH New Bernd Kuhls 2 participants 2 messages Aug 24, 2018

lore
Share & tools
lore

Bernd Kuhls

author
#0 lore raw

PATCH 1/1 python-libconfig: fix python-3.7 support

Fixes
http://autobuild.buildroot.net/results/32f/32f7e38895e7c491916bd69ed91bbe016e8b3ff6/

Signed-off-by Bernd Kuhls <[email protected]>
---
 ...termine-boost_python-name-depending-on-current-Pyt.patch | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)
package/python-libconfig/0001-Determine-boost_python-name-depending-on-current-Pyt.patch +4 -2
index 478bac76f6..ec7aec4072 100644
@@ -4,11 +4,13 @@ Date: Thu, 27 Jul 2017 09:46:07 +0200
 Subject: [PATCH] Determine boost_python name depending on current Python
  version
 
-Python2 requires boost_python27 and Python3 requires boost_python36
+Python2 requires boost_python27 and Python3 requires boost_python37
 (since boost 1.67).
 
 Signed-off-by: Yegor Yefremov <[email protected]>
 Signed-off-by: Fabrice Fontaine <[email protected]>
+[Bernd: adapted for python-3.7]
+Signed-off-by: Bernd Kuhls <[email protected]>
 ---
  setup.py | 6 +++++-
  1 file changed, 5 insertions(+), 1 deletion(-)

@@ -21,7 +23,7 @@ index fdf5c27..4b38569 100644
  from setuptools import setup, find_packages, Extension
  import sys, os
  
-+boostlib = "boost_python36"
++boostlib = "boost_python37"
 +if sys.version_info < (3, 0):
 +    boostlib = "boost_python27"
 +
Signature · 1 line
2.18.0
Applied
#1 lore raw

PATCH 1/1 python-libconfig: fix python-3.7 support

Hello,

Quoted 4 lines · Bernd Kuhls

Fixes
http://autobuild.buildroot.net/results/32f/32f7e38895e7c491916bd69ed91bbe016e8b3ff6/

Signed-off-by Bernd Kuhls <[email protected]>

Applied to next, thanks, however...

Quoted 4 lines

-+boostlib = "boost_python36"
++boostlib = "boost_python37"
+if sys.version_info < (3, 0):
+ boostlib = "boost_python27"

Can we do better than this piece of code that we will forget to update pretty much every time we bump Python ? Yegor ? Fabrice ? Any idea here ?

Thanks!

Thomas

Signature · 3 lines
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com