* [meta-oe][PATCH] modemmanager: Use a simpler workaround for the clang build
@ 2019-08-06 15:39 Adrian Bunk
2019-08-06 18:09 ` Khem Raj
0 siblings, 1 reply; 3+ messages in thread
From: Adrian Bunk @ 2019-08-06 15:39 UTC (permalink / raw)
To: openembedded-devel
The check whether the compiler understands an option is broken
in m4/compiler_warnings.m4, but if we are just workarounding it
then do a minimal workaround without patching.
Signed-off-by: Adrian Bunk <bunk@stusta.de>
---
...-not-set-Wno-unused-but-set-variable.patch | 30 -------------------
.../modemmanager/modemmanager_1.10.0.bb | 3 +-
2 files changed, 1 insertion(+), 32 deletions(-)
delete mode 100644 meta-oe/recipes-connectivity/modemmanager/modemmanager/0001-Do-not-set-Wno-unused-but-set-variable.patch
diff --git a/meta-oe/recipes-connectivity/modemmanager/modemmanager/0001-Do-not-set-Wno-unused-but-set-variable.patch b/meta-oe/recipes-connectivity/modemmanager/modemmanager/0001-Do-not-set-Wno-unused-but-set-variable.patch
deleted file mode 100644
index dcbe01d5e..000000000
--- a/meta-oe/recipes-connectivity/modemmanager/modemmanager/0001-Do-not-set-Wno-unused-but-set-variable.patch
+++ /dev/null
@@ -1,30 +0,0 @@
-From d4f73b037d11d20f080e74c05df28853fb833075 Mon Sep 17 00:00:00 2001
-From: Khem Raj <raj.khem@gmail.com>
-Date: Fri, 7 Dec 2018 22:59:47 -0800
-Subject: [PATCH] Do not set Wno-unused-but-set-variable
-
-This is gcc specific warning, not implemented by pretending to be gcc
-like compilers e.g. clang.
-
-We will pass this option from recipe
-
-Upstream-Status: Inappropriate [OE-Specific]
-
-Signed-off-by: Khem Raj <raj.khem@gmail.com>
----
- m4/compiler_warnings.m4 | 2 +-
- 1 file changed, 1 insertion(+), 1 deletion(-)
-
-diff --git a/m4/compiler_warnings.m4 b/m4/compiler_warnings.m4
-index f498a28..e65e97f 100644
---- a/m4/compiler_warnings.m4
-+++ b/m4/compiler_warnings.m4
-@@ -11,7 +11,7 @@ if test "$GCC" = "yes" -a "$set_more_warnings" != "no"; then
- -Wdeclaration-after-statement -Wstrict-prototypes \
- -Wno-unused-parameter -Wno-sign-compare \
- -Wno-deprecated-declarations \
-- -Wno-unused-but-set-variable -Wformat-security; do
-+ -Wno-unused -Wformat-security; do
- SAVE_CFLAGS="$CFLAGS"
- CFLAGS="$CFLAGS $option"
- AC_MSG_CHECKING([whether gcc understands $option])
diff --git a/meta-oe/recipes-connectivity/modemmanager/modemmanager_1.10.0.bb b/meta-oe/recipes-connectivity/modemmanager/modemmanager_1.10.0.bb
index 1ad19e8f7..ef5af3a57 100644
--- a/meta-oe/recipes-connectivity/modemmanager/modemmanager_1.10.0.bb
+++ b/meta-oe/recipes-connectivity/modemmanager/modemmanager_1.10.0.bb
@@ -12,7 +12,6 @@ inherit gnomebase gettext systemd vala gobject-introspection bash-completion
DEPENDS = "glib-2.0 libgudev intltool-native libxslt-native"
SRC_URI = "http://www.freedesktop.org/software/ModemManager/ModemManager-${PV}.tar.xz \
- file://0001-Do-not-set-Wno-unused-but-set-variable.patch \
"
SRC_URI[md5sum] = "92d25176d0cc9d588ca29700b33c7d98"
SRC_URI[sha256sum] = "fd0f39996025dac96995daea8a58ec7dd571582f7563a8ae0d5f65b571b76ee2"
@@ -33,7 +32,7 @@ PACKAGECONFIG[qmi] = "--with-qmi,--without-qmi,libqmi"
EXTRA_OECONF = " \
--with-udev-base-dir=${nonarch_base_libdir}/udev \
"
-CFLAGS_append_toolchain-gcc = " -Wno-unused-but-set-variable"
+EXTRA_OECONF_append_toolchain-clang = " --enable-more-warnings=no"
FILES_${PN} += " \
${datadir}/icons \
--
2.17.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [meta-oe][PATCH] modemmanager: Use a simpler workaround for the clang build
2019-08-06 15:39 [meta-oe][PATCH] modemmanager: Use a simpler workaround for the clang build Adrian Bunk
@ 2019-08-06 18:09 ` Khem Raj
2019-08-06 18:37 ` Adrian Bunk
0 siblings, 1 reply; 3+ messages in thread
From: Khem Raj @ 2019-08-06 18:09 UTC (permalink / raw)
To: Adrian Bunk; +Cc: openembeded-devel
On Tue, Aug 6, 2019 at 8:39 AM Adrian Bunk <bunk@stusta.de> wrote:
>
> The check whether the compiler understands an option is broken
> in m4/compiler_warnings.m4, but if we are just workarounding it
> then do a minimal workaround without patching.
>
> Signed-off-by: Adrian Bunk <bunk@stusta.de>
> ---
> ...-not-set-Wno-unused-but-set-variable.patch | 30 -------------------
> .../modemmanager/modemmanager_1.10.0.bb | 3 +-
> 2 files changed, 1 insertion(+), 32 deletions(-)
> delete mode 100644 meta-oe/recipes-connectivity/modemmanager/modemmanager/0001-Do-not-set-Wno-unused-but-set-variable.patch
>
> diff --git a/meta-oe/recipes-connectivity/modemmanager/modemmanager/0001-Do-not-set-Wno-unused-but-set-variable.patch b/meta-oe/recipes-connectivity/modemmanager/modemmanager/0001-Do-not-set-Wno-unused-but-set-variable.patch
> deleted file mode 100644
> index dcbe01d5e..000000000
> --- a/meta-oe/recipes-connectivity/modemmanager/modemmanager/0001-Do-not-set-Wno-unused-but-set-variable.patch
> +++ /dev/null
> @@ -1,30 +0,0 @@
> -From d4f73b037d11d20f080e74c05df28853fb833075 Mon Sep 17 00:00:00 2001
> -From: Khem Raj <raj.khem@gmail.com>
> -Date: Fri, 7 Dec 2018 22:59:47 -0800
> -Subject: [PATCH] Do not set Wno-unused-but-set-variable
> -
> -This is gcc specific warning, not implemented by pretending to be gcc
> -like compilers e.g. clang.
> -
> -We will pass this option from recipe
> -
> -Upstream-Status: Inappropriate [OE-Specific]
> -
> -Signed-off-by: Khem Raj <raj.khem@gmail.com>
> ----
> - m4/compiler_warnings.m4 | 2 +-
> - 1 file changed, 1 insertion(+), 1 deletion(-)
> -
> -diff --git a/m4/compiler_warnings.m4 b/m4/compiler_warnings.m4
> -index f498a28..e65e97f 100644
> ---- a/m4/compiler_warnings.m4
> -+++ b/m4/compiler_warnings.m4
> -@@ -11,7 +11,7 @@ if test "$GCC" = "yes" -a "$set_more_warnings" != "no"; then
> - -Wdeclaration-after-statement -Wstrict-prototypes \
> - -Wno-unused-parameter -Wno-sign-compare \
> - -Wno-deprecated-declarations \
> -- -Wno-unused-but-set-variable -Wformat-security; do
> -+ -Wno-unused -Wformat-security; do
> - SAVE_CFLAGS="$CFLAGS"
> - CFLAGS="$CFLAGS $option"
> - AC_MSG_CHECKING([whether gcc understands $option])
> diff --git a/meta-oe/recipes-connectivity/modemmanager/modemmanager_1.10.0.bb b/meta-oe/recipes-connectivity/modemmanager/modemmanager_1.10.0.bb
> index 1ad19e8f7..ef5af3a57 100644
> --- a/meta-oe/recipes-connectivity/modemmanager/modemmanager_1.10.0.bb
> +++ b/meta-oe/recipes-connectivity/modemmanager/modemmanager_1.10.0.bb
> @@ -12,7 +12,6 @@ inherit gnomebase gettext systemd vala gobject-introspection bash-completion
> DEPENDS = "glib-2.0 libgudev intltool-native libxslt-native"
>
> SRC_URI = "http://www.freedesktop.org/software/ModemManager/ModemManager-${PV}.tar.xz \
> - file://0001-Do-not-set-Wno-unused-but-set-variable.patch \
> "
> SRC_URI[md5sum] = "92d25176d0cc9d588ca29700b33c7d98"
> SRC_URI[sha256sum] = "fd0f39996025dac96995daea8a58ec7dd571582f7563a8ae0d5f65b571b76ee2"
> @@ -33,7 +32,7 @@ PACKAGECONFIG[qmi] = "--with-qmi,--without-qmi,libqmi"
> EXTRA_OECONF = " \
> --with-udev-base-dir=${nonarch_base_libdir}/udev \
> "
> -CFLAGS_append_toolchain-gcc = " -Wno-unused-but-set-variable"
> +EXTRA_OECONF_append_toolchain-clang = " --enable-more-warnings=no"
>
I think this will disable extra warnings but since the patch was
anyway doing same although just for one warning
this is fine by me. I still would like to have an option to be able to
build with default warning set and have easy way for OE devs to
fix/report such issues to package upstream instead of masking them
out.
> FILES_${PN} += " \
> ${datadir}/icons \
> --
> 2.17.1
>
> --
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-devel
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [meta-oe][PATCH] modemmanager: Use a simpler workaround for the clang build
2019-08-06 18:09 ` Khem Raj
@ 2019-08-06 18:37 ` Adrian Bunk
0 siblings, 0 replies; 3+ messages in thread
From: Adrian Bunk @ 2019-08-06 18:37 UTC (permalink / raw)
To: Khem Raj; +Cc: openembeded-devel
On Tue, Aug 06, 2019 at 11:09:50AM -0700, Khem Raj wrote:
> On Tue, Aug 6, 2019 at 8:39 AM Adrian Bunk <bunk@stusta.de> wrote:
>...
> > @@ -33,7 +32,7 @@ PACKAGECONFIG[qmi] = "--with-qmi,--without-qmi,libqmi"
> > EXTRA_OECONF = " \
> > --with-udev-base-dir=${nonarch_base_libdir}/udev \
> > "
> > -CFLAGS_append_toolchain-gcc = " -Wno-unused-but-set-variable"
> > +EXTRA_OECONF_append_toolchain-clang = " --enable-more-warnings=no"
> >
>
> I think this will disable extra warnings but since the patch was
> anyway doing same although just for one warning
> this is fine by me.
And it's only for clang.
> I still would like to have an option to be able to
> build with default warning set and have easy way for OE devs to
> fix/report such issues to package upstream instead of masking them
> out.
If someone wants to fix or report warnings upstream, then a stable
release that might be years behind upstream master is usually not
a good starting point - when you are upstream in OE you are also
not very enthusiastic when people send patches for Yocto 2.0 today.
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-08-06 18:37 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-08-06 15:39 [meta-oe][PATCH] modemmanager: Use a simpler workaround for the clang build Adrian Bunk
2019-08-06 18:09 ` Khem Raj
2019-08-06 18:37 ` Adrian Bunk
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.