<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On 30 March 2016 at 23:03, Dirk Hohndel <span dir="ltr"><<a href="mailto:dirk@hohndel.org" target="_blank">dirk@hohndel.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="">On Wed, Mar 30, 2016 at 06:12:45PM +1100, Rick Walsh wrote:<br>
> On 15 March 2016 at 19:04, Rick Walsh <<a href="mailto:rickmwalsh@gmail.com">rickmwalsh@gmail.com</a>> wrote:<br>
><br>
> > Hi,<br>
> ><br>
> > There's no Bluetooth support in the current master.  I bisected it down to<br>
> > 39313c5 Clean up handling of various include file, but I have no idea what<br>
> > caused the problem.<br>
><br>
> Does anyone know what might be going on here?  Is the Bluetooth download<br>
> option shown for other people on the current (or recent) master?<br></span></blockquote><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="">
<br>
</span>Oops, I missed the original email or I would have investigated this long<br>
ago. The offending commit seems... harmless. There is one bug in there but<br>
that would IMHO only create compile failures, not a failure of BT to work.<br></blockquote><div><br>Yes, it looks harmless.  I tracked it down to the individual lines of the commit that broke Bluetooth for me.  I'm not sure why, but it has something to do with including libdivecomputer/version.h<br><br></div><div>With the following changes made to master (reverting a couple of lines from commit <span class="">39313c5)</span>, the option to download with bluetooth shows again.<br><br><span class="">diff --git a/core/configuredivecomputerthreads.cpp b/core/configuredivecomputerthreads.cpp<br>index b229fc8..c86c096 100644<br>--- a/core/configuredivecomputerthreads.cpp<br>+++ b/core/configuredivecomputerthreads.cpp<br>@@ -1,7 +1,6 @@<br> #include "configuredivecomputerthreads.h"<br> #include "libdivecomputer/hw.h"<br> #include "libdivecomputer.h"<br>-#include <libdivecomputer/version.h><br> <br> #define OSTC3_GAS1                     0x10<br> #define OSTC3_GAS2                     0x11<br>diff --git a/core/libdivecomputer.h b/core/libdivecomputer.h<br>index 99f1c24..f52f695 100644<br>--- a/core/libdivecomputer.h<br>+++ b/core/libdivecomputer.h<br>@@ -7,9 +7,7 @@<br> #ifdef DC_VERSION /* prevent a warning with wingdi.h */<br> #undef DC_VERSION<br> #endif<br>-#ifdef HAVE_LIBDIVECOMPUTER<br> #include <libdivecomputer/version.h><br>-#endif<br> #include <libdivecomputer/device.h><br> #include <libdivecomputer/parser.h><br></span><br><br></div><div>Now I can do the Bluetooth scan and pair, but I get a segfault every time I try to download from my DC (Petrel 2).  Here's a backtrace.<br><br>(gdb) bt<br>#0  0x0000000000737e08 in dc_device_foreach (device=0x7fffbc00b420, callback=0x69d31a <dive_cb>, userdata=0x7fffffffd1e0) at ../../src/device.c:371<br>#1  0x000000000069de35 in do_device_import (data=0x7fffffffd1e0) at /home/rick/src/subsurface/core/libdivecomputer.c:915<br>#2  0x000000000069e129 in do_libdivecomputer_import (data=0x7fffffffd1e0) at /home/rick/src/subsurface/core/libdivecomputer.c:996<br>#3  0x00000000005e1fa6 in DownloadThread::run (this=0x15a8f30) at /home/rick/src/subsurface/desktop-widgets/downloadfromdivecomputer.cpp:615<br>#4  0x00007ffff03513de in QThreadPrivate::start (arg=0x15a8f30) at thread/qthread_unix.cpp:331<br>#5  0x00007ffff67fe60a in start_thread (arg=0x7fffc37fe700) at pthread_create.c:334<br>#6  0x00007fffef82aa4d in clone () from /lib64/libc.so.6<br><br></div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
I actually brought a BT dive computer with me on this trip but haven't<br>
prioritized downloading from it, yet. I'll try that today and see if I can<br>
figure out what's going on.<br>
<br></blockquote><div>Please do try - I am completely lost as to what is going on.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
With stuff like this it really helps if you file a trac ticket, definitely<br>
if it doesn't get addressed right away.<br>
<br></blockquote><div>I'll do that.<br><br></div><div>Cheers,<br><br></div><div>Rick<br></div><div> <br></div></div></div></div>