{"action":"created","issue":{"url":"https://api.github.com/repos/tapjs/spawn-wrap/issues/14","repository_url":"https://api.github.com/repos/tapjs/spawn-wrap","labels_url":"https://api.github.com/repos/tapjs/spawn-wrap/issues/14/labels{/name}","comments_url":"https://api.github.com/repos/tapjs/spawn-wrap/issues/14/comments","events_url":"https://api.github.com/repos/tapjs/spawn-wrap/issues/14/events","html_url":"https://github.com/tapjs/spawn-wrap/pull/14","id":145465979,"number":14,"title":"fixes shebang execution on Node 5.8.x","user":{"login":"bcoe","id":194609,"avatar_url":"https://avatars.githubusercontent.com/u/194609?v=3","gravatar_id":"","url":"https://api.github.com/users/bcoe","html_url":"https://github.com/bcoe","followers_url":"https://api.github.com/users/bcoe/followers","following_url":"https://api.github.com/users/bcoe/following{/other_user}","gists_url":"https://api.github.com/users/bcoe/gists{/gist_id}","starred_url":"https://api.github.com/users/bcoe/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bcoe/subscriptions","organizations_url":"https://api.github.com/users/bcoe/orgs","repos_url":"https://api.github.com/users/bcoe/repos","events_url":"https://api.github.com/users/bcoe/events{/privacy}","received_events_url":"https://api.github.com/users/bcoe/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":5,"created_at":"2016-04-03T07:49:52Z","updated_at":"2016-04-03T18:31:51Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/tapjs/spawn-wrap/pulls/14","html_url":"https://github.com/tapjs/spawn-wrap/pull/14","diff_url":"https://github.com/tapjs/spawn-wrap/pull/14.diff","patch_url":"https://github.com/tapjs/spawn-wrap/pull/14.patch"},"body":"working from:\r\n\r\nhttps://github.com/tapjs/spawn-wrap/pull/12\r\n\r\nfixes shebang handling on Node 5.8.x\r\n\r\nlooks to be almost identical to:\r\n\r\nhttps://github.com/tapjs/spawn-wrap/pull/13/files\r\n\r\nso I'll let @isaacs decide what we land."},"comment":{"url":"https://api.github.com/repos/tapjs/spawn-wrap/issues/comments/205026154","html_url":"https://github.com/tapjs/spawn-wrap/pull/14#issuecomment-205026154","issue_url":"https://api.github.com/repos/tapjs/spawn-wrap/issues/14","id":205026154,"user":{"login":"addaleax","id":899444,"avatar_url":"https://avatars.githubusercontent.com/u/899444?v=3","gravatar_id":"","url":"https://api.github.com/users/addaleax","html_url":"https://github.com/addaleax","followers_url":"https://api.github.com/users/addaleax/followers","following_url":"https://api.github.com/users/addaleax/following{/other_user}","gists_url":"https://api.github.com/users/addaleax/gists{/gist_id}","starred_url":"https://api.github.com/users/addaleax/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/addaleax/subscriptions","organizations_url":"https://api.github.com/users/addaleax/orgs","repos_url":"https://api.github.com/users/addaleax/repos","events_url":"https://api.github.com/users/addaleax/events{/privacy}","received_events_url":"https://api.github.com/users/addaleax/received_events","type":"User","site_admin":false},"created_at":"2016-04-03T18:31:50Z","updated_at":"2016-04-03T18:31:50Z","body":"Um, I just saw these files from the npm-cli repo: [bin/npm](https://github.com/npm/npm/blob/master/bin/npm) [bin/npm.cmd](https://github.com/npm/npm/blob/master/bin/npm.cmd). I don’t really know what’s up with them, but I imagine there are cases where these are used when `npm` is invoked from the command line?\r\nIf so, I’m tempted to change my opinion again – possibly even back to just checking for “npm” on the command line, no shebang magic stuff – because both of them carry out the `` `dirname \"$0\"`/node ``-approach in different, even less “detectable” ways."}}