No git binary found in $PATH

[gbluv]

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.

[gbluv]

can repository type be changed?

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

[MrRio]

Hey. Thanks for the ticket. What to?

On Wed, 23 Aug 2017 at 22:14, gbluv ***@***.***> wrote: can repository type be changed? "repository": { "type": "git", "url": "https://github.com/MrRio/jsPDF.git" }, — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#1389 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAC5s3J0KVLLCyrmmw-dproRzjIYTKfbks5sbJYwgaJpZM4O-0p2> .

-- James Hall Director Parallax Agency Ltd M. 07894950320

[gbluv]

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]

Ah ok yeah. Maybe we could bring these deps in as npm modules now.

On Wed, 23 Aug 2017 at 23:07 gbluv ***@***.***> wrote: 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: ? — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#1389 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAC5s9U8fcLP1CNWixcn8Xm7eG6biRcpks5sbKKYgaJpZM4O-0p2> .

-- James Hall Director Parallax Agency Ltd M. 07894950320

[gbluv]

+1

[gbluv]

Hi James, just curious, any eta on that?

[MrRio]

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

[MrRio]

Fixed in #1432

[gbluv]

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

[hackerrdave]

@gbluv yea the change hasn't been published yet

[MrRio]

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

[gbluv]

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]

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