All of lore.kernel.org
 help / color / mirror / Atom feed
* [meta-python][PATCH v2] python-pyparted: update to 3.10.5
@ 2015-08-27 15:05 Tim Orling
  2015-08-27 16:28 ` Martin Jansa
  0 siblings, 1 reply; 2+ messages in thread
From: Tim Orling @ 2015-08-27 15:05 UTC (permalink / raw)
  To: openembedded-devel

Changes since v1:
  * use git tag rather than discouraged github archives
  * add patch to fix float cast error when version is x.y.z form
  * add minimal parted version (>= 2.3) to RDEPENDS

NOTE:
  If your tree defaults to an earlier parted version (e.g. 1.8.6),
  you will need to add PREFERRED_VERSION_parted = "3.%" to local.conf
  or you will see a do_compile error (complaining about required
  libparted version).
---
 .../0001-fix-version-float-check.patch             | 26 +++++++++++++++++++++
 .../python-pyparted/python-pyparted_3.10.5.bb      | 27 ++++++++++++++++++++++
 .../python-pyparted/python-pyparted_3.9.bb         | 23 ------------------
 3 files changed, 53 insertions(+), 23 deletions(-)
 create mode 100644 meta-python/recipes-extended/python-pyparted/python-pyparted/0001-fix-version-float-check.patch
 create mode 100644 meta-python/recipes-extended/python-pyparted/python-pyparted_3.10.5.bb
 delete mode 100644 meta-python/recipes-extended/python-pyparted/python-pyparted_3.9.bb

diff --git a/meta-python/recipes-extended/python-pyparted/python-pyparted/0001-fix-version-float-check.patch b/meta-python/recipes-extended/python-pyparted/python-pyparted/0001-fix-version-float-check.patch
new file mode 100644
index 0000000..db3ab9f
--- /dev/null
+++ b/meta-python/recipes-extended/python-pyparted/python-pyparted/0001-fix-version-float-check.patch
@@ -0,0 +1,26 @@
+check_mod_version breaks if module version is of x.y.z form
+use a regex to help cast x.y.z version to a float
+
+Upstream-Status: Pending
+
+Index: git/setup.py
+===================================================================
+--- git.orig/setup.py
++++ git/setup.py
+@@ -25,6 +25,7 @@ import glob
+ import os
+ import platform
+ import sys
++import re
+ from distutils.ccompiler import new_compiler
+ from distutils.errors import CompileError
+ from distutils.errors import LinkError
+@@ -51,6 +52,8 @@ def pkgconfig(*packages, **kwargs):
+ 
+ def check_mod_version(module, version):
+     modversion = subprocess.check_output(["pkg-config", "--modversion", module])
++    match = re.search('([0-9]+\.[0-9]+)', modversion)
++    modversion = match.group(0)
+     if not float(modversion) >= float(version):
+         sys.stderr.write("*** Minimum required %s version: %s, found: %s\n" % (module, version, modversion,))
+         sys.exit(1)
diff --git a/meta-python/recipes-extended/python-pyparted/python-pyparted_3.10.5.bb b/meta-python/recipes-extended/python-pyparted/python-pyparted_3.10.5.bb
new file mode 100644
index 0000000..d5b8f9a
--- /dev/null
+++ b/meta-python/recipes-extended/python-pyparted/python-pyparted_3.10.5.bb
@@ -0,0 +1,27 @@
+DESCRIPTION = "pyparted is a set of Python modules that provide Python programmers \
+an interface to libparted, the GNU parted library for disk partitioning and \
+filesystem manipulation."
+SUMMARY = "Python bindings for libparted"
+HOMEPAGE = "https://github.com/rhinstaller/pyparted"
+LICENSE = "GPL-2.0+"
+LIC_FILES_CHKSUM = "\
+    file://COPYING;md5=8ca43cbc842c2336e835926c2166c28b \
+    file://src/_pedmodule.c;startline=10;endline=22;md5=ebcb25dde1ac9c46e986dec79b404e7e \
+"
+DEPENDS += "parted python-re"
+
+PV = "3.10.5+git${SRCPV}"
+
+# upstream only publishes releases in github archives which are discouraged
+SRCREV = "75bba5a5c8b65b21979d69421f661f7708304191"
+SRC_URI = "git://github.com/rhinstaller/pyparted.git;protocol=https \
+           file://0001-fix-version-float-check.patch \
+"
+S = "${WORKDIR}/git"
+
+inherit distutils
+
+RDEPENDS_${PN} += "python-stringold python-codecs python-math parted (>= 2.3)"
+RDEPENDS_${PN}_class-native = ""
+
+BBCLASSEXTEND += "native"
diff --git a/meta-python/recipes-extended/python-pyparted/python-pyparted_3.9.bb b/meta-python/recipes-extended/python-pyparted/python-pyparted_3.9.bb
deleted file mode 100644
index bfb4f6f..0000000
--- a/meta-python/recipes-extended/python-pyparted/python-pyparted_3.9.bb
+++ /dev/null
@@ -1,23 +0,0 @@
-DESCRIPTION = "pyparted is a set of Python modules that provide Python programmers \
-an interface to libparted, the GNU parted library for disk partitioning and \
-filesystem manipulation."
-SUMMARY = "Python bindings for libparted"
-HOMEPAGE = "https://fedorahosted.org/pyparted/"
-LICENSE = "GPL-2.0+"
-LIC_FILES_CHKSUM = "\
-    file://COPYING;md5=8ca43cbc842c2336e835926c2166c28b \
-    file://src/_pedmodule.c;startline=10;endline=22;md5=70c62bd73782a03f56a0571a9f08ea46 \
-"
-DEPENDS += "parted"
-
-SRC_URI = "https://fedorahosted.org/releases/p/y/pyparted/pyparted-${PV}.tar.gz"
-SRC_URI[md5sum] = "f16c7ef7f5fa4a43fcb2a4654b487e39"
-SRC_URI[sha256sum] = "a56712e3d058ce3d859c158236dbbf45224018919efd3d880ea80f9e0d0bebbb"
-S = "${WORKDIR}/pyparted-${PV}"
-
-inherit distutils
-
-RDEPENDS_${PN} += "python-stringold python-codecs python-math"
-RDEPENDS_${PN}_class-native = ""
-
-BBCLASSEXTEND += "native"
-- 
2.1.0



^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [meta-python][PATCH v2] python-pyparted: update to 3.10.5
  2015-08-27 15:05 [meta-python][PATCH v2] python-pyparted: update to 3.10.5 Tim Orling
@ 2015-08-27 16:28 ` Martin Jansa
  0 siblings, 0 replies; 2+ messages in thread
