Export to GitHub

btstack - issue #429

Cygwin and MSYS2/MinGW64


Posted on Nov 27, 2014 by Happy Panda

Hello,

First of all, thanks for your work on btstack!

I was able to build and run btstack with both Cygwin and MSYS2/MinGW64. A few changes/fixes were needed. If you want to merge them, you can read the commits of the following git repository: https://github.com/matlo/btstack/commits/master

While reading the source code I saw some system calls with unchecked return values. This means some errors probably have unpredictable results. I can try to fix a few of these if you want.

Comment #1

Posted on Nov 27, 2014 by Happy Panda

The changes/fixes are summarized here: https://github.com/matlo/btstack/compare/5d0bf6b73814f19cee3a01cf352cf579c0f3d167...5f6bf7913a614a29a2a9a21f4201ba854ade81bf

Comment #2

Posted on Nov 28, 2014 by Swift Ox

Hi Mathieu. Thanks for trying the fresh Win32 port and even improving it. I don't know much about Windows and was just happy when it worked the first time under msys32. I'll apply/merge the changes, but won't be able to do it right away. .. I might ask about individual patches if I don't understand why. :)

Comment #3

Posted on Nov 28, 2014 by Happy Panda

I realized the license of btstack does not meet my needs: I wanted to use it with GPL'd source code, but the non-commercial clause of btstack does not allow it.

As I only need l2cap in Windows I will fork btstack at revision r1665 and try to make it work.

Status: Started

Labels:
Type-Defect Priority-Medium