-
Notifications
You must be signed in to change notification settings - Fork 52
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Import native mac bindings from notjosh/bleno-mac. #13
Conversation
WIP / seeking advice / feedback. Potentially fixes #10 and other issues. I ran into an issue where advertising wasn't working on macOS 10.14. Through this ticket noble#404 I found https://github.com/notjosh/bleno-mac which includes native macOS bindings for bleno, based on the noble bindings from Timeular/noble-mac that were imported into abandonware/noble in this commit: abandonware/noble@556f5fb about 18 months ago. I have tested this briefly for my use-case https://github.com/ptx2/gymnasticon on macOS 10.13 and 10.14 and it seems to work. I have not tested any of the beacon functionality or any other versions of macOS as these two are all I have available right now. I figured I'd open this to see if it's appropriate to pursue merging these bindings at all and start a discussion re: next steps. Thanks for keeping this library alive! |
I am not against merging this but I'll like that commit message to be updated why var is prefered over const ? |
Thanks for the quick reply. I don't actually prefer var ;) but const was causing the build to fail due to the .jshintrc settings. I could put it back to const and update the jshintrc to |
yes it looks to me a better option |
I was looking more closely at these bindings vs the noble ones. It looks like this will only compile on macOS 10.13+ whereas noble has some I also found some applications which don't work with these bindings. I'll try to look into resolving both of those this week. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Fix issues on macOS 10.14+ where advertising does not work. The code is from notjosh/bleno-mac and is based on Timeular/noble-mac which has already been added to abandonware/noble.