{"action":"created","issue":{"url":"https://api.github.com/repos/nodejs/node/issues/5978","repository_url":"https://api.github.com/repos/nodejs/node","labels_url":"https://api.github.com/repos/nodejs/node/issues/5978/labels{/name}","comments_url":"https://api.github.com/repos/nodejs/node/issues/5978/comments","events_url":"https://api.github.com/repos/nodejs/node/issues/5978/events","html_url":"https://github.com/nodejs/node/pull/5978","id":144972358,"number":5978,"title":"Fix make tar-headers for Linux","user":{"login":"gibm","id":15943089,"avatar_url":"https://avatars.githubusercontent.com/u/15943089?v=3","gravatar_id":"","url":"https://api.github.com/users/gibm","html_url":"https://github.com/gibm","followers_url":"https://api.github.com/users/gibm/followers","following_url":"https://api.github.com/users/gibm/following{/other_user}","gists_url":"https://api.github.com/users/gibm/gists{/gist_id}","starred_url":"https://api.github.com/users/gibm/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/gibm/subscriptions","organizations_url":"https://api.github.com/users/gibm/orgs","repos_url":"https://api.github.com/users/gibm/repos","events_url":"https://api.github.com/users/gibm/events{/privacy}","received_events_url":"https://api.github.com/users/gibm/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/nodejs/node/labels/build","name":"build","color":"eb6420"}],"state":"open","locked":false,"assignee":{"login":"mhdawson","id":9373002,"avatar_url":"https://avatars.githubusercontent.com/u/9373002?v=3","gravatar_id":"","url":"https://api.github.com/users/mhdawson","html_url":"https://github.com/mhdawson","followers_url":"https://api.github.com/users/mhdawson/followers","following_url":"https://api.github.com/users/mhdawson/following{/other_user}","gists_url":"https://api.github.com/users/mhdawson/gists{/gist_id}","starred_url":"https://api.github.com/users/mhdawson/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mhdawson/subscriptions","organizations_url":"https://api.github.com/users/mhdawson/orgs","repos_url":"https://api.github.com/users/mhdawson/repos","events_url":"https://api.github.com/users/mhdawson/events{/privacy}","received_events_url":"https://api.github.com/users/mhdawson/received_events","type":"User","site_admin":false},"milestone":null,"comments":15,"created_at":"2016-03-31T17:52:52Z","updated_at":"2016-04-19T15:52:16Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/nodejs/node/pulls/5978","html_url":"https://github.com/nodejs/node/pull/5978","diff_url":"https://github.com/nodejs/node/pull/5978.diff","patch_url":"https://github.com/nodejs/node/pull/5978.patch"},"body":"### Pull Request check-list\r\n- [x] Does `make -j8 test` (UNIX) or `vcbuild test nosign` (Windows) pass with\r\n  this change (including linting)?\r\n- [x] Is the commit message formatted according to [CONTRIBUTING.md][0]?\r\n- [x] ~~If this change fixes a bug (or a performance problem), is a regression\r\n  test (or a benchmark) included?~~\r\n- [x] ~~Is a documentation update included (if this change modifies\r\n  existing APIs, or introduces new ones)?~~\r\n\r\n### Affected core subsystem(s)\r\nbuild\r\n[0]: https://github.com/nodejs/node/blob/master/CONTRIBUTING.md#step-3-commit\r\n### Description of change\r\nThe make tar-headers target currently fails on Linux due to the command `find $(TARNAME)/ -type l | xargs rm` as the find is unable to find any links, so the rm fails with a no arguments error. The simplest solution is to simply remove this line (assuming we aren't expecting an links). If anyone thinks they can still appear, then I can do something like:\r\n```shell\r\nif [ `find $(TARNAME)/ -type l` ]; then\r\n  find $(TARNAME)/ -type l | xargs rm\r\nfi\r\n```\r\n\r\nI also removed the dependency on config.gypi, as the tar-headers target runs configure itself.\r\n\r\n\r\nRelated Issue: #4149  (see [this comment](https://github.com/nodejs/node/pull/4149#issuecomment-161875369))"},"comment":{"url":"https://api.github.com/repos/nodejs/node/issues/comments/211990419","html_url":"https://github.com/nodejs/node/pull/5978#issuecomment-211990419","issue_url":"https://api.github.com/repos/nodejs/node/issues/5978","id":211990419,"user":{"login":"bnoordhuis","id":275871,"avatar_url":"https://avatars.githubusercontent.com/u/275871?v=3","gravatar_id":"","url":"https://api.github.com/users/bnoordhuis","html_url":"https://github.com/bnoordhuis","followers_url":"https://api.github.com/users/bnoordhuis/followers","following_url":"https://api.github.com/users/bnoordhuis/following{/other_user}","gists_url":"https://api.github.com/users/bnoordhuis/gists{/gist_id}","starred_url":"https://api.github.com/users/bnoordhuis/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bnoordhuis/subscriptions","organizations_url":"https://api.github.com/users/bnoordhuis/orgs","repos_url":"https://api.github.com/users/bnoordhuis/repos","events_url":"https://api.github.com/users/bnoordhuis/events{/privacy}","received_events_url":"https://api.github.com/users/bnoordhuis/received_events","type":"User","site_admin":false},"created_at":"2016-04-19T15:52:16Z","updated_at":"2016-04-19T15:52:16Z","body":"LGTM.  I'll let @mhdawson land it. =)"}}