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

Merge back downstream fixes from abandonware #122

Open
wants to merge 23 commits into
base: master
Choose a base branch
from

Conversation

rzr
Copy link

@rzr rzr commented Jan 24, 2020

johnny and others added 23 commits December 14, 2018 01:56
This fix comes from
noble#84 (comment)
,  so I can't personally vouch for it.

Change-Id: Ifde2a3b996ab89a4b12b0eb155c081aca2b11dc3
Origin: https://github.com/jrobeson/node-bluetooth-hci-socket
Forwarded: noble#91
Change-Id: Ide752983f00395cf8cb59ab727a58ff0692112dd
Origin: https://github.com/jrobeson/node-bluetooth-hci-socket
Forwarded: noble#91
Change-Id: I94fdcbea6957d9f2c7eb5e28d8364363be6a3520
Forwarded: noble#102
Signed-off-by: Philippe Coval <p.coval@samsung.com>
- current Zadig advice "Delete the driver software for this device"
- Zadig tries to create a system restoral point but this usually fails on my Windows 10 (but thanks Zadig for trying)

Change-Id: I004df41220c7e12b7cf8b1f1f1a65aab701ffe32
Origin: https://github.com/noble/node-bluetooth-hci-socket/commit/1620b37d47b5168decffbf23c82d6c038a2d0e5a.patch
Forwarded: noble#54
Origin: https://github.com/abandonware/node-bluetooth-hci-socket
Change-Id: I0b1c9c2a6950fa562c0e59ebdc780960bf219a6a
Signed-off-by: Philippe Coval <p.coval@samsung.com>
Before updating revision

Change-Id: I4e19251e6ec5ea7d69b2390fe152fe337862ccb3
Forwarded: noble#105
Origin: https://github.com/abandonware/node-bluetooth-hci-socket/
Bug: noble#85
Signed-off-by: Philippe Coval <p.coval@samsung.com>
Might be caused by parallel jobs?

```
> @abandonware/bluetooth-hci-socket@0.5.3-0 install /local/home/philippe/var/cache/url/git/ssh/github.com/tizenteam/node-bluetooth-hci-socket/src/node-bluetooth-hci-socket
> node-pre-gyp install --fallback-to-build
(...)
Release/obj.target/binding/src/BluetoothHciSocket.o: file not recognized: File truncated
collect2: error: ld returned 1 exit status
(...)
```
Or

```
rm: cannot remove './Release/.deps/Release/obj.target/binding/src/BluetoothHciSocket.o.d.raw': No such file or directory
(...)
make: *** [Release/obj.target/binding/src/BluetoothHciSocket.o] Error 1
```

Origin: #4
Relate-to: noble#91
Change-Id: I34726c8adffc7c39f585f42da1f7d776c5761a17
Signed-off-by: Philippe Coval <p.coval@samsung.com>
Change-Id: I5fbe3eb7463edc95bb44558d5874ff2123e7e46c
Signed-off-by: Philippe Coval <p.coval@samsung.com>
@rzr
Copy link
Author

rzr commented Jan 26, 2021

This review is still pending for a year,

Can we discuss about maintenance strategy at:
#123

@rzr
Copy link
Author

rzr commented Oct 1, 2021

Those changes are released in downstream version:
https://www.npmjs.com/package/@abandonware/bluetooth-hci-socket/v/0.5.3-8

@ebaauw
Copy link

ebaauw commented Oct 1, 2021

Would like to see these changes include as well. Currently using the downstream fork by abandonware.

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

Successfully merging this pull request may close these issues.

7 participants