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

No git binary found in $PATH #1389

Closed
gbluv opened this issue Aug 22, 2017 · 14 comments
Closed

No git binary found in $PATH #1389

gbluv opened this issue Aug 22, 2017 · 14 comments

Comments

@gbluv
Copy link

gbluv commented Aug 22, 2017

When I try to install jspdf, npm complains that no git binary found. Usually npm does not require git executables to be present on the system. However, if I do put git in my path it works. It is a huge impediment during deployment because I can't guarantee that git client is installed on the server.

image

@gbluv
Copy link
Author

gbluv commented Aug 23, 2017

can repository type be changed?

"repository": {
"type": "git",
"url": "https://github.com/MrRio/jsPDF.git"
},

@MrRio
Copy link
Member

MrRio commented Aug 23, 2017 via email

@gbluv
Copy link
Author

gbluv commented Aug 23, 2017

something tells npm to start using git executable. Could it be related to dependencies?

"dependencies": {
"adler32cs": "github:chick307/adler32cs.js",
"cf-blob.js": "0.0.1",
"filesaver.js": "github:andyinabox/FileSaver.js"
},

maybe we could give it https path instead of github: ?

@MrRio
Copy link
Member

MrRio commented Aug 24, 2017 via email

@gbluv
Copy link
Author

gbluv commented Aug 24, 2017

+1

@gbluv
Copy link
Author

gbluv commented Aug 31, 2017

Hi James, just curious, any eta on that?

@MrRio
Copy link
Member

MrRio commented Sep 2, 2017

In the next week I reckon. If you could make and test a PR that would be even better! :)

@MrRio
Copy link
Member

MrRio commented Sep 11, 2017

Fixed in #1432

@MrRio MrRio closed this as completed Sep 11, 2017
@gbluv
Copy link
Author

gbluv commented Sep 12, 2017

James, I still get this error is that because release with this change was not generated yet?

@hackerrdave
Copy link
Contributor

@gbluv yea the change hasn't been published yet

@MrRio
Copy link
Member

MrRio commented Sep 18, 2017

Hey there, it should be live in v1.3.5 – can you try?

@gbluv
Copy link
Author

gbluv commented Sep 18, 2017

Thanks Guys, all is there. This was my old comment. Everything works perfectly had to do only one extra require for html2canvas otherwise all is good

@maxaqq
Copy link

maxaqq commented Nov 23, 2017

yo sigo teniendo el mismo error cuando quiero instalar el npm install nodebots-interchange

@mahdieh-dev
Copy link

solved this issue by uninstalling Node.js and then reinstalling it using installer in Node page

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

5 participants