{"action":"created","issue":{"url":"https://api.github.com/repos/npm/npm/issues/12366","repository_url":"https://api.github.com/repos/npm/npm","labels_url":"https://api.github.com/repos/npm/npm/issues/12366/labels{/name}","comments_url":"https://api.github.com/repos/npm/npm/issues/12366/comments","events_url":"https://api.github.com/repos/npm/npm/issues/12366/events","html_url":"https://github.com/npm/npm/pull/12366","id":148686265,"number":12366,"title":"Patch for issue 7357, courtesy of @mathieucarbou","user":{"login":"bsclifton","id":4733304,"avatar_url":"https://avatars.githubusercontent.com/u/4733304?v=3","gravatar_id":"","url":"https://api.github.com/users/bsclifton","html_url":"https://github.com/bsclifton","followers_url":"https://api.github.com/users/bsclifton/followers","following_url":"https://api.github.com/users/bsclifton/following{/other_user}","gists_url":"https://api.github.com/users/bsclifton/gists{/gist_id}","starred_url":"https://api.github.com/users/bsclifton/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bsclifton/subscriptions","organizations_url":"https://api.github.com/users/bsclifton/orgs","repos_url":"https://api.github.com/users/bsclifton/repos","events_url":"https://api.github.com/users/bsclifton/events{/privacy}","received_events_url":"https://api.github.com/users/bsclifton/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/npm/npm/labels/git","name":"git","color":"DA655E"},{"url":"https://api.github.com/repos/npm/npm/labels/supportability","name":"supportability","color":"A675FE"}],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":28,"created_at":"2016-04-15T15:10:51Z","updated_at":"2016-08-18T14:58:28Z","closed_at":"2016-04-15T15:57:20Z","pull_request":{"url":"https://api.github.com/repos/npm/npm/pulls/12366","html_url":"https://github.com/npm/npm/pull/12366","diff_url":"https://github.com/npm/npm/pull/12366.diff","patch_url":"https://github.com/npm/npm/pull/12366.patch"},"body":"See https://github.com/npm/npm/issues/7357#issuecomment-101039135 for the patch by @mathieucarbou. This PR is me manually applying the patch, which has slight changes.\r\n\r\nI confirmed it works great by testing locally (before and after). Before `npm install` would fail, after it works great (see other comments in the issue for reports of +1s)"},"comment":{"url":"https://api.github.com/repos/npm/npm/issues/comments/240750074","html_url":"https://github.com/npm/npm/pull/12366#issuecomment-240750074","issue_url":"https://api.github.com/repos/npm/npm/issues/12366","id":240750074,"user":{"login":"bsclifton","id":4733304,"avatar_url":"https://avatars.githubusercontent.com/u/4733304?v=3","gravatar_id":"","url":"https://api.github.com/users/bsclifton","html_url":"https://github.com/bsclifton","followers_url":"https://api.github.com/users/bsclifton/followers","following_url":"https://api.github.com/users/bsclifton/following{/other_user}","gists_url":"https://api.github.com/users/bsclifton/gists{/gist_id}","starred_url":"https://api.github.com/users/bsclifton/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bsclifton/subscriptions","organizations_url":"https://api.github.com/users/bsclifton/orgs","repos_url":"https://api.github.com/users/bsclifton/repos","events_url":"https://api.github.com/users/bsclifton/events{/privacy}","received_events_url":"https://api.github.com/users/bsclifton/received_events","type":"User","site_admin":false},"created_at":"2016-08-18T14:58:28Z","updated_at":"2016-08-18T14:58:28Z","body":"@emigenix I agree that this patch could have helped. I championed it for a while and after attending their weekly meeting, it's safe to say Cygwin will never be supported by them.\r\n\r\nIn the meantime, besides applying the patch, you can try redefining git in your bashrc:\r\n```bash\r\nfunction git {\r\n    declare -a ARGS\r\n    for n in \"$@\" ; do ARGS+=(\"$(cygpath -u -- \"${n}\")\") ; done\r\n        command git \"${ARGS[@]}\"\r\n    }\r\n}\r\n\r\nI didn't get a chance to finish it... it works as long as the paths are quoted. Shout-outs to Eliot Moss on the Cygwin mailing list for suggesting this and also giving me a rev1 to work with."}}
{"actor":{"display_login":"bsclifton"}}