sprhawk / btstack

Automatically exported from code.google.com/p/btstack
0 stars 0 forks source link

Cygwin and MSYS2/MinGW64 #429

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
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.

Original issue reported on code.google.com by mathieu....@gmail.com on 27 Nov 2014 at 8:36

GoogleCodeExporter commented 9 years ago
The changes/fixes are summarized here:
https://github.com/matlo/btstack/compare/5d0bf6b73814f19cee3a01cf352cf579c0f3d16
7...5f6bf7913a614a29a2a9a21f4201ba854ade81bf

Original comment by mathieu....@gmail.com on 27 Nov 2014 at 8:55

GoogleCodeExporter commented 9 years ago
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. :)

Original comment by matthias.ringwald@gmail.com on 28 Nov 2014 at 7:03

GoogleCodeExporter commented 9 years ago
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.

Original comment by mathieu....@gmail.com on 28 Nov 2014 at 9:44