)]}'
{"tools/create_userrc.sh":[{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"3f5fd3d2ac35ff09ab22c46ee5fac689d54281e2","unresolved":false,"context_lines":[{"line_number":178,"context_line":"    local ec2_access_key ec2_secret_key"},{"line_number":179,"context_line":"    read ec2_access_key ec2_secret_key \u003c\u003c\u003c  `echo $line | awk \u0027{print $2 \" \" $4 }\u0027`"},{"line_number":180,"context_line":""},{"line_number":181,"context_line":"    local ec2_cert\u003d\"\""},{"line_number":182,"context_line":"    local ec2_private_key\u003d\"\""},{"line_number":183,"context_line":"    if [ \"$SKIP_CERTS\" !\u003d \"True\" ]; then"},{"line_number":184,"context_line":"        # The certs subject part are the tenant ID \"dash\" user ID, but the CN should be the first part of the DN"}],"source_content_type":"text/x-sh","patch_set":3,"id":"9aa53dc9_2c8071bf","line":181,"updated":"2015-04-10 08:35:54.000000000","message":"I am not sure why you had to add these 2 lines ?","commit_id":"b081d8aff321498208f6861afa9c161c53474954"},{"author":{"_account_id":10215,"name":"Peter Stachowski","email":"peter@tesora.com","username":"peterstac"},"change_message_id":"703583172aed489e3012235ea9978fc4cda1b0d9","unresolved":false,"context_lines":[{"line_number":202,"context_line":"                mv -f \"$ec2_cert.old\" \"$ec2_cert\""},{"line_number":203,"context_line":"            fi"},{"line_number":204,"context_line":"        fi"},{"line_number":205,"context_line":"    fi"},{"line_number":206,"context_line":""},{"line_number":207,"context_line":"    mkdir -p \"$ACCOUNT_DIR/$tenant_name\""},{"line_number":208,"context_line":"    local rcfile\u003d\"$ACCOUNT_DIR/$tenant_name/$user_name\""}],"source_content_type":"text/x-sh","patch_set":3,"id":"5aa745d1_107556ca","line":205,"updated":"2015-04-16 15:20:42.000000000","message":"I agree with the other comments (adding the two lines to initialize unused variables and moving the dir/file setup lines) - it looks like putting the \u0027if\u0027 around the relevant code blocks is all that\u0027s required.","commit_id":"b081d8aff321498208f6861afa9c161c53474954"},{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"087304c3f1559a17773747ff6e188fd7ca03fdd2","unresolved":false,"context_lines":[{"line_number":205,"context_line":"    fi"},{"line_number":206,"context_line":""},{"line_number":207,"context_line":"    mkdir -p \"$ACCOUNT_DIR/$tenant_name\""},{"line_number":208,"context_line":"    local rcfile\u003d\"$ACCOUNT_DIR/$tenant_name/$user_name\""},{"line_number":209,"context_line":""},{"line_number":210,"context_line":"    cat \u003e\"$rcfile\" \u003c\u003cEOF"},{"line_number":211,"context_line":"# you can source this file"}],"source_content_type":"text/x-sh","patch_set":3,"id":"7aaa499b_3b73b04f","line":208,"updated":"2015-04-10 22:10:35.000000000","message":"this seems to have been duplicated with line 168... probably easiest to just keep this simple and not reorder unless necessary","commit_id":"b081d8aff321498208f6861afa9c161c53474954"},{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"b31161d3578ea3606eaf4cde92857a49ef4b056e","unresolved":false,"context_lines":[{"line_number":205,"context_line":"    fi"},{"line_number":206,"context_line":""},{"line_number":207,"context_line":"    mkdir -p \"$ACCOUNT_DIR/$tenant_name\""},{"line_number":208,"context_line":"    local rcfile\u003d\"$ACCOUNT_DIR/$tenant_name/$user_name\""},{"line_number":209,"context_line":""},{"line_number":210,"context_line":"    cat \u003e\"$rcfile\" \u003c\u003cEOF"},{"line_number":211,"context_line":"# you can source this file"}],"source_content_type":"text/x-sh","patch_set":3,"id":"7aaa499b_13c67467","line":208,"in_reply_to":"7aaa499b_3b73b04f","updated":"2015-04-13 15:55:44.000000000","message":"I agree with Gordon here.","commit_id":"b081d8aff321498208f6861afa9c161c53474954"}]}