Commit 39313c5 broke Bluetooth support (Clean up handling of various include file)

Miika Turkia miika.turkia at gmail.com
Wed Mar 30 05:56:56 PDT 2016



> On 30 Mar 2016, at 20:03, Dirk Hohndel <dirk at hohndel.org> wrote:
> 
>> On Wed, Mar 30, 2016 at 06:12:45PM +1100, Rick Walsh wrote:
>>> On 15 March 2016 at 19:04, Rick Walsh <rickmwalsh at gmail.com> wrote:
>>> 
>>> Hi,
>>> 
>>> There's no Bluetooth support in the current master.  I bisected it down to
>>> 39313c5 Clean up handling of various include file, but I have no idea what
>>> caused the problem.
>> 
>> Does anyone know what might be going on here?  Is the Bluetooth download
>> option shown for other people on the current (or recent) master?
> 
> Oops, I missed the original email or I would have investigated this long
> ago. The offending commit seems... harmless. There is one bug in there but
> that would IMHO only create compile failures, not a failure of BT to work.
> 
> I actually brought a BT dive computer with me on this trip but haven't
> prioritized downloading from it, yet. I'll try that today and see if I can
> figure out what's going on.

FWIW I have mostly failed to dl over bt with the latest released version. However, the dl does succeed occasionally. I have also had a couple of crashes related to bt dl attempts. I'll try to take a closer look, at least to grab backtrace, in a few days...

miika


More information about the subsurface mailing list