Re: [OE-core] [PATCH 13/15] python-pygobject: fix buildpath QA issue

2016-03-08 Thread Hongxu Jia
On 03/08/2016 11:24 PM, Burton, Ross wrote: On 26 February 2016 at 01:45, Hongxu Jia > wrote: +do_install_append_class-target() { + sed -i -e "s:${PYTHON}:\`which python\`:g" \ + ${D}${bindir}/pygobject-codegen-2.0 +} If you're

Re: [OE-core] [PATCH 13/15] python-pygobject: fix buildpath QA issue

2016-03-08 Thread Alexander Kanavin
On 03/08/2016 05:24 PM, Burton, Ross wrote: +do_install_append_class-target() { + sed -i -e "s:${PYTHON}:\`which python\`:g" \ + ${D}${bindir}/pygobject-codegen-2.0 +} If you're going to expect the shell to find the right python on the target there's no need to

Re: [OE-core] [PATCH 13/15] python-pygobject: fix buildpath QA issue

2016-03-08 Thread Burton, Ross
On 26 February 2016 at 01:45, Hongxu Jia wrote: > +do_install_append_class-target() { > + sed -i -e "s:${PYTHON}:\`which python\`:g" \ > + ${D}${bindir}/pygobject-codegen-2.0 > +} > If you're going to expect the shell to find the right python on the target there's no need to use

[OE-core] [PATCH 13/15] python-pygobject: fix buildpath QA issue

2016-02-25 Thread Hongxu Jia
The variable PYTHON is assigned with native python. But for target, it caused pygobject-codegen has build path. vim ${S}/codegen/pygobject-codegen-2.0.in [snip]... exec @PYTHON@ $codegendir/codegen.py "$@" [snip]... So we replace it by invoking "which python". [YOCTO #7058] Signed-off-by: Hongxu