{"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":24,"created_at":"2016-04-15T15:10:51Z","updated_at":"2016-08-11T12:21:54Z","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/239145470","html_url":"https://github.com/npm/npm/pull/12366#issuecomment-239145470","issue_url":"https://api.github.com/repos/npm/npm/issues/12366","id":239145470,"user":{"login":"ankostis","id":501585,"avatar_url":"https://avatars.githubusercontent.com/u/501585?v=3","gravatar_id":"","url":"https://api.github.com/users/ankostis","html_url":"https://github.com/ankostis","followers_url":"https://api.github.com/users/ankostis/followers","following_url":"https://api.github.com/users/ankostis/following{/other_user}","gists_url":"https://api.github.com/users/ankostis/gists{/gist_id}","starred_url":"https://api.github.com/users/ankostis/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ankostis/subscriptions","organizations_url":"https://api.github.com/users/ankostis/orgs","repos_url":"https://api.github.com/users/ankostis/repos","events_url":"https://api.github.com/users/ankostis/events{/privacy}","received_events_url":"https://api.github.com/users/ankostis/received_events","type":"User","site_admin":false},"created_at":"2016-08-11T12:21:54Z","updated_at":"2016-08-11T12:21:54Z","body":"> Cygwin is massive\r\n\r\nIndeed!\r\nBut I can also understand the *npm* developers wanting to avoid the burden of supporting one more *git-platform*.\r\nWe have to somehow offload this duty from them.\r\n\r\n>  MS tried to fix it with PowerShell and they failed, now they are going with full Linux shell support.\r\n\r\nNot surer if this will work at all.\r\nThe new \"ubuntu\" bashis an isolated \"world\", probably you would [not be able to invoke non `apt-get` installed commands](](http://www.howtogeek.com/249966/how-to-install-and-use-the-linux-bash-shell-on-windows-10/)).\r\nThat means, **invoking standalone `npm` from \"ubuntu\" bash will not work!**\r\n\r\nWill install ubuntu-bash later today when I get back home,. and check that for my self."}}
{"actor":{"display_login":"ankostis"}}