{"comment":{"url":"https://api.github.com/repos/mysociety/alaveteli/comments/11424664","html_url":"https://github.com/mysociety/alaveteli/commit/9c66a11f383f24b930e89126fd473c2e0480e577#commitcomment-11424664","id":11424664,"user":{"login":"crowbot","id":2912,"avatar_url":"https://avatars.githubusercontent.com/u/2912?v=3","gravatar_id":"","url":"https://api.github.com/users/crowbot","html_url":"https://github.com/crowbot","followers_url":"https://api.github.com/users/crowbot/followers","following_url":"https://api.github.com/users/crowbot/following{/other_user}","gists_url":"https://api.github.com/users/crowbot/gists{/gist_id}","starred_url":"https://api.github.com/users/crowbot/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/crowbot/subscriptions","organizations_url":"https://api.github.com/users/crowbot/orgs","repos_url":"https://api.github.com/users/crowbot/repos","events_url":"https://api.github.com/users/crowbot/events{/privacy}","received_events_url":"https://api.github.com/users/crowbot/received_events","type":"User","site_admin":false},"position":4,"line":90,"path":"script/install-as-user","commit_id":"9c66a11f383f24b930e89126fd473c2e0480e577","created_at":"2015-05-29T10:39:54Z","updated_at":"2015-05-29T10:39:54Z","body":"Ah, I hadn't seen the significance of the export. However, when `$DEVELOPMENT_INSTALL` is undefined, the script produces the warning \"[: =: unary operator expected\", as the syntax effectively expands to \r\n\r\n    if [ = true ]\r\n\r\nso you should quote the `$DEVELOPMENT_INSTALL` variable: \r\n\r\n     if [ \"$DEVELOPMENT_INSTALL\" = true ]; then"}}