Hi Graham,
> > that linker option is needed to restrict export of symbols. Can you try
> > again with the latest ConnMan git version. The new build system might
> > have fixed this since we include less weird libraries now.
>
> The same issue exists with 0.40.
that is too bad. So it is the final linking process that causes the
problem here.
> > Also maybe compiling with --enable-*=builtin might help to avoid this on
> > mipsel. Otherwise I am out of ideas and it could be a real binutils
> > issue.
>
> The --enable-foo configure options seem to have been removed. Am I
> missing something?
No idea what you are talking about. Everything is still there.
> I concur, probably a binutils issue. Most likely related to the mplt
> stuff that went in during 2008.
You might wanna file a bug report against binutils since there is
nothing much I can do from my side here.
> PS: I get this when building the bluetooth plugin:
>
> | mipsel-angstrom-linux-libtool: compile: mipsel-angstrom-linux-gcc
> -DHAVE_CONFIG_H -I. -I.. -I../include
> -isystem/home/grg/oe/tmp/staging/mipsel-angstrom-linux/usr/include
> -I/home/grg/oe/tmp/staging/mipsel-angstrom-linux/usr/include/glib-2.0
> -I/home/grg/oe/tmp/staging/mipsel-angstrom-linux/usr/lib/glib-2.0/include
> -I/home/grg/oe/tmp/staging/mipsel-angstrom-linux/usr/include/dbus-1.0
> -I/home/grg/oe/tmp/staging/mipsel-angstrom-linux/usr/lib/dbus-1.0/include
> -I../gdbus -isystem/home/grg/oe/tmp/staging/mipsel-angstrom-linux/usr/include
> -fexpensive-optimizations -frename-registers -fomit-frame-pointer -O2 -ggdb3
> -MT bluetooth.lo -MD -MP -MF .deps/bluetooth.Tpo -c bluetooth.c -fPIC -DPIC
> -o .libs/bluetooth.o
> | bluetooth.c:133: error: 'signal' redeclared as different kind of symbol
> | /home/grg/oe/tmp/staging/mipsel-angstrom-linux/usr/include/signal.h:92:
> note: previous declaration of 'signal' was here
> | make[2]: *** [bluetooth.lo] Error 1
> | make[2]: Leaving directory
> `/home/grg/oe/tmp/work/mipsel-angstrom-linux/connman-0.40+gitr103706c4f4da0c89f8b6f33cccdc3149f4ecc09a-r0/git/plugins'
>
> Needs the following change:
>
> --- connman/plugins/bluetooth.c.orig 2009-09-01 13:10:47.000000000 +0930
> +++ connman/plugins/bluetooth.c 2009-09-01 13:10:52.000000000 +0930
> @@ -130,7 +130,7 @@
> };
>
> static DBusConnection *connection;
> -static guint signal;
> +static guint bluez_signal;
>
> static int bluetooth_init(void)
> {
> @@ -140,7 +140,7 @@
> if (connection == NULL)
> return -EIO;
>
> - signal = g_dbus_add_signal_watch(connection, "sender=org.bluez",
> + bluez_signal = g_dbus_add_signal_watch(connection, "sender=org.bluez",
> bluetooth_signal, NULL, NULL);
>
> err = connman_driver_register(&bluetooth_driver);
> @@ -158,7 +158,7 @@
> {
> connman_driver_unregister(&bluetooth_driver);
>
> - g_dbus_remove_watch(connection, signal);
> + g_dbus_remove_watch(connection, bluez_signal);
>
> dbus_connection_unref(connection);
> }
That code is not in 0.40 at all.
Regards
Marcel
_______________________________________________
connman mailing list
[email protected]
http://lists.connman.net/listinfo/connman