From: Martin Jansa @ 2015-08-27 16:28 UTC (permalink / raw)
  To: openembedded-devel

[-- Attachment #1: Type: text/plain, Size: 5665 bytes --]

On Thu, Aug 27, 2015 at 08:05:31AM -0700, Tim Orling wrote:
> Changes since v1:
>   * use git tag rather than discouraged github archives
>   * add patch to fix float cast error when version is x.y.z form
>   * add minimal parted version (>= 2.3) to RDEPENDS

Please put these review changes under --- separator, so that they aren't
part of final commit message.

> 
> NOTE:
>   If your tree defaults to an earlier parted version (e.g. 1.8.6),
>   you will need to add PREFERRED_VERSION_parted = "3.%" to local.conf
>   or you will see a do_compile error (complaining about required
>   libparted version).
> ---
>  .../0001-fix-version-float-check.patch             | 26 +++++++++++++++++++++
>  .../python-pyparted/python-pyparted_3.10.5.bb      | 27 ++++++++++++++++++++++
>  .../python-pyparted/python-pyparted_3.9.bb         | 23 ------------------
>  3 files changed, 53 insertions(+), 23 deletions(-)
>  create mode 100644 meta-python/recipes-extended/python-pyparted/python-pyparted/0001-fix-version-float-check.patch
>  create mode 100644 meta-python/recipes-extended/python-pyparted/python-pyparted_3.10.5.bb
>  delete mode 100644 meta-python/recipes-extended/python-pyparted/python-pyparted_3.9.bb
> 
> diff --git a/meta-python/recipes-extended/python-pyparted/python-pyparted/0001-fix-version-float-check.patch b/meta-python/recipes-extended/python-pyparted/python-pyparted/0001-fix-version-float-check.patch
> new file mode 100644
> index 0000000..db3ab9f
> --- /dev/null
> +++ b/meta-python/recipes-extended/python-pyparted/python-pyparted/0001-fix-version-float-check.patch
> @@ -0,0 +1,26 @@
> +check_mod_version breaks if module version is of x.y.z form
> +use a regex to help cast x.y.z version to a float
> +
> +Upstream-Status: Pending
> +
> +Index: git/setup.py
> +===================================================================
> +--- git.orig/setup.py
> ++++ git/setup.py
> +@@ -25,6 +25,7 @@ import glob
> + import os
> + import platform
> + import sys
> ++import re
> + from distutils.ccompiler import new_compiler
> + from distutils.errors import CompileError
> + from distutils.errors import LinkError
> +@@ -51,6 +52,8 @@ def pkgconfig(*packages, **kwargs):
> + 
> + def check_mod_version(module, version):
> +     modversion = subprocess.check_output(["pkg-config", "--modversion", module])
> ++    match = re.search('([0-9]+\.[0-9]+)', modversion)
> ++    modversion = match.group(0)
> +     if not float(modversion) >= float(version):
> +         sys.stderr.write("*** Minimum required %s version: %s, found: %s\n" % (module, version, modversion,))
> +         sys.exit(1)
> diff --git a/meta-python/recipes-extended/python-pyparted/python-pyparted_3.10.5.bb b/meta-python/recipes-extended/python-pyparted/python-pyparted_3.10.5.bb
> new file mode 100644
> index 0000000..d5b8f9a
> --- /dev/null
> +++ b/meta-python/recipes-extended/python-pyparted/python-pyparted_3.10.5.bb
> @@ -0,0 +1,27 @@
> +DESCRIPTION = "pyparted is a set of Python modules that provide Python programmers \
> +an interface to libparted, the GNU parted library for disk partitioning and \
> +filesystem manipulation."
> +SUMMARY = "Python bindings for libparted"
> +HOMEPAGE = "https://github.com/rhinstaller/pyparted"
> +LICENSE = "GPL-2.0+"
> +LIC_FILES_CHKSUM = "\
> +    file://COPYING;md5=8ca43cbc842c2336e835926c2166c28b \
> +    file://src/_pedmodule.c;startline=10;endline=22;md5=ebcb25dde1ac9c46e986dec79b404e7e \
> +"
> +DEPENDS += "parted python-re"
> +
> +PV = "3.10.5+git${SRCPV}"
> +
> +# upstream only publishes releases in github archives which are discouraged
> +SRCREV = "75bba5a5c8b65b21979d69421f661f7708304191"
> +SRC_URI = "git://github.com/rhinstaller/pyparted.git;protocol=https \
> +           file://0001-fix-version-float-check.patch \
> +"
> +S = "${WORKDIR}/git"
> +
> +inherit distutils
> +
> +RDEPENDS_${PN} += "python-stringold python-codecs python-math parted (>= 2.3)"
> +RDEPENDS_${PN}_class-native = ""
> +
> +BBCLASSEXTEND += "native"
> diff --git a/meta-python/recipes-extended/python-pyparted/python-pyparted_3.9.bb b/meta-python/recipes-extended/python-pyparted/python-pyparted_3.9.bb
> deleted file mode 100644
> index bfb4f6f..0000000
> --- a/meta-python/recipes-extended/python-pyparted/python-pyparted_3.9.bb
> +++ /dev/null
> @@ -1,23 +0,0 @@
> -DESCRIPTION = "pyparted is a set of Python modules that provide Python programmers \
> -an interface to libparted, the GNU parted library for disk partitioning and \
> -filesystem manipulation."
> -SUMMARY = "Python bindings for libparted"
> -HOMEPAGE = "https://fedorahosted.org/pyparted/"
> -LICENSE = "GPL-2.0+"
> -LIC_FILES_CHKSUM = "\
> -    file://COPYING;md5=8ca43cbc842c2336e835926c2166c28b \
> -    file://src/_pedmodule.c;startline=10;endline=22;md5=70c62bd73782a03f56a0571a9f08ea46 \
> -"
> -DEPENDS += "parted"
> -
> -SRC_URI = "https://fedorahosted.org/releases/p/y/pyparted/pyparted-${PV}.tar.gz"
> -SRC_URI[md5sum] = "f16c7ef7f5fa4a43fcb2a4654b487e39"
> -SRC_URI[sha256sum] = "a56712e3d058ce3d859c158236dbbf45224018919efd3d880ea80f9e0d0bebbb"
> -S = "${WORKDIR}/pyparted-${PV}"
> -
> -inherit distutils
> -
> -RDEPENDS_${PN} += "python-stringold python-codecs python-math"
> -RDEPENDS_${PN}_class-native = ""
> -
> -BBCLASSEXTEND += "native"
> -- 
> 2.1.0
> 
> -- 
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-devel

-- 
Martin 'JaMa' Jansa     jabber: Martin.Jansa@gmail.com

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 188 bytes --]

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2015-08-27 16:28 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-08-27 15:05 [meta-python][PATCH v2] python-pyparted: update to 3.10.5 Tim Orling
2015-08-27 16:28 ` Martin Jansa

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.