{"action":"created","issue":{"url":"https://api.github.com/repos/dnschneid/crouton/issues/1629","labels_url":"https://api.github.com/repos/dnschneid/crouton/issues/1629/labels{/name}","comments_url":"https://api.github.com/repos/dnschneid/crouton/issues/1629/comments","events_url":"https://api.github.com/repos/dnschneid/crouton/issues/1629/events","html_url":"https://github.com/dnschneid/crouton/issues/1629","id":68841876,"number":1629,"title":"crouton -d -f should confirm directory is writable *before* downloading everything","user":{"login":"dnschneid","id":522200,"avatar_url":"https://avatars.githubusercontent.com/u/522200?v=3","gravatar_id":"","url":"https://api.github.com/users/dnschneid","html_url":"https://github.com/dnschneid","followers_url":"https://api.github.com/users/dnschneid/followers","following_url":"https://api.github.com/users/dnschneid/following{/other_user}","gists_url":"https://api.github.com/users/dnschneid/gists{/gist_id}","starred_url":"https://api.github.com/users/dnschneid/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dnschneid/subscriptions","organizations_url":"https://api.github.com/users/dnschneid/orgs","repos_url":"https://api.github.com/users/dnschneid/repos","events_url":"https://api.github.com/users/dnschneid/events{/privacy}","received_events_url":"https://api.github.com/users/dnschneid/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/dnschneid/crouton/labels/bug","name":"bug","color":"fc2929"},{"url":"https://api.github.com/repos/dnschneid/crouton/labels/pie","name":"pie","color":"bfe5bf"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":29,"created_at":"2015-04-16T04:32:22Z","updated_at":"2015-04-28T06:22:02Z","closed_at":null,"body":"If you try to create a bootstrap file in a read-only directory, crouton will fail at the final step.  It should check ahead of time that the directory is writable.\r\n\r\ne.g., `sudo crouton -f /fail` should fail at the start on a Chromebook (where rootfs is read-only), not at the end."},"comment":{"url":"https://api.github.com/repos/dnschneid/crouton/issues/comments/96941099","html_url":"https://github.com/dnschneid/crouton/issues/1629#issuecomment-96941099","issue_url":"https://api.github.com/repos/dnschneid/crouton/issues/1629","id":96941099,"user":{"login":"tedm","id":3884283,"avatar_url":"https://avatars.githubusercontent.com/u/3884283?v=3","gravatar_id":"","url":"https://api.github.com/users/tedm","html_url":"https://github.com/tedm","followers_url":"https://api.github.com/users/tedm/followers","following_url":"https://api.github.com/users/tedm/following{/other_user}","gists_url":"https://api.github.com/users/tedm/gists{/gist_id}","starred_url":"https://api.github.com/users/tedm/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tedm/subscriptions","organizations_url":"https://api.github.com/users/tedm/orgs","repos_url":"https://api.github.com/users/tedm/repos","events_url":"https://api.github.com/users/tedm/events{/privacy}","received_events_url":"https://api.github.com/users/tedm/received_events","type":"User","site_admin":false},"created_at":"2015-04-28T06:22:02Z","updated_at":"2015-04-28T06:22:02Z","body":"OK, I'll keep working on the base path validation.\r\n\r\nQuestion, In the lines of installer/main.sh , the big block starting at 377 down, is there a write access test that is not using `-w` that I am missing here?\r\n\r\n`# Confirm we have write access to the directory before starting.`\r\n`if [ -z \"$RESTOREBIN$DOWNLOADONLY\" ]; then`\r\n` ... `\r\n\r\n"}}