-
Notifications
You must be signed in to change notification settings - Fork 25
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
install fails at osrm #54
Comments
Is it my |
@todrobbins it looks like you might be running on a much older version of Node (v7.2?) can you confirm if this is the case? The CLI currently requires node v10+ (it should do a check during install). |
@todrobbins FYI, i've just added Docker setup steps to the README: https://github.com/sharedstreets/sharedstreets-js/blob/master/README.md#docker You should be able to use locally if you can upgrade Node, or use these. |
I think you're right, I show the following for
And:
🤷🏻♂️ |
@todrobbins not sure what's happening there -- the node version does show v12.2 but it looks like it's attempting to download OSRM for node v7.2:
Not sure why that's happening but might be related to a conflict between nvm and another installed version of node? |
Strike that: looks like the v72 in the URL is the "node module version" which is Node v12.2. It appears that Mapbox hasn't released OSRM binaries for Node v12 yet. We'll update the docs to reflect this. |
I ran into the same issue (node 12 instead of 10). With nvm, I was able to install an additional node10 version:
|
I was able to successfully use the CLI after installing node11 with nvm. I had the same issue as others when using the most recent version of node (node14). I didn't attempt to use the CLI with node12. Seems sharedstreets/trip-simulator#18 is related issue. |
Not sure what's wrong on my end, but this is what I get when trying to
npm install -g sharedstreets
:The text was updated successfully, but these errors were encountered: