From: Carlos Santos <casantos@datacom.ind.br>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH v2] modem-manager: remove -Werror from CFLAGS
Date: Mon, 21 May 2018 18:17:18 -0300 [thread overview]
Message-ID: <20180521211718.5465-1-casantos@datacom.ind.br> (raw)
In-Reply-To: <CAGm1_kuEqW+Mito5P8rMxdM+zrCFAxpu2TzsZa=mvDAK8d1e1w@mail.gmail.com>
We are aproaching the 2018.05 release and modem-manager is still broken
due to warnings like this:
mm-base-manager.c: In function 'handle_set_logging':
mm-base-manager.c:680:15: warning: assignment from incompatible pointer type [-Wincompatible-pointer-types]
ctx->self = g_object_ref (manager);
^
There is a macro defined in build/libglib2-2.56.1/gobject/gobject.h
which leads to the assignment errors:
511 /* Make reference APIs type safe with macros */
512 #define g_object_ref(Obj) ((__typeof__(Obj)) (g_object_ref) (Obj))
The problem can be easily reproduced with this sample code:
$ cat test.c
extern void *g_object_ref(void *);
#define g_object_ref(Obj) ((__typeof__(Obj)) (g_object_ref) (Obj))
extern double *new_double(int *);
double *new_double(int *ip) {
double *dp;
dp = g_object_ref(ip);
return dp;
}
$ gcc -Wall -Werror -c /tmp/test.c -o /tmp/test.o
/tmp/test.c: In function ?new_context?:
/tmp/test.c:19:13: error: assignment from incompatible pointer type [-Werror=incompatible-pointer-types]
ctx->self = g_object_ref(om);
^
cc1: all warnings being treated as errors
Fixing the code would require either changing the 120 ofending calls to
g_object_ref or a refactoring to make all types assignment compatible.
Let's take a simpler approach, disabling extra warnings, and wait for
the next release of ModemManager.
Signed-off-by: Carlos Santos <casantos@datacom.ind.br>
---
package/modem-manager/modem-manager.mk | 1 +
1 file changed, 1 insertion(+)
diff --git a/package/modem-manager/modem-manager.mk b/package/modem-manager/modem-manager.mk
index 100c4a2941..fe4a7a95da 100644
--- a/package/modem-manager/modem-manager.mk
+++ b/package/modem-manager/modem-manager.mk
@@ -11,6 +11,7 @@ MODEM_MANAGER_LICENSE = GPL-2.0+ (programs, plugins), LGPL-2.0+ (libmm-glib)
MODEM_MANAGER_LICENSE_FILES = COPYING COPYING.LIB
MODEM_MANAGER_DEPENDENCIES = host-pkgconf host-intltool libglib2 libgudev
MODEM_MANAGER_INSTALL_STAGING = YES
+MODEM_MANAGER_CONF_OPTS = --disable-more-warnings
ifeq ($(BR2_PACKAGE_MODEM_MANAGER_LIBQMI),y)
MODEM_MANAGER_DEPENDENCIES += libqmi
--
2.17.0
prev parent reply other threads:[~2018-05-21 21:17 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-04-18 12:32 [Buildroot] modem-manager: build issue Yegor Yefremov
2018-04-18 13:30 ` Thomas Petazzoni
2018-04-18 13:36 ` Yegor Yefremov
2018-04-18 13:51 ` Thomas Petazzoni
2018-04-18 14:33 ` Carlos Santos
2018-04-20 3:23 ` Carlos Santos
2018-04-20 6:38 ` Thomas Petazzoni
2018-04-20 7:28 ` Yegor Yefremov
2018-05-21 11:45 ` [Buildroot] [PATCH] modem-manager: remove -Werror from CFLAGS Carlos Santos
2018-05-21 15:54 ` Peter Korsgaard
2018-05-21 21:13 ` Carlos Santos
2018-05-21 21:34 ` Peter Korsgaard
2018-05-21 21:17 ` Carlos Santos [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20180521211718.5465-1-casantos@datacom.ind.br \
--to=casantos@datacom.ind.br \
--cc=buildroot@busybox.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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.