* [PATCH 2/3] lsof: Make it compatible with externalsrc
2018-09-24 19:03 [PATCH 1/3] acl: Make it compatible with externalsrc Peter Kjellerstedt
@ 2018-09-24 19:03 ` Peter Kjellerstedt
2018-09-24 19:03 ` [PATCH 3/3] libxml2: " Peter Kjellerstedt
2018-09-25 8:13 ` [PATCH 1/3] acl: " Richard Purdie
2 siblings, 0 replies; 7+ messages in thread
From: Peter Kjellerstedt @ 2018-09-24 19:03 UTC (permalink / raw)
To: openembedded-core
Make the unpack task do nothing if externalsrc is in use. This avoids
the following error after having done `devtool modify lsof`:
ERROR: lsof-4.91-r0 do_unpack: Unpack failure for URL:
'file://.../builds/qemux86-64/tmp/work/core2-64-poky-linux/lsof/4.91-r0/lsof_4.91/lsof_4.91_src.tar'.
Unpack command PATH="..." tar x --no-same-owner -f
.../builds/qemux86-64/tmp/work/core2-64-poky-linux/lsof/4.91-r0/lsof_4.91/lsof_4.91_src.tar
failed with return value 2
Signed-off-by: Peter Kjellerstedt <peter.kjellerstedt@axis.com>
---
meta/recipes-extended/lsof/lsof_4.91.bb | 23 +++++++++++------------
1 file changed, 11 insertions(+), 12 deletions(-)
diff --git a/meta/recipes-extended/lsof/lsof_4.91.bb b/meta/recipes-extended/lsof/lsof_4.91.bb
index 0128de4a6e..b3adfd57af 100644
--- a/meta/recipes-extended/lsof/lsof_4.91.bb
+++ b/meta/recipes-extended/lsof/lsof_4.91.bb
@@ -23,18 +23,17 @@ LOCALSRC = "file://${WORKDIR}/lsof_${PV}/lsof_${PV}_src.tar"
S = "${WORKDIR}/lsof_${PV}_src"
python do_unpack () {
- # temporarily change S for unpack
- # of lsof_${PV}
- s = d.getVar('S', False)
- d.setVar('S', '${WORKDIR}/lsof_${PV}')
- bb.build.exec_func('base_do_unpack', d)
- # temporarily change SRC_URI for unpack
- # of lsof_${PV}_src
- src_uri = d.getVar('SRC_URI', False)
- d.setVar('SRC_URI', '${LOCALSRC}')
- d.setVar('S', s)
- bb.build.exec_func('base_do_unpack', d)
- d.setVar('SRC_URI', src_uri)
+ if not bb.data.inherits_class('externalsrc', d) or not d.getVar('EXTERNALSRC'):
+ # temporarily change S for unpack of lsof_${PV}
+ s = d.getVar('S', False)
+ d.setVar('S', '${WORKDIR}/lsof_${PV}')
+ bb.build.exec_func('base_do_unpack', d)
+ # temporarily change SRC_URI for unpack of lsof_${PV}_src
+ src_uri = d.getVar('SRC_URI', False)
+ d.setVar('SRC_URI', '${LOCALSRC}')
+ d.setVar('S', s)
+ bb.build.exec_func('base_do_unpack', d)
+ d.setVar('SRC_URI', src_uri)
}
export LSOF_INCLUDE = "${STAGING_INCDIR}"
--
2.12.0
^ permalink raw reply related [flat|nested] 7+ messages in thread* [PATCH 3/3] libxml2: Make it compatible with externalsrc
2018-09-24 19:03 [PATCH 1/3] acl: Make it compatible with externalsrc Peter Kjellerstedt
2018-09-24 19:03 ` [PATCH 2/3] lsof: " Peter Kjellerstedt
@ 2018-09-24 19:03 ` Peter Kjellerstedt
2018-09-25 8:13 ` [PATCH 1/3] acl: " Richard Purdie
2 siblings, 0 replies; 7+ messages in thread
From: Peter Kjellerstedt @ 2018-09-24 19:03 UTC (permalink / raw)
To: openembedded-core
Fetch the test tar ball to a subdirectory in ${S}. This avoids the
following error after having done `devtool modify libxml2`:
| DEBUG: Executing shell function do_configure
| find: ‘.../build/tmp/work/mips32r2el-nf-poky-linux/libxml2/2.9.4-r0/xmlconf/’: No such file or directory
Signed-off-by: Peter Kjellerstedt <peter.kjellerstedt@axis.com>
---
meta/recipes-core/libxml/libxml2_2.9.8.bb | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/meta/recipes-core/libxml/libxml2_2.9.8.bb b/meta/recipes-core/libxml/libxml2_2.9.8.bb
index f01cb2cd34..4ec03c6d15 100644
--- a/meta/recipes-core/libxml/libxml2_2.9.8.bb
+++ b/meta/recipes-core/libxml/libxml2_2.9.8.bb
@@ -12,7 +12,7 @@ LIC_FILES_CHKSUM = "file://Copyright;md5=2044417e2e5006b65a8b9067b683fcf1 \
DEPENDS = "zlib virtual/libiconv"
SRC_URI = "http://www.xmlsoft.org/sources/libxml2-${PV}.tar.gz;name=libtar \
- http://www.w3.org/XML/Test/xmlts20080827.tar.gz;name=testtar \
+ http://www.w3.org/XML/Test/xmlts20080827.tar.gz;subdir=${BP};name=testtar \
file://libxml-64bit.patch \
file://runtest.patch \
file://run-ptest \
@@ -76,7 +76,7 @@ FILES_${PN}-python += "${PYTHON_SITEPACKAGES_DIR}"
do_configure_prepend () {
# executables take longer to package: these should not be executable
- find ${WORKDIR}/xmlconf/ -type f -exec chmod -x {} \+
+ find ${S}/xmlconf/ -type f -exec chmod -x {} \+
}
do_compile_ptest() {
@@ -84,7 +84,7 @@ do_compile_ptest() {
}
do_install_ptest () {
- cp -r ${WORKDIR}/xmlconf ${D}${PTEST_PATH}
+ cp -r ${S}/xmlconf ${D}${PTEST_PATH}
if [ "${@bb.utils.filter('PACKAGECONFIG', 'python', d)}" ]; then
sed -i -e 's|^\(PYTHON = \).*|\1${USRBINPATH}/${PYTHON_PN}|' \
${D}${PTEST_PATH}/python/tests/Makefile
--
2.12.0
^ permalink raw reply related [flat|nested] 7+ messages in thread* Re: [PATCH 1/3] acl: Make it compatible with externalsrc
2018-09-24 19:03 [PATCH 1/3] acl: Make it compatible with externalsrc Peter Kjellerstedt
2018-09-24 19:03 ` [PATCH 2/3] lsof: " Peter Kjellerstedt
2018-09-24 19:03 ` [PATCH 3/3] libxml2: " Peter Kjellerstedt
@ 2018-09-25 8:13 ` Richard Purdie
2018-09-25 9:17 ` Peter Kjellerstedt
2 siblings, 1 reply; 7+ messages in thread
From: Richard Purdie @ 2018-09-25 8:13 UTC (permalink / raw)
To: Peter Kjellerstedt, openembedded-core
On Mon, 2018-09-24 at 21:03 +0200, Peter Kjellerstedt wrote:
> Make the subdir fetch path for configure.ac relative. This avoids the
> following error after having done `devtool modify acl`:
>
> ERROR: acl-2.2.52-r0 do_unpack: Unpack failure for URL:
> 'file://configure.ac;subdir=.../builds/qemux86-
> 64/workspace/sources/acl'.
> subdir argument isn't a subdirectory of unpack root
> .../builds/qemux86-64/tmp/work/core2-64-poky-linux/acl/2.2.52-r0
>
> Signed-off-by: Peter Kjellerstedt <peter.kjellerstedt@axis.com>
> ---
> meta/recipes-support/attr/acl_2.2.52.bb | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/meta/recipes-support/attr/acl_2.2.52.bb b/meta/recipes-
> support/attr/acl_2.2.52.bb
> index 8f3dc45cf7..8b89de9b42 100644
> --- a/meta/recipes-support/attr/acl_2.2.52.bb
> +++ b/meta/recipes-support/attr/acl_2.2.52.bb
> @@ -11,7 +11,7 @@ LIC_FILES_CHKSUM = "file://doc/COPYING;md5=c781d70e
> d2b4d48995b790403217a249 \
> DEPENDS = "attr"
>
> SRC_URI = "${SAVANNAH_GNU_MIRROR}/acl/${BP}.src.tar.gz \
> - file://configure.ac;subdir=${S} \
> + file://configure.ac;subdir=${BP} \
> file://run-ptest \
> file://acl-fix-the-order-of-expected-output-of-
> getfacl.patch \
> file://test-fix-insufficient-quoting-of.patch \
This isn't making sense to me. Doesn't externalsrc set ${S} to the
external source location?
Also, doesn't it remove the do_unpack task?
Cheers,
Richard
^ permalink raw reply [flat|nested] 7+ messages in thread* Re: [PATCH 1/3] acl: Make it compatible with externalsrc
2018-09-25 8:13 ` [PATCH 1/3] acl: " Richard Purdie
@ 2018-09-25 9:17 ` Peter Kjellerstedt
2018-10-02 10:25 ` Peter Kjellerstedt
0 siblings, 1 reply; 7+ messages in thread
From: Peter Kjellerstedt @ 2018-09-25 9:17 UTC (permalink / raw)
To: Richard Purdie, openembedded-core@lists.openembedded.org
> -----Original Message-----
> From: openembedded-core-bounces@lists.openembedded.org <openembedded-
> core-bounces@lists.openembedded.org> On Behalf Of Richard Purdie
> Sent: den 25 september 2018 10:13
> To: Peter Kjellerstedt <peter.kjellerstedt@axis.com>; openembedded-
> core@lists.openembedded.org
> Subject: Re: [OE-core] [PATCH 1/3] acl: Make it compatible with
> externalsrc
>
> On Mon, 2018-09-24 at 21:03 +0200, Peter Kjellerstedt wrote:
> > Make the subdir fetch path for configure.ac relative. This avoids the
> > following error after having done `devtool modify acl`:
> >
> > ERROR: acl-2.2.52-r0 do_unpack: Unpack failure for URL:
> > 'file://configure.ac;subdir=.../builds/qemux86-
> > 64/workspace/sources/acl'.
> > subdir argument isn't a subdirectory of unpack root
> > .../builds/qemux86-64/tmp/work/core2-64-poky-linux/acl/2.2.52-r0
> >
> > Signed-off-by: Peter Kjellerstedt <peter.kjellerstedt@axis.com>
> > ---
> > meta/recipes-support/attr/acl_2.2.52.bb | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/meta/recipes-support/attr/acl_2.2.52.bb b/meta/recipes-
> > support/attr/acl_2.2.52.bb
> > index 8f3dc45cf7..8b89de9b42 100644
> > --- a/meta/recipes-support/attr/acl_2.2.52.bb
> > +++ b/meta/recipes-support/attr/acl_2.2.52.bb
> > @@ -11,7 +11,7 @@ LIC_FILES_CHKSUM = "file://doc/COPYING;md5=c781d70e
> > d2b4d48995b790403217a249 \
> > DEPENDS = "attr"
> >
> > SRC_URI = "${SAVANNAH_GNU_MIRROR}/acl/${BP}.src.tar.gz \
> > - file://configure.ac;subdir=${S} \
> > + file://configure.ac;subdir=${BP} \
> > file://run-ptest \
> > file://acl-fix-the-order-of-expected-output-of-getfacl.patch \
> > file://test-fix-insufficient-quoting-of.patch \
>
> This isn't making sense to me. Doesn't externalsrc set ${S} to the
> external source location?
Yes, it does, but it does not change ${WORKDIR}. So when it tries
to unpack to the absolute path ${S}, which is outside ${WORKDIR}
when externalsrc is in use, then the above error occurs. Using a
relative path avoids the problem.
> Also, doesn't it remove the do_unpack task?
No, it keeps the task but removes everything except file:// URLs
and URLs with type=kmeta from the SRC_URI. This is also why I had
to check for externalsrc in the do_unpack task of lsof.
> Cheers,
>
> Richard
//Peter
^ permalink raw reply [flat|nested] 7+ messages in thread* Re: [PATCH 1/3] acl: Make it compatible with externalsrc
2018-09-25 9:17 ` Peter Kjellerstedt
@ 2018-10-02 10:25 ` Peter Kjellerstedt
2018-10-10 8:21 ` Peter Kjellerstedt
0 siblings, 1 reply; 7+ messages in thread
From: Peter Kjellerstedt @ 2018-10-02 10:25 UTC (permalink / raw)
To: Richard Purdie, openembedded-core@lists.openembedded.org
*ping*
//Peter
> -----Original Message-----
> From: openembedded-core-bounces@lists.openembedded.org <openembedded-
> core-bounces@lists.openembedded.org> On Behalf Of Peter Kjellerstedt
> Sent: den 25 september 2018 11:17
> To: Richard Purdie <richard.purdie@linuxfoundation.org>; openembedded-
> core@lists.openembedded.org
> Subject: Re: [OE-core] [PATCH 1/3] acl: Make it compatible with
> externalsrc
>
> > -----Original Message-----
> > From: openembedded-core-bounces@lists.openembedded.org <openembedded-
> > core-bounces@lists.openembedded.org> On Behalf Of Richard Purdie
> > Sent: den 25 september 2018 10:13
> > To: Peter Kjellerstedt <peter.kjellerstedt@axis.com>; openembedded-
> > core@lists.openembedded.org
> > Subject: Re: [OE-core] [PATCH 1/3] acl: Make it compatible with
> > externalsrc
> >
> > On Mon, 2018-09-24 at 21:03 +0200, Peter Kjellerstedt wrote:
> > > Make the subdir fetch path for configure.ac relative. This avoids
> the
> > > following error after having done `devtool modify acl`:
> > >
> > > ERROR: acl-2.2.52-r0 do_unpack: Unpack failure for URL:
> > > 'file://configure.ac;subdir=.../builds/qemux86-
> > > 64/workspace/sources/acl'.
> > > subdir argument isn't a subdirectory of unpack root
> > > .../builds/qemux86-64/tmp/work/core2-64-poky-linux/acl/2.2.52-r0
> > >
> > > Signed-off-by: Peter Kjellerstedt <peter.kjellerstedt@axis.com>
> > > ---
> > > meta/recipes-support/attr/acl_2.2.52.bb | 2 +-
> > > 1 file changed, 1 insertion(+), 1 deletion(-)
> > >
> > > diff --git a/meta/recipes-support/attr/acl_2.2.52.bb
> b/meta/recipes-
> > > support/attr/acl_2.2.52.bb
> > > index 8f3dc45cf7..8b89de9b42 100644
> > > --- a/meta/recipes-support/attr/acl_2.2.52.bb
> > > +++ b/meta/recipes-support/attr/acl_2.2.52.bb
> > > @@ -11,7 +11,7 @@ LIC_FILES_CHKSUM =
> "file://doc/COPYING;md5=c781d70e
> > > d2b4d48995b790403217a249 \
> > > DEPENDS = "attr"
> > >
> > > SRC_URI = "${SAVANNAH_GNU_MIRROR}/acl/${BP}.src.tar.gz \
> > > - file://configure.ac;subdir=${S} \
> > > + file://configure.ac;subdir=${BP} \
> > > file://run-ptest \
> > > file://acl-fix-the-order-of-expected-output-of-
> getfacl.patch \
> > > file://test-fix-insufficient-quoting-of.patch \
> >
> > This isn't making sense to me. Doesn't externalsrc set ${S} to the
> > external source location?
>
> Yes, it does, but it does not change ${WORKDIR}. So when it tries
> to unpack to the absolute path ${S}, which is outside ${WORKDIR}
> when externalsrc is in use, then the above error occurs. Using a
> relative path avoids the problem.
>
> > Also, doesn't it remove the do_unpack task?
>
> No, it keeps the task but removes everything except file:// URLs
> and URLs with type=kmeta from the SRC_URI. This is also why I had
> to check for externalsrc in the do_unpack task of lsof.
>
> > Cheers,
> >
> > Richard
>
> //Peter
>
> --
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core
^ permalink raw reply [flat|nested] 7+ messages in thread* Re: [PATCH 1/3] acl: Make it compatible with externalsrc
2018-10-02 10:25 ` Peter Kjellerstedt
@ 2018-10-10 8:21 ` Peter Kjellerstedt
0 siblings, 0 replies; 7+ messages in thread
From: Peter Kjellerstedt @ 2018-10-10 8:21 UTC (permalink / raw)
To: Richard Purdie; +Cc: openembedded-core@lists.openembedded.org
Except for the questions asked and answered below, is there
anything else preventing these three patches from being merged?
//Peter
> -----Original Message-----
> From: openembedded-core-bounces@lists.openembedded.org <openembedded-
> core-bounces@lists.openembedded.org> On Behalf Of Peter Kjellerstedt
> Sent: den 2 oktober 2018 12:26
> To: Richard Purdie <richard.purdie@linuxfoundation.org>; openembedded-
> core@lists.openembedded.org
> Subject: Re: [OE-core] [PATCH 1/3] acl: Make it compatible with
> externalsrc
>
> *ping*
>
> //Peter
>
> > -----Original Message-----
> > From: openembedded-core-bounces@lists.openembedded.org <openembedded-
> > core-bounces@lists.openembedded.org> On Behalf Of Peter Kjellerstedt
> > Sent: den 25 september 2018 11:17
> > To: Richard Purdie <richard.purdie@linuxfoundation.org>; openembedded-core@lists.openembedded.org
> > Subject: Re: [OE-core] [PATCH 1/3] acl: Make it compatible with externalsrc
> >
> > > -----Original Message-----
> > > From: openembedded-core-bounces@lists.openembedded.org <openembedded-
> > > core-bounces@lists.openembedded.org> On Behalf Of Richard Purdie
> > > Sent: den 25 september 2018 10:13
> > > To: Peter Kjellerstedt <peter.kjellerstedt@axis.com>; openembedded-
> > > core@lists.openembedded.org
> > > Subject: Re: [OE-core] [PATCH 1/3] acl: Make it compatible with
> > > externalsrc
> > >
> > > On Mon, 2018-09-24 at 21:03 +0200, Peter Kjellerstedt wrote:
> > > > Make the subdir fetch path for configure.ac relative. This avoids the
> > > > following error after having done `devtool modify acl`:
> > > >
> > > > ERROR: acl-2.2.52-r0 do_unpack: Unpack failure for URL:
> > > > 'file://configure.ac;subdir=.../builds/qemux86-64/workspace/sources/acl'.
> > > > subdir argument isn't a subdirectory of unpack root
> > > > .../builds/qemux86-64/tmp/work/core2-64-poky-linux/acl/2.2.52-r0
> > > >
> > > > Signed-off-by: Peter Kjellerstedt <peter.kjellerstedt@axis.com>
> > > > ---
> > > > meta/recipes-support/attr/acl_2.2.52.bb | 2 +-
> > > > 1 file changed, 1 insertion(+), 1 deletion(-)
> > > >
> > > > diff --git a/meta/recipes-support/attr/acl_2.2.52.bb b/meta/recipes-support/attr/acl_2.2.52.bb
> > > > index 8f3dc45cf7..8b89de9b42 100644
> > > > --- a/meta/recipes-support/attr/acl_2.2.52.bb
> > > > +++ b/meta/recipes-support/attr/acl_2.2.52.bb
> > > > @@ -11,7 +11,7 @@ LIC_FILES_CHKSUM = "file://doc/COPYING;md5=c781d70ed2b4d48995b790403217a249 \
> > > > DEPENDS = "attr"
> > > >
> > > > SRC_URI = "${SAVANNAH_GNU_MIRROR}/acl/${BP}.src.tar.gz \
> > > > - file://configure.ac;subdir=${S} \
> > > > + file://configure.ac;subdir=${BP} \
> > > > file://run-ptest \
> > > > file://acl-fix-the-order-of-expected-output-of-getfacl.patch \
> > > > file://test-fix-insufficient-quoting-of.patch \
> > >
> > > This isn't making sense to me. Doesn't externalsrc set ${S} to the
> > > external source location?
> >
> > Yes, it does, but it does not change ${WORKDIR}. So when it tries
> > to unpack to the absolute path ${S}, which is outside ${WORKDIR}
> > when externalsrc is in use, then the above error occurs. Using a
> > relative path avoids the problem.
> >
> > > Also, doesn't it remove the do_unpack task?
> >
> > No, it keeps the task but removes everything except file:// URLs
> > and URLs with type=kmeta from the SRC_URI. This is also why I had
> > to check for externalsrc in the do_unpack task of lsof.
> >
> > > Cheers,
> > >
> > > Richard
> >
> > //Peter
^ permalink raw reply [flat|nested] 7+ messages in thread