-
Notifications
You must be signed in to change notification settings - Fork 1.3k
@mapbox/mapbox-gl-native not available (403 status code) on npm #16485
Comments
+1 |
|
Just ran into this today........ |
Isn't that a Mapbox-gl-js fork, not Mapbox-gl-native? |
@markgis hard to tell. Would be great to see a working published version of native if possible! |
Yep, looks like you're right, @markgis. Different packages, I'll update my comment above! |
This error is happening because this library does not support prebuilt binaries for node 12. It is available on Node 10, however |
I read somewhere (sorry will try and find the link) that the Mapbox team have stopped creating Node binaries all together and users will have to build themselves. Not sure. @okdistribute Is it worth creating a fork just with later Node bindings? Probably a lot of work though to keep up with releases. 🤔 Would it be possible to use Github actions to auto pull latest code form main mapbox repo and update Publish Scripts? |
That's really unfortunate to hear! I can't get things to compile on my linux machine, and there isn't much in the CONTRIBUTING.md file. I don't really have time to go down this rabbit hole right now |
@okdistribute So found the comment on another thread. I would maybe be up for helping with post node 10 builds as tmpsantos suggests, but not sure how much time I have right now. I'm sure there are enough people who want native-gl compiled builds in node 12-14 to help out though? |
Oh interesting. I don't have a ton of time to help out either, but I could do some basic things if necessary. I think we would benefit greatly from having this in native-gl in node 12. The newest versions of electron that have security upgrades and also depend upon Node 12.4, so until this is done, no electron application can use mapbox-gl native unless they use an insecure version of electron. Right now as a workaround, we are running any necessary background mapbox-gl tasks using the javascript cc @gmaclennan |
I've managed to create successful builds in Node 11 but not 12 yet. There was a V8 bump in Node 12 to I don't mind trying to create bindings and helping keep them up to date but I don't know enough about Cmake and NAN to fix it. I was hoping a nan version bump would solve it, but it looks like it's still broken. I'm not sure if the issues lies with |
Yeah, N-API is much more reliable and easier to work with, it has better asynchronous support as well IIRC. It may not a huge task to migrate from NaN to N-API, but it could take much longer if there is a snag along the way. |
For anyone interested, we forked the node binding and added support for Node 12, 13, and 14 in naturalatlas/mapbox-gl-native-node@668a290 (still using NAN). The repo is here @naturalatlas/mapbox-gl-native. It uses the EGL backend because I for the life of me couldn't get the GLX backend compiling / working on Travis.
|
@brianreavis I'm working on https://github.com/maptiler/tileserver-gl/ and it's having dependencies on this lib. I took the source code, replace with your lib and applied your 1.0.2 version number in package.json; apart from that had to replace in two other JS files also in the repo, and then did
|
Platform: macOS
Steps to trigger behavior
npm install @mapbox/mapbox-gl-native
Expected behavior
I expect that the npm module is installed correctly.
Actual behavior
The tarball file can't be downloaded:
The text was updated successfully, but these errors were encountered: