Skip to content
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

Node 10 support on linux #23

Closed
hannseman opened this issue Jan 26, 2019 · 5 comments
Closed

Node 10 support on linux #23

hannseman opened this issue Jan 26, 2019 · 5 comments

Comments

@hannseman
Copy link

I understand that the main purpose of this package is to bring support for macOS Mojave but I would like to ask you if you could consider depending on https://github.com/abandonware/noble which fixes Node 10 support for linux. This is the root issue noble/node-bluetooth-hci-socket#84.

This would enable people who have to support both macOS Mojave and Node >=10 on Linux to use this library.

@geovie
Copy link
Collaborator

geovie commented Feb 4, 2019

Hi, for now I won't change from noble/noble to abandonware/noble as it is already possible to use a custom noble (abandonware/noble or any other) by using noble-mac as binding only (take a look at index.js).
However, I'd like to integrate noble-mac directly in to noble, so if abandonware/noble is actively maintained and they're interested in Mojave support, I'm happy to create a PR.

@hannseman
Copy link
Author

@geovie looks like abandonware/noble is interested in integrating. Please see abandonware/noble#4.

That fork is pretty popular with 1600 weekly downloads.
https://www.npmjs.com/package/@abandonware/noble

Compare that to the original package with has 1800 weekly downloads.
https://www.npmjs.com/package/noble

@rzr
Copy link

rzr commented Aug 6, 2019

Are there any patches from this fork that are pending upstream ?

If yes they could be merged into @abandonware/noble

Again efforts to track changes are appreciated:
abandonware/noble#2

@geovie
Copy link
Collaborator

geovie commented Aug 6, 2019

@rzr No @abandonware/noble already contains the latest changes abandonware/noble#8

@rzr
Copy link

rzr commented Aug 6, 2019

Feel free to rebase on @abandonware/noble if you want, I plan to keep it based on upstream too

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants