{"action":"created","issue":{"url":"https://api.github.com/repos/docker/docker/issues/11748","labels_url":"https://api.github.com/repos/docker/docker/issues/11748/labels{/name}","comments_url":"https://api.github.com/repos/docker/docker/issues/11748/comments","events_url":"https://api.github.com/repos/docker/docker/issues/11748/events","html_url":"https://github.com/docker/docker/pull/11748","id":64297816,"number":11748,"title":"removed unnecessary tabs from /usr/sbin/policy-rc.d, and /sbin/initctl for mkimage.sh debootstrap","user":{"login":"minimum2scp","id":162862,"avatar_url":"https://avatars.githubusercontent.com/u/162862?v=3","gravatar_id":"","url":"https://api.github.com/users/minimum2scp","html_url":"https://github.com/minimum2scp","followers_url":"https://api.github.com/users/minimum2scp/followers","following_url":"https://api.github.com/users/minimum2scp/following{/other_user}","gists_url":"https://api.github.com/users/minimum2scp/gists{/gist_id}","starred_url":"https://api.github.com/users/minimum2scp/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/minimum2scp/subscriptions","organizations_url":"https://api.github.com/users/minimum2scp/orgs","repos_url":"https://api.github.com/users/minimum2scp/repos","events_url":"https://api.github.com/users/minimum2scp/events{/privacy}","received_events_url":"https://api.github.com/users/minimum2scp/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/docker/docker/labels/dco/yes","name":"dco/yes","color":"1693A7"},{"url":"https://api.github.com/repos/docker/docker/labels/status/needs-code-review","name":"status/needs-code-review","color":"F56991"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2015-03-25T15:24:27Z","updated_at":"2015-03-25T15:34:40Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/docker/docker/pulls/11748","html_url":"https://github.com/docker/docker/pull/11748","diff_url":"https://github.com/docker/docker/pull/11748.diff","patch_url":"https://github.com/docker/docker/pull/11748.patch"},"body":"`contrib/mkimage.sh debootstrap` creates a debian image which\r\nhas unnecessary tabs in /usr/sbin/policy-rc.d, and /sbin/initctl.\r\n\r\n```\r\n% sudo contrib/mkimage.sh debootstrap --arch=amd64 --variant=minbase --components=main stable http://ftp.jp.debian.org/debian                   \r\n+ mkdir -p /var/tmp/docker-mkimage.kqap1CTM6d/rootfs\r\n+ debootstrap --arch=amd64 --variant=minbase --components=main stable /var/tmp/docker-mkimage.kqap1CTM6d/rootfs http://ftp.jp.debian.org/debian\r\nI: Retrieving Release \r\nI: Retrieving Release.gpg \r\nI: Checking Release signature\r\n:\r\n(snip)\r\n:\r\nRemoving intermediate container 735f944a4e73\r\nSuccessfully built cdea84963ea8\r\n+ rm -rf /var/tmp/docker-mkimage.kqap1CTM6d\r\n% docker run --rm cdea84963ea8 cat /usr/sbin/policy-rc.d\r\n\t#!/bin/sh\r\n\r\n\t# For most Docker users, \"apt-get install\" only happens during \"docker build\",\r\n\t# where starting services doesn't work and often fails in humorous ways. This\r\n\t# prevents those failures by stopping the services from attempting to start.\r\n\r\n\texit 101\r\n% docker run --rm cdea84963ea8 cat /sbin/initctl\r\n\t#!/bin/sh\r\n\r\n\t# For most Docker users, \"apt-get install\" only happens during \"docker build\",\r\n\t# where starting services doesn't work and often fails in humorous ways. This\r\n\t# prevents those failures by stopping the services from attempting to start.\r\n\r\n\texit 101\r\n```\r\n\r\nThis pull request removes those tabs.\r\n\r\n```\r\n% git checkout fix/debootstrap\r\nSwitched to branch 'fix/debootstrap'\r\nYour branch is up-to-date with 'origin/fix/debootstrap'.\r\n% sudo contrib/mkimage.sh debootstrap --arch=amd64 --variant=minbase --components=main stable http://ftp.jp.debian.org/debian\r\n+ mkdir -p /var/tmp/docker-mkimage.uePTXpg4pc/rootfs\r\n+ debootstrap --arch=amd64 --variant=minbase --components=main stable /var/tmp/docker-mkimage.uePTXpg4pc/rootfs http://ftp.jp.debian.org/debian\r\nI: Retrieving Release \r\nI: Retrieving Release.gpg \r\nI: Checking Release signature\r\n:\r\n(snip)\r\n:\r\nRemoving intermediate container f51d632cb42d\r\nSuccessfully built d21a4c46d84d\r\n+ rm -rf /var/tmp/docker-mkimage.uePTXpg4pc\r\n% docker run --rm d21a4c46d84d cat /usr/sbin/policy-rc.d\r\n#!/bin/sh\r\n\r\n# For most Docker users, \"apt-get install\" only happens during \"docker build\",\r\n# where starting services doesn't work and often fails in humorous ways. This\r\n# prevents those failures by stopping the services from attempting to start.\r\n\r\nexit 101\r\n% docker run --rm d21a4c46d84d cat /sbin/initctl\r\n#!/bin/sh\r\n\r\n# For most Docker users, \"apt-get install\" only happens during \"docker build\",\r\n# where starting services doesn't work and often fails in humorous ways. This\r\n# prevents those failures by stopping the services from attempting to start.\r\n\r\nexit 0\r\n```\r\n"},"comment":{"url":"https://api.github.com/repos/docker/docker/issues/comments/86084471","html_url":"https://github.com/docker/docker/pull/11748#issuecomment-86084471","issue_url":"https://api.github.com/repos/docker/docker/issues/11748","id":86084471,"user":{"login":"tianon","id":161631,"avatar_url":"https://avatars.githubusercontent.com/u/161631?v=3","gravatar_id":"","url":"https://api.github.com/users/tianon","html_url":"https://github.com/tianon","followers_url":"https://api.github.com/users/tianon/followers","following_url":"https://api.github.com/users/tianon/following{/other_user}","gists_url":"https://api.github.com/users/tianon/gists{/gist_id}","starred_url":"https://api.github.com/users/tianon/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tianon/subscriptions","organizations_url":"https://api.github.com/users/tianon/orgs","repos_url":"https://api.github.com/users/tianon/repos","events_url":"https://api.github.com/users/tianon/events{/privacy}","received_events_url":"https://api.github.com/users/tianon/received_events","type":"User","site_admin":false},"created_at":"2015-03-25T15:34:40Z","updated_at":"2015-03-25T15:34:40Z","body":"LGTM"}}