{"comment":{"url":"https://api.github.com/repos/mgorny/pkgcore/comments/12767467","html_url":"https://github.com/mgorny/pkgcore/commit/6182485f5ea927b301d17e5036caa91c6facdb47#commitcomment-12767467","id":12767467,"user":{"login":"mgorny","id":110765,"avatar_url":"https://avatars.githubusercontent.com/u/110765?v=3","gravatar_id":"","url":"https://api.github.com/users/mgorny","html_url":"https://github.com/mgorny","followers_url":"https://api.github.com/users/mgorny/followers","following_url":"https://api.github.com/users/mgorny/following{/other_user}","gists_url":"https://api.github.com/users/mgorny/gists{/gist_id}","starred_url":"https://api.github.com/users/mgorny/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mgorny/subscriptions","organizations_url":"https://api.github.com/users/mgorny/orgs","repos_url":"https://api.github.com/users/mgorny/repos","events_url":"https://api.github.com/users/mgorny/events{/privacy}","received_events_url":"https://api.github.com/users/mgorny/received_events","type":"User","site_admin":false},"position":23,"line":218,"path":"pkgcore/spawn.py","commit_id":"6182485f5ea927b301d17e5036caa91c6facdb47","created_at":"2015-08-18T18:19:36Z","updated_at":"2015-08-18T18:19:36Z","body":"I was thinking about that but assumed it doesn't really matter since the function is internal. But if you want, I can rework the whole thing in a separate commit. For example, some of the `if`s inside would use `is not None`, you can't do `umask=0` now :)."}}