)]}'
{"integration-test/test.sh":[{"author":{"_account_id":23851,"name":"Riccardo Pittau","email":"elfosardo@gmail.com","username":"elfosardo"},"change_message_id":"5b98d14f1a097c07a8a7ac03404406f187edd4e9","unresolved":false,"context_lines":[{"line_number":10,"context_line":"set -e"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"REPO_ROOT\u003d${REPO_ROOT:-git://git.openstack.org}"},{"line_number":13,"context_line":"HACKING\u003d`pwd`"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"if [[ $# -lt 2 ]] ; then"},{"line_number":16,"context_line":"    echo \"Script needs at least two arguments:\""}],"source_content_type":"text/x-sh","patch_set":21,"id":"9f6a8fd7_9239dbf3","line":13,"updated":"2018-04-27 09:14:11.000000000","message":"I suggest to use the preferred form $() for command substitution.\nAlso, since you need the literal result and it\u0027s a path, I would quote the expression.\nReference:\nhttp://mywiki.wooledge.org/BashFAQ/082","commit_id":"beeff2f9674495f14a387363714c7dcbdb5587f2"},{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"1ef3a1781d271226d844fa98e0d40471311c1146","unresolved":false,"context_lines":[{"line_number":10,"context_line":"set -e"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"REPO_ROOT\u003d${REPO_ROOT:-git://git.openstack.org}"},{"line_number":13,"context_line":"HACKING\u003d`pwd`"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"if [[ $# -lt 2 ]] ; then"},{"line_number":16,"context_line":"    echo \"Script needs at least two arguments:\""}],"source_content_type":"text/x-sh","patch_set":21,"id":"9f6a8fd7_955545fd","line":13,"in_reply_to":"9f6a8fd7_9239dbf3","updated":"2018-04-27 09:19:22.000000000","message":"Done","commit_id":"beeff2f9674495f14a387363714c7dcbdb5587f2"}]}