Patchwork [68/77] systemd: add target python to depends

login
register
mail settings
Submitter Martin Jansa
Date Dec. 5, 2012, 6:30 p.m.
Message ID <70ef34edf29d62cfc815aa5c70186026cb05c4cf.1354731625.git.Martin.Jansa@gmail.com>
Download mbox | patch
Permalink /patch/40487/
State Accepted
Commit d0d77c7ba4d8163f0125a9500e1752dea79fde7c
Headers show

Comments

Martin Jansa - Dec. 5, 2012, 6:30 p.m.
* pythonnative adds only python-native to DEPENDS, but we need target python for working python-config
checking for python-config... /OE/jansa-test/shr-core/tmp-eglibc/sysroots/x86_64-linux/usr/bin/python-native/python-config
Traceback (most recent call last):
  File "/OE/jansa-test/shr-core/tmp-eglibc/sysroots/x86_64-linux/usr/bin/python-native/python-config", line 24, in <module>
    pyver = sysconfig.get_config_var('VERSION')
  File "/OE/jansa-test/shr-core/tmp-eglibc/sysroots/x86_64-linux/usr/lib/python2.7/distutils/sysconfig.py", line 602, in get_config_var
    return get_config_vars().get(name)
  File "/OE/jansa-test/shr-core/tmp-eglibc/sysroots/x86_64-linux/usr/lib/python2.7/distutils/sysconfig.py", line 518, in get_config_vars
    func()
  File "/OE/jansa-test/shr-core/tmp-eglibc/sysroots/x86_64-linux/usr/lib/python2.7/distutils/sysconfig.py", line 416, in _init_posix
    raise DistutilsPlatformError(my_msg)
distutils.errors.DistutilsPlatformError: invalid Python installation: unable to open /OE/jansa-test/shr-core/tmp-eglibc/sysroots/om-gta02/usr/lib/python2.7/config/Makefile (No such file or directory)

Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
---
 meta-systemd/recipes-core/systemd/systemd_git.bb | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
Anders Darander - Dec. 6, 2012, 5:48 a.m.
* Martin Jansa <martin.jansa@gmail.com> [121205 19:32]:
> * pythonnative adds only python-native to DEPENDS, but we need target python for working python-config
> checking for python-config... /OE/jansa-test/shr-core/tmp-eglibc/sysroots/x86_64-linux/usr/bin/python-native/python-config
> Traceback (most recent call last):
>   File "/OE/jansa-test/shr-core/tmp-eglibc/sysroots/x86_64-linux/usr/bin/python-native/python-config", line 24, in <module>
>     pyver = sysconfig.get_config_var('VERSION')
>   File "/OE/jansa-test/shr-core/tmp-eglibc/sysroots/x86_64-linux/usr/lib/python2.7/distutils/sysconfig.py", line 602, in get_config_var
>     return get_config_vars().get(name)
>   File "/OE/jansa-test/shr-core/tmp-eglibc/sysroots/x86_64-linux/usr/lib/python2.7/distutils/sysconfig.py", line 518, in get_config_vars
>     func()
>   File "/OE/jansa-test/shr-core/tmp-eglibc/sysroots/x86_64-linux/usr/lib/python2.7/distutils/sysconfig.py", line 416, in _init_posix
>     raise DistutilsPlatformError(my_msg)
> distutils.errors.DistutilsPlatformError: invalid Python installation: unable to open /OE/jansa-test/shr-core/tmp-eglibc/sysroots/om-gta02/usr/lib/python2.7/config/Makefile (No such file or directory)

Which steps are introducing these errors?

I'm curious as systemd has been built without this native python
previously, and we're also adding --without-python to EXTRA_OECONF.
Thus, to me it seems fishy if we should need to add that depends.
Though, there might be a good reason, I guess. 

As usual I'm slightly worried about build time...

