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

[WIP] Fix build on node 10.x #91

Closed
wants to merge 2 commits into from
Closed

[WIP] Fix build on node 10.x #91

wants to merge 2 commits into from

Conversation

ghost
Copy link

@ghost ghost commented Jun 18, 2018

This fix comes from
#84 (comment)
, so I can't personally vouch for it.

This fix comes from
#84 (comment)
,  so I can't personally vouch for it.
@yourealwaysbe
Copy link

Thanks for this. To get it to work i had to edit the package.json path lib/notify.js to start with ../

@ghost ghost changed the title Fix build on node 10.x [WIP] Fix build on node 10.x Jun 21, 2018
@ghost
Copy link
Author

ghost commented Jun 22, 2018

lib/notify where?

@yourealwaysbe
Copy link

yourealwaysbe commented Jun 22, 2018

Oops, sorry. I meant lib/native.js.

Took this from a commit in kounch's repo.

@ghost
Copy link
Author

ghost commented Jun 23, 2018

I'll add that, but I'm also working on a massive refactoring of the repo.

@ghost
Copy link
Author

ghost commented Jun 23, 2018

done

@matthewhague
Copy link

Thanks!

@notabene00
Copy link

Неплохо бы уже заиметь поддержку десятой ноды, да.

@rzr
Copy link

rzr commented Jul 6, 2018

It worked for me, but I had to disable parallel building:

MAKEFLAGS=-j1 npm install https://github.com/jrobeson/node-bluetooth-hci-socket/#fix-builds-for-node-10

Probably not related to that patch.

@rzr
Copy link

rzr commented Jul 11, 2018

This is affecting "usb" module too, workaround is:

cd node_modules/usb/ &&  MAKEFLAGS=-j1 npm install https://github.com/jrobeson/node-bluetooth-hci-socket/#fix-builds-for-node-10

This patch would be welcome in next release:
#85

@Apollon77
Copy link

Are there any news? Last is one month ago ... when it is planned to be available?

@rzr
Copy link

rzr commented Aug 15, 2018

meanwhile this can be forked into npm 'org namespace do you recommend any ?

@petersaints
Copy link

Node 10 has now reached LTS status, is there any official fix coming?

@rzr
Copy link

rzr commented Nov 8, 2018

Is upstream ready to merge this change for supporting current node LTS?
I am a bit concerned because last change from master was in april.

Let's try to push a simple change to see how active is project:
#102

Relate-to: #84

@ghost
Copy link
Author

ghost commented Nov 8, 2018

I think sandeep is just too busy to properly maintain noble. I'm using my own typescriptified fork of noble instead.

@rzr
Copy link

rzr commented Nov 8, 2018

So single maintenance model can be transferred to co-maintenance ?

Any opinion on @sandeepmistry on this suggestion?

@AlCalzone
Copy link

Please merge :(

rzr pushed a commit to abandonware/node-bluetooth-hci-socket that referenced this pull request Dec 14, 2018
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
rzr pushed a commit to abandonware/node-bluetooth-hci-socket that referenced this pull request Dec 14, 2018
Change-Id: Ide752983f00395cf8cb59ab727a58ff0692112dd
Origin: https://github.com/jrobeson/node-bluetooth-hci-socket
Forwarded: noble#91
rzr added a commit to abandonware/node-bluetooth-hci-socket that referenced this pull request Dec 19, 2018
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: https://github.com/abandonware/node-bluetooth-hci-socket/pulls
Relate-to: noble#91
Change-Id: I34726c8adffc7c39f585f42da1f7d776c5761a17
Signed-off-by: Philippe Coval <p.coval@samsung.com>
rzr added a commit to abandonware/node-bluetooth-hci-socket that referenced this pull request Dec 19, 2018
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>
@rzr
Copy link

rzr commented Dec 19, 2018

Released in

https://www.npmjs.com/package/@abandonware/bluetooth-hci-socket/v/0.5.3-0

rzr added a commit to abandonware/node-bluetooth-hci-socket that referenced this pull request Jan 24, 2020
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>
@rzr
Copy link

rzr commented Jan 6, 2021

Feel free to comment this related ticket:

abandonware#9

@ghost ghost closed this by deleting the head repository Nov 11, 2023
This pull request was closed.
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