* [PATCH] python2/3: Remove building host path from packages
@ 2017-04-11 8:03 Li Zhou
2017-04-11 15:44 ` Richard Purdie
0 siblings, 1 reply; 2+ messages in thread
From: Li Zhou @ 2017-04-11 8:03 UTC (permalink / raw)
To: openembedded-core
Remove building host related path from python2 and python3 packages.
Signed-off-by: Li Zhou <li.zhou@windriver.com>
---
meta/recipes-devtools/python/python3_3.5.2.bb | 8 +++++++-
meta/recipes-devtools/python/python_2.7.13.bb | 6 ++++++
2 files changed, 13 insertions(+), 1 deletion(-)
diff --git a/meta/recipes-devtools/python/python3_3.5.2.bb b/meta/recipes-devtools/python/python3_3.5.2.bb
index 2ff7c9e..ca8dfe3 100644
--- a/meta/recipes-devtools/python/python3_3.5.2.bb
+++ b/meta/recipes-devtools/python/python3_3.5.2.bb
@@ -185,9 +185,15 @@ py_package_preprocess () {
install -m 0644 ${B}/Makefile.orig ${PKGD}/${libdir}/python${PYTHON_MAJMIN}/config-${PYTHON_MAJMIN}${PYTHON_ABI}/Makefile
# Remove references to buildmachine paths in target Makefile and _sysconfigdata
sed -i -e 's:--sysroot=${STAGING_DIR_TARGET}::g' -e s:'--with-libtool-sysroot=${STAGING_DIR_TARGET}'::g \
+ -e 's:${STAGING_DIR}/${MACHINE}${datadir}/pkgconfig:${datadir}/pkgconfig:g' \
+ -e 's:${STAGING_DIR_NATIVE}::g' \
+ -e 's:${STAGING_DIR_HOST}::g' \
+ -e 's:${WORKDIR}::g' \
+ -e 's:${TMPDIR}::g' \
${PKGD}/${libdir}/python${PYTHON_MAJMIN}/config/Makefile \
${PKGD}/${libdir}/python${PYTHON_MAJMIN}/config-${PYTHON_MAJMIN}${PYTHON_ABI}/Makefile \
- ${PKGD}/${libdir}/python${PYTHON_MAJMIN}/_sysconfigdata.py
+ ${PKGD}/${libdir}/python${PYTHON_MAJMIN}/_sysconfigdata.py \
+ ${PKGD}/${bindir}/python${PYTHON_MAJMIN}${PYTHON_ABI}-config
}
require python-${PYTHON_MAJMIN}-manifest.inc
diff --git a/meta/recipes-devtools/python/python_2.7.13.bb b/meta/recipes-devtools/python/python_2.7.13.bb
index 4ef9952..b4bde5f 100644
--- a/meta/recipes-devtools/python/python_2.7.13.bb
+++ b/meta/recipes-devtools/python/python_2.7.13.bb
@@ -131,7 +131,13 @@ py_package_preprocess () {
# Remove references to buildmachine paths in target Makefile and _sysconfigdata
sed -i -e 's:--sysroot=${STAGING_DIR_TARGET}::g' -e s:'--with-libtool-sysroot=${STAGING_DIR_TARGET}'::g \
+ -e 's:${STAGING_DIR}/${MACHINE}${datadir}/pkgconfig:${datadir}/pkgconfig:g' \
+ -e 's:${STAGING_DIR_NATIVE}::g' \
+ -e 's:${STAGING_DIR_HOST}::g' \
+ -e 's:${WORKDIR}::g' \
+ -e 's:${TMPDIR}::g' \
${PKGD}/${libdir}/python${PYTHON_MAJMIN}/config/Makefile \
+ ${PKGD}/${libdir}/python${PYTHON_MAJMIN}/config/config.c \
${PKGD}/${libdir}/python${PYTHON_MAJMIN}/_sysconfigdata.py
python -m py_compile ${PKGD}/${libdir}/python${PYTHON_MAJMIN}/_sysconfigdata.py
}
--
1.9.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] python2/3: Remove building host path from packages
2017-04-11 8:03 [PATCH] python2/3: Remove building host path from packages Li Zhou
@ 2017-04-11 15:44 ` Richard Purdie
0 siblings, 0 replies; 2+ messages in thread
From: Richard Purdie @ 2017-04-11 15:44 UTC (permalink / raw)
To: Li Zhou, openembedded-core
On Tue, 2017-04-11 at 16:03 +0800, Li Zhou wrote:
>
> Remove building host related path from python2 and python3 packages.
>
> Signed-off-by: Li Zhou <li.zhou@windriver.com>
> ---
> meta/recipes-devtools/python/python3_3.5.2.bb | 8 +++++++-
> meta/recipes-devtools/python/python_2.7.13.bb | 6 ++++++
> 2 files changed, 13 insertions(+), 1 deletion(-)
>
> diff --git a/meta/recipes-devtools/python/python3_3.5.2.bb b/meta/recipes-devtools/python/python3_3.5.2.bb
> index 2ff7c9e..ca8dfe3 100644
> --- a/meta/recipes-devtools/python/python3_3.5.2.bb
> +++ b/meta/recipes-devtools/python/python3_3.5.2.bb
> @@ -185,9 +185,15 @@ py_package_preprocess () {
> install -m 0644 ${B}/Makefile.orig ${PKGD}/${libdir}/python${PYTHON_MAJMIN}/config-${PYTHON_MAJMIN}${PYTHON_ABI}/Makefile
> # Remove references to buildmachine paths in target Makefile and _sysconfigdata
> sed -i -e 's:--sysroot=${STAGING_DIR_TARGET}::g' -e s:'--with-libtool-sysroot=${STAGING_DIR_TARGET}'::g \
> + -e 's:${STAGING_DIR}/${MACHINE}${datadir}/pkgconfig:${datadir}/pkgconfig:g' \
> + -e 's:${STAGING_DIR_NATIVE}::g' \
> + -e 's:${STAGING_DIR_HOST}::g' \
> + -e 's:${WORKDIR}::g' \
> + -e 's:${TMPDIR}::g' \
> ${PKGD}/${libdir}/python${PYTHON_MAJMIN}/config/Makefile \
> ${PKGD}/${libdir}/python${PYTHON_MAJMIN}/config-${PYTHON_MAJMIN}${PYTHON_ABI}/Makefile \
> - ${PKGD}/${libdir}/python${PYTHON_MAJMIN}/_sysconfigdata.py
> + ${PKGD}/${libdir}/python${PYTHON_MAJMIN}/_sysconfigdata.py \
> + ${PKGD}/${bindir}/python${PYTHON_MAJMIN}${PYTHON_ABI}-config
> }
This breaks our builds since it regresses
$ oe-selftest -r sstatetests.SStateTests.test_sstate_sametune_samesigs
which is due to the use of MACHINE above. It also raises the question
about why we're fixing paths which do not exist in master?
This is the second patch in 24 hours from Wind River which is not
appropriate for master, not tested against it and actively breaks
things.
In case its not clear, the
path s:${STAGING_DIR}/${MACHINE}${datadir}/pkgconfig does not exist in
master after the RSS. You're therefore trying to get us to add complex
pointless broken code.
Yes, I'm not happy at all in case that isn't clear either as this is a
total waste of my time.
Richard
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2017-04-11 16:01 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-04-11 8:03 [PATCH] python2/3: Remove building host path from packages Li Zhou
2017-04-11 15:44 ` Richard Purdie
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.