Cheers,
Anders
Martin Jansa - Dec. 6, 2012, 5:10 p.m.
Yes but there is patch to reenable python support in systemd in this
   patchset. The main motivation was to fix python-pyudev build, but it's
   not enough, so I'm open to remove both patches from patchset.
   Cheers

   -- Sent from my HP TouchPad
   __________________________________________________________________

   On 6.12.2012 6:48, Anders Darander <anders@chargestorm.se> wrote:
   * Martin Jansa <martin.jansa@gmail.com> [121205 19:32]:
   > * pythonnative adds only python-native to DEPENDS, but we need target
   python for working python-config
   > checking for python-config...
   /OE/jansa-test/shr-core/tmp-eglibc/sysroots/x86_64-linux/usr/bin/python
   -native/python-config
   > Traceback (most recent call last):
   > File
   "/OE/jansa-test/shr-core/tmp-eglibc/sysroots/x86_64-linux/usr/bin/pytho
   n-native/python-config", line 24, in <module>
   > pyver = sysconfig.get_config_var('VERSION')
   > File
   "/OE/jansa-test/shr-core/tmp-eglibc/sysroots/x86_64-linux/usr/lib/pytho
   n2.7/distutils/sysconfig.py", line 602, in get_config_var
   > return get_config_vars().get(name)
   > File
   "/OE/jansa-test/shr-core/tmp-eglibc/sysroots/x86_64-linux/usr/lib/pytho
   n2.7/distutils/sysconfig.py", line 518, in get_config_vars
   > func()
   > File
   "/OE/jansa-test/shr-core/tmp-eglibc/sysroots/x86_64-linux/usr/lib/pytho
   n2.7/distutils/sysconfig.py", line 416, in _init_posix
   > raise DistutilsPlatformError(my_msg)
   > distutils.errors.DistutilsPlatformError: invalid Python installation:
   unable to open
   /OE/jansa-test/shr-core/tmp-eglibc/sysroots/om-gta02/usr/lib/python2.7/
   config/Makefile (No such file or directory)
   Which steps are introducing these errors?
   I'm curious as systemd has been built without this native python
   previously, and we're also adding --without-python to EXTRA_OECONF.
   Thus, to me it seems fishy if we should need to add that depends.
   Though, there might be a good reason, I guess.
   As usual I'm slightly worried about build time...
   Cheers,
   Anders
   --
   Anders Darander
   ChargeStorm AB / eStorm AB
   _______________________________________________
   Openembedded-devel mailing list
   Openembedded-devel@lists.openembedded.org
   http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel

Patch

diff --git a/meta-systemd/recipes-core/systemd/systemd_git.bb b/meta-systemd/recipes-core/systemd/systemd_git.bb
index 341e85d..9005abd 100644
--- a/meta-systemd/recipes-core/systemd/systemd_git.bb
+++ b/meta-systemd/recipes-core/systemd/systemd_git.bb
@@ -8,7 +8,7 @@  LIC_FILES_CHKSUM = "file://LICENSE.GPL2;md5=751419260aa954499f7abaabaa882bbe \
 
 PROVIDES = "udev"
 
-DEPENDS = "xz kmod docbook-sgml-dtd-4.1-native intltool-native gperf-native acl readline dbus libcap libcgroup tcp-wrappers usbutils glib-2.0 libgcrypt"
+DEPENDS = "xz kmod docbook-sgml-dtd-4.1-native intltool-native gperf-native acl readline dbus libcap libcgroup tcp-wrappers usbutils glib-2.0 libgcrypt python"
 DEPENDS += "${@base_contains('DISTRO_FEATURES', 'pam', 'libpam', '', d)}"
 
 SERIAL_CONSOLE ?= "115200 /dev/ttyS0"
@@ -19,7 +19,7 @@  inherit gitpkgv
 PKGV = "v${GITPKGVTAG}"
 
 PV = "git"
-PR = "r11"
+PR = "r12"
 
 # need to export these variables for python-config to work
 export BUILD_SYS