* [PATCH v2] ofono: upgrade 1.25 -> 1.29
@ 2019-05-08 20:01 Adrian Bunk
2019-05-09 6:48 ` Mittal, Anuj
0 siblings, 1 reply; 3+ messages in thread
From: Adrian Bunk @ 2019-05-08 20:01 UTC (permalink / raw)
To: openembedded-core
Remove patch applied upstream.
Fix a new parallel build failure.
Signed-off-by: Adrian Bunk <bunk@stusta.de>
---
...a-race-condition-where-ell-ell.h-was.patch | 28 +++++++++++++++++++
.../ofono/ofono/use-python3.patch | 27 ------------------
meta/recipes-connectivity/ofono/ofono_1.25.bb | 9 ------
meta/recipes-connectivity/ofono/ofono_1.29.bb | 9 ++++++
4 files changed, 37 insertions(+), 36 deletions(-)
create mode 100644 meta/recipes-connectivity/ofono/ofono/0001-Makefile.am-Fix-a-race-condition-where-ell-ell.h-was.patch
delete mode 100644 meta/recipes-connectivity/ofono/ofono/use-python3.patch
delete mode 100644 meta/recipes-connectivity/ofono/ofono_1.25.bb
create mode 100644 meta/recipes-connectivity/ofono/ofono_1.29.bb
diff --git a/meta/recipes-connectivity/ofono/ofono/0001-Makefile.am-Fix-a-race-condition-where-ell-ell.h-was.patch b/meta/recipes-connectivity/ofono/ofono/0001-Makefile.am-Fix-a-race-condition-where-ell-ell.h-was.patch
new file mode 100644
index 0000000000..1dfc977add
--- /dev/null
+++ b/meta/recipes-connectivity/ofono/ofono/0001-Makefile.am-Fix-a-race-condition-where-ell-ell.h-was.patch
@@ -0,0 +1,28 @@
+From 61c78979eab39ead47a91aeacad43ad72d162052 Mon Sep 17 00:00:00 2001
+From: Adrian Bunk <bunk@kernel.org>
+Date: Wed, 8 May 2019 21:58:44 +0300
+Subject: Makefile.am: Fix a race condition where ell/ell.h was written before
+ ell/ existed
+
+Signed-off-by: Adrian Bunk <bunk@kernel.org>
+Upstream-Status: Pending
+
+---
+ Makefile.am | 1 +
+ 1 file changed, 1 insertion(+)
+
+diff --git a/Makefile.am b/Makefile.am
+index a569c4a3..d2e310d5 100644
+--- a/Makefile.am
++++ b/Makefile.am
+@@ -1107,6 +1107,7 @@ ell/internal: Makefile
+ done > $@
+
+ ell/ell.h: Makefile
++ $(AM_V_at)$(MKDIR_P) ell
+ $(AM_V_at)echo -n > $@
+ $(AM_V_GEN)for f in $(ell_headers) ; do \
+ echo "#include <$$f>" >> $@ ; \
+--
+2.20.1
+
diff --git a/meta/recipes-connectivity/ofono/ofono/use-python3.patch b/meta/recipes-connectivity/ofono/ofono/use-python3.patch
deleted file mode 100644
index 7b84075257..0000000000
--- a/meta/recipes-connectivity/ofono/ofono/use-python3.patch
+++ /dev/null
@@ -1,27 +0,0 @@
-set-ddr should use Python3 like all the other tests.
-
-Upstream-Status: Submitted
-Signed-off-by: Ross Burton <ross.burton@intel.com>
-
-From 17b69cd1da4c5c5f732acb38ca1602446c567ee7 Mon Sep 17 00:00:00 2001
-From: Ross Burton <ross.burton@intel.com>
-Date: Mon, 29 Jan 2018 11:31:25 +0000
-Subject: [PATCH] test/setddr: use Python 3
-
-All the other tests use Python 3, so this should to.
----
- test/set-ddr | 2 +-
- 1 file changed, 1 insertion(+), 1 deletion(-)
-
-diff --git a/test/set-ddr b/test/set-ddr
-index 5d061b95..33631f31 100755
---- a/test/set-ddr
-+++ b/test/set-ddr
-@@ -1,4 +1,4 @@
--#!/usr/bin/python
-+#!/usr/bin/python3
-
- import sys
- import dbus
---
-2.11.0
diff --git a/meta/recipes-connectivity/ofono/ofono_1.25.bb b/meta/recipes-connectivity/ofono/ofono_1.25.bb
deleted file mode 100644
index 3688b9d2fc..0000000000
--- a/meta/recipes-connectivity/ofono/ofono_1.25.bb
+++ /dev/null
@@ -1,9 +0,0 @@
-require ofono.inc
-
-SRC_URI = "\
- ${KERNELORG_MIRROR}/linux/network/${BPN}/${BP}.tar.xz \
- file://ofono \
- file://use-python3.patch \
-"
-SRC_URI[md5sum] = "31450cabdd8dbbf3f808ea2f2f066863"
-SRC_URI[sha256sum] = "eb011fcd3080e93f3a56f96be60350b6595a8b5f36b61646312ba41b0bcb0d75"
diff --git a/meta/recipes-connectivity/ofono/ofono_1.29.bb b/meta/recipes-connectivity/ofono/ofono_1.29.bb
new file mode 100644
index 0000000000..8539aa1372
--- /dev/null
+++ b/meta/recipes-connectivity/ofono/ofono_1.29.bb
@@ -0,0 +1,9 @@
+require ofono.inc
+
+SRC_URI = "\
+ ${KERNELORG_MIRROR}/linux/network/${BPN}/${BP}.tar.xz \
+ file://ofono \
+ file://0001-Makefile.am-Fix-a-race-condition-where-ell-ell.h-was.patch \
+"
+SRC_URI[md5sum] = "4fa0372630ff03f223452e4d05efa8f8"
+SRC_URI[sha256sum] = "67f0f8e5740dea5b46309e40667d1e560be39c90ef08dd01ff9e9ce8e61f0679"
--
2.17.1
^ permalink raw reply related [flat|nested] 3+ messages in thread* Re: [PATCH v2] ofono: upgrade 1.25 -> 1.29
2019-05-08 20:01 [PATCH v2] ofono: upgrade 1.25 -> 1.29 Adrian Bunk
@ 2019-05-09 6:48 ` Mittal, Anuj
2019-05-09 7:50 ` Adrian Bunk
0 siblings, 1 reply; 3+ messages in thread
From: Mittal, Anuj @ 2019-05-09 6:48 UTC (permalink / raw)
To: openembedded-core@lists.openembedded.org, bunk@stusta.de
This is failing with error:
| In file included from ../ofono-1.29/ell/random.c:34:
| ../ofono-1.29/ell/missing.h:59:20: error: static declaration of
'explicit_bzero' follows non-static declaration
| static inline void explicit_bzero(void *s, size_t n)
| ^~~~~~~~~~~~~~
| In file included from ../ofono-1.29/ell/util.h:26,
| from ../ofono-1.29/ell/private.h:26,
| from ../ofono-1.29/ell/random.c:33:
| /home/pokybuild/yocto-worker/musl-qemux86-
64/build/build/tmp/work/core2-64-poky-linux-musl/ofono/1.29-r0/recipe-
sysroot/usr/include/string.h:85:6: note: previous declaration of
'explicit_bzero' was here
| void explicit_bzero (void *, size_t);
| ^~~~~~~~~~~~~~
| make[1]: *** [ell/random.lo] Error 1
| make[1]: *** Waiting for unfinished jobs....
Logs here:
https://autobuilder.yoctoproject.org/typhoon/#/builders/45/builds/578/steps/7/logs/step1b
Thanks,
Anuj
On Wed, 2019-05-08 at 23:01 +0300, Adrian Bunk wrote:
> Remove patch applied upstream.
> Fix a new parallel build failure.
>
> Signed-off-by: Adrian Bunk <bunk@stusta.de>
> ---
> ...a-race-condition-where-ell-ell.h-was.patch | 28
> +++++++++++++++++++
> .../ofono/ofono/use-python3.patch | 27 ---------------
> ---
> meta/recipes-connectivity/ofono/ofono_1.25.bb | 9 ------
> meta/recipes-connectivity/ofono/ofono_1.29.bb | 9 ++++++
> 4 files changed, 37 insertions(+), 36 deletions(-)
> create mode 100644 meta/recipes-connectivity/ofono/ofono/0001-
> Makefile.am-Fix-a-race-condition-where-ell-ell.h-was.patch
> delete mode 100644 meta/recipes-connectivity/ofono/ofono/use-
> python3.patch
> delete mode 100644 meta/recipes-connectivity/ofono/ofono_1.25.bb
> create mode 100644 meta/recipes-connectivity/ofono/ofono_1.29.bb
>
> diff --git a/meta/recipes-connectivity/ofono/ofono/0001-Makefile.am-
> Fix-a-race-condition-where-ell-ell.h-was.patch b/meta/recipes-
> connectivity/ofono/ofono/0001-Makefile.am-Fix-a-race-condition-where-
> ell-ell.h-was.patch
> new file mode 100644
> index 0000000000..1dfc977add
> --- /dev/null
> +++ b/meta/recipes-connectivity/ofono/ofono/0001-Makefile.am-Fix-a-
> race-condition-where-ell-ell.h-was.patch
> @@ -0,0 +1,28 @@
> +From 61c78979eab39ead47a91aeacad43ad72d162052 Mon Sep 17 00:00:00
> 2001
> +From: Adrian Bunk <bunk@kernel.org>
> +Date: Wed, 8 May 2019 21:58:44 +0300
> +Subject: Makefile.am: Fix a race condition where ell/ell.h was
> written before
> + ell/ existed
> +
> +Signed-off-by: Adrian Bunk <bunk@kernel.org>
> +Upstream-Status: Pending
> +
> +---
> + Makefile.am | 1 +
> + 1 file changed, 1 insertion(+)
> +
> +diff --git a/Makefile.am b/Makefile.am
> +index a569c4a3..d2e310d5 100644
> +--- a/Makefile.am
> ++++ b/Makefile.am
> +@@ -1107,6 +1107,7 @@ ell/internal: Makefile
> + done > $@
> +
> + ell/ell.h: Makefile
> ++ $(AM_V_at)$(MKDIR_P) ell
> + $(AM_V_at)echo -n > $@
> + $(AM_V_GEN)for f in $(ell_headers) ; do \
> + echo "#include <$$f>" >> $@ ; \
> +--
> +2.20.1
> +
> diff --git a/meta/recipes-connectivity/ofono/ofono/use-python3.patch
> b/meta/recipes-connectivity/ofono/ofono/use-python3.patch
> deleted file mode 100644
> index 7b84075257..0000000000
> --- a/meta/recipes-connectivity/ofono/ofono/use-python3.patch
> +++ /dev/null
> @@ -1,27 +0,0 @@
> -set-ddr should use Python3 like all the other tests.
> -
> -Upstream-Status: Submitted
> -Signed-off-by: Ross Burton <ross.burton@intel.com>
> -
> -From 17b69cd1da4c5c5f732acb38ca1602446c567ee7 Mon Sep 17 00:00:00
> 2001
> -From: Ross Burton <ross.burton@intel.com>
> -Date: Mon, 29 Jan 2018 11:31:25 +0000
> -Subject: [PATCH] test/setddr: use Python 3
> -
> -All the other tests use Python 3, so this should to.
> ----
> - test/set-ddr | 2 +-
> - 1 file changed, 1 insertion(+), 1 deletion(-)
> -
> -diff --git a/test/set-ddr b/test/set-ddr
> -index 5d061b95..33631f31 100755
> ---- a/test/set-ddr
> -+++ b/test/set-ddr
> -@@ -1,4 +1,4 @@
> --#!/usr/bin/python
> -+#!/usr/bin/python3
> -
> - import sys
> - import dbus
> ---
> -2.11.0
> diff --git a/meta/recipes-connectivity/ofono/ofono_1.25.bb
> b/meta/recipes-connectivity/ofono/ofono_1.25.bb
> deleted file mode 100644
> index 3688b9d2fc..0000000000
> --- a/meta/recipes-connectivity/ofono/ofono_1.25.bb
> +++ /dev/null
> @@ -1,9 +0,0 @@
> -require ofono.inc
> -
> -SRC_URI = "\
> - ${KERNELORG_MIRROR}/linux/network/${BPN}/${BP}.tar.xz \
> - file://ofono \
> - file://use-python3.patch \
> -"
> -SRC_URI[md5sum] = "31450cabdd8dbbf3f808ea2f2f066863"
> -SRC_URI[sha256sum] =
> "eb011fcd3080e93f3a56f96be60350b6595a8b5f36b61646312ba41b0bcb0d75"
> diff --git a/meta/recipes-connectivity/ofono/ofono_1.29.bb
> b/meta/recipes-connectivity/ofono/ofono_1.29.bb
> new file mode 100644
> index 0000000000..8539aa1372
> --- /dev/null
> +++ b/meta/recipes-connectivity/ofono/ofono_1.29.bb
> @@ -0,0 +1,9 @@
> +require ofono.inc
> +
> +SRC_URI = "\
> + ${KERNELORG_MIRROR}/linux/network/${BPN}/${BP}.tar.xz \
> + file://ofono \
> + file://0001-Makefile.am-Fix-a-race-condition-where-ell-ell.h-
> was.patch \
> +"
> +SRC_URI[md5sum] = "4fa0372630ff03f223452e4d05efa8f8"
> +SRC_URI[sha256sum] =
> "67f0f8e5740dea5b46309e40667d1e560be39c90ef08dd01ff9e9ce8e61f0679"
> --
> 2.17.1
>
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [PATCH v2] ofono: upgrade 1.25 -> 1.29
2019-05-09 6:48 ` Mittal, Anuj
@ 2019-05-09 7:50 ` Adrian Bunk
0 siblings, 0 replies; 3+ messages in thread
From: Adrian Bunk @ 2019-05-09 7:50 UTC (permalink / raw)
To: Mittal, Anuj; +Cc: openembedded-core@lists.openembedded.org
On Thu, May 09, 2019 at 06:48:38AM +0000, Mittal, Anuj wrote:
> This is failing with error:
>
> | In file included from ../ofono-1.29/ell/random.c:34:
> | ../ofono-1.29/ell/missing.h:59:20: error: static declaration of
> 'explicit_bzero' follows non-static declaration
> | static inline void explicit_bzero(void *s, size_t n)
> | ^~~~~~~~~~~~~~
> | In file included from ../ofono-1.29/ell/util.h:26,
> | from ../ofono-1.29/ell/private.h:26,
> | from ../ofono-1.29/ell/random.c:33:
> | /home/pokybuild/yocto-worker/musl-qemux86-
> 64/build/build/tmp/work/core2-64-poky-linux-musl/ofono/1.29-r0/recipe-
> sysroot/usr/include/string.h:85:6: note: previous declaration of
> 'explicit_bzero' was here
> | void explicit_bzero (void *, size_t);
> | ^~~~~~~~~~~~~~
> | make[1]: *** [ell/random.lo] Error 1
> | make[1]: *** Waiting for unfinished jobs....
>
> Logs here:
>
> https://autobuilder.yoctoproject.org/typhoon/#/builders/45/builds/578/steps/7/logs/step1b
Thanks, got a fix for that one now.
The next problem is that musl does not currently provide
TEMP_FAILURE_RETRY, and the correct fix for that will be
that I'll try to get this fixed in musl instead of creating
yet another copy of the glibc macro.
I'll come back to ofono after that.
> Thanks,
>
> Anuj
cu
Adrian
--
"Is there not promise of rain?" Ling Tan asked suddenly out
of the darkness. There had been need of rain for many days.
"Only a promise," Lao Er said.
Pearl S. Buck - Dragon Seed
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2019-05-09 7:50 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-05-08 20:01 [PATCH v2] ofono: upgrade 1.25 -> 1.29 Adrian Bunk
2019-05-09 6:48 ` Mittal, Anuj
2019-05-09 7:50 ` Adrian Bunk
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox