{"action":"created","issue":{"url":"https://api.github.com/repos/davenuman/bowline/issues/29","labels_url":"https://api.github.com/repos/davenuman/bowline/issues/29/labels{/name}","comments_url":"https://api.github.com/repos/davenuman/bowline/issues/29/comments","events_url":"https://api.github.com/repos/davenuman/bowline/issues/29/events","html_url":"https://github.com/davenuman/bowline/pull/29","id":82603444,"number":29,"title":"Syntax issue on mac. Removed commas from variable","user":{"login":"ianlintner","id":500914,"avatar_url":"https://avatars.githubusercontent.com/u/500914?v=3","gravatar_id":"","url":"https://api.github.com/users/ianlintner","html_url":"https://github.com/ianlintner","followers_url":"https://api.github.com/users/ianlintner/followers","following_url":"https://api.github.com/users/ianlintner/following{/other_user}","gists_url":"https://api.github.com/users/ianlintner/gists{/gist_id}","starred_url":"https://api.github.com/users/ianlintner/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ianlintner/subscriptions","organizations_url":"https://api.github.com/users/ianlintner/orgs","repos_url":"https://api.github.com/users/ianlintner/repos","events_url":"https://api.github.com/users/ianlintner/events{/privacy}","received_events_url":"https://api.github.com/users/ianlintner/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-05-29T21:56:58Z","updated_at":"2015-05-30T13:31:56Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/davenuman/bowline/pulls/29","html_url":"https://github.com/davenuman/bowline/pull/29","diff_url":"https://github.com/davenuman/bowline/pull/29.diff","patch_url":"https://github.com/davenuman/bowline/pull/29.patch"},"body":"Don't know if the commas I removed are intended. I am not a great bash scripter, but it would not run on my mac until I removed the commas. \r\n\r\nThanks for the great project!"},"comment":{"url":"https://api.github.com/repos/davenuman/bowline/issues/comments/107041671","html_url":"https://github.com/davenuman/bowline/pull/29#issuecomment-107041671","issue_url":"https://api.github.com/repos/davenuman/bowline/issues/29","id":107041671,"user":{"login":"davenuman","id":644041,"avatar_url":"https://avatars.githubusercontent.com/u/644041?v=3","gravatar_id":"","url":"https://api.github.com/users/davenuman","html_url":"https://github.com/davenuman","followers_url":"https://api.github.com/users/davenuman/followers","following_url":"https://api.github.com/users/davenuman/following{/other_user}","gists_url":"https://api.github.com/users/davenuman/gists{/gist_id}","starred_url":"https://api.github.com/users/davenuman/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/davenuman/subscriptions","organizations_url":"https://api.github.com/users/davenuman/orgs","repos_url":"https://api.github.com/users/davenuman/repos","events_url":"https://api.github.com/users/davenuman/events{/privacy}","received_events_url":"https://api.github.com/users/davenuman/received_events","type":"User","site_admin":false},"created_at":"2015-05-30T13:31:56Z","updated_at":"2015-05-30T13:31:56Z","body":"@ianlintner the double comma is intentional. It changes the value of the variable to lowercase (see http://stackoverflow.com/questions/11392189/converting-string-from-uppercase-to-lowercase-in-bash-shell-scripting) but it looks like that was added to bash in version 4.0. Doing this makes it case-insensitive which is slightly nicer.\r\n\r\nI think Bash 4.0 was released some years ago. Is it trivial for you to upgrade? If not then I'm happy to accept this PR and the trade-off of greater compatibility over the slightly smarter input handling. If you still want to remove the commas then you might want to do so here as well, and add that to the PR:\r\nhttps://github.com/ianlintner/bowline/blob/master/lib/rigging/drupal-core-dev/drupal-core-dev.hoist#L5\r\n\r\nThanks!"}}