)]}'
{"milestone.sh":[{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"fe9540e5e4e545b55011db65bc00899c9c2b87a4","unresolved":false,"context_lines":[{"line_number":60,"context_line":"    echo \"Could not determine pre-version from setup.cfg\""},{"line_number":61,"context_line":"    exit 1"},{"line_number":62,"context_line":"fi"},{"line_number":63,"context_line":"VERSION\u003d\"${RELVERSION}b${MILESTONE: -1}\""},{"line_number":64,"context_line":"echo \"${PROJECT^} $MILESTONE is $VERSION (final being $RELVERSION)\""},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"title \"Tagging $TARGETSHA as $VERSION\""}],"source_content_type":"text/x-sh","patch_set":1,"id":"fa32b979_fd4f1568","line":63,"updated":"2015-06-22 17:06:16.000000000","message":"The space after the MILESTONE: looks like a typo, unless it\u0027s a bash thing I\u0027m not familiar with?\n\n  ${MILESTONE:-1} yields \"1\"\n  ${MILESTONE: -1} yields \"\"","commit_id":"d3ebc510175c2842ed6cde3834399e94bea70cfa"},{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"571ce4dc9ab5e7c2864075168972b7736539ffb9","unresolved":false,"context_lines":[{"line_number":60,"context_line":"    echo \"Could not determine pre-version from setup.cfg\""},{"line_number":61,"context_line":"    exit 1"},{"line_number":62,"context_line":"fi"},{"line_number":63,"context_line":"VERSION\u003d\"${RELVERSION}b${MILESTONE: -1}\""},{"line_number":64,"context_line":"echo \"${PROJECT^} $MILESTONE is $VERSION (final being $RELVERSION)\""},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"title \"Tagging $TARGETSHA as $VERSION\""}],"source_content_type":"text/x-sh","patch_set":1,"id":"fa32b979_7a63e441","line":63,"in_reply_to":"fa32b979_eeb09a8d","updated":"2015-06-23 11:39:35.000000000","message":"OK, the last character of an empty string is nothing, so I think it was working right I just wasn\u0027t testing it properly. I don\u0027t think I\u0027ve ever seen the construct used that way, to get a substring, but it does work properly for me.","commit_id":"d3ebc510175c2842ed6cde3834399e94bea70cfa"},{"author":{"_account_id":308,"name":"Thierry Carrez","email":"thierry@openstack.org","username":"ttx"},"change_message_id":"55238b42c380facb3083a78f07626736be64d93c","unresolved":false,"context_lines":[{"line_number":60,"context_line":"    echo \"Could not determine pre-version from setup.cfg\""},{"line_number":61,"context_line":"    exit 1"},{"line_number":62,"context_line":"fi"},{"line_number":63,"context_line":"VERSION\u003d\"${RELVERSION}b${MILESTONE: -1}\""},{"line_number":64,"context_line":"echo \"${PROJECT^} $MILESTONE is $VERSION (final being $RELVERSION)\""},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"title \"Tagging $TARGETSHA as $VERSION\""}],"source_content_type":"text/x-sh","patch_set":1,"id":"fa32b979_eeb09a8d","line":63,"in_reply_to":"fa32b979_fd4f1568","updated":"2015-06-23 10:23:51.000000000","message":"You should get this:\n\n  $ echo \"${M:-1}\"\n  1\n  $ M\u003d\"liberty-2\"\n  $ echo \"${M:-1}\"\n  liberty-2\n  $ echo \"${M: -1}\"\n  2\n\nBash man page says \"Note that a negative offset must be separated from the colon by at least one space to avoid being confused with  the  :-  expansion.\"","commit_id":"d3ebc510175c2842ed6cde3834399e94bea70cfa"}]}