)]}'
{"imagebuild/coreos/Makefile":[{"author":{"_account_id":11076,"name":"Shivanand Tendulker","email":"stendulker@gmail.com","username":"stendulker"},"change_message_id":"24039f6ccf23b974caaf3ac4a701dff49beb3e11","unresolved":false,"context_lines":[{"line_number":8,"context_line":"\t./docker_build.bash"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"coreos:"},{"line_number":11,"context_line":"\tmkdir -p UPLOAD"},{"line_number":12,"context_line":"\t./coreos-oem-inject.py oem UPLOAD"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"clean:"}],"source_content_type":"application/octet-stream","patch_set":3,"id":"fa98f980_a89ce45e","line":11,"updated":"2014-09-16 17:07:16.000000000","message":"Better to replace the variable name BUILD_DIR instead of using UPLOAD.","commit_id":"2fe9f711da43863de1f2e7d543eb36968943cf54"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"4d9e0fdd029307b037776a007f60c06ea8188aa4","unresolved":false,"context_lines":[{"line_number":8,"context_line":"\t./docker_build.bash"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"coreos:"},{"line_number":11,"context_line":"\tmkdir -p UPLOAD"},{"line_number":12,"context_line":"\t./coreos-oem-inject.py oem UPLOAD"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"clean:"}],"source_content_type":"application/octet-stream","patch_set":3,"id":"fa98f980_63ecdd2a","line":11,"in_reply_to":"fa98f980_a89ce45e","updated":"2014-09-17 00:35:23.000000000","message":"Nice call, I agree.\n\nJust make sure it stays UPLOAD since we have other scripts (the uploader from infra) that depend on this location.","commit_id":"2fe9f711da43863de1f2e7d543eb36968943cf54"}],"imagebuild/coreos/iso-image-create":[{"author":{"_account_id":11076,"name":"Shivanand Tendulker","email":"stendulker@gmail.com","username":"stendulker"},"change_message_id":"24039f6ccf23b974caaf3ac4a701dff49beb3e11","unresolved":false,"context_lines":[{"line_number":40,"context_line":"    rm -rf $TMP_BUILD_DIR"},{"line_number":41,"context_line":"}"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"function d_print() {"},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"    echo \"$*\""},{"line_number":46,"context_line":"    return"}],"source_content_type":"application/x-shellscript","patch_set":3,"id":"fa98f980_28e0f495","line":43,"updated":"2014-09-16 17:07:16.000000000","message":"Where is d_print() used?","commit_id":"2fe9f711da43863de1f2e7d543eb36968943cf54"},{"author":{"_account_id":11076,"name":"Shivanand Tendulker","email":"stendulker@gmail.com","username":"stendulker"},"change_message_id":"24039f6ccf23b974caaf3ac4a701dff49beb3e11","unresolved":false,"context_lines":[{"line_number":74,"context_line":""},{"line_number":75,"context_line":"# Verify whether kernel, initrd, and the image file is present"},{"line_number":76,"context_line":"if [ -z \"$OUTPUT_FILENAME\" ]; then"},{"line_number":77,"context_line":"    echo \"ERROR: Output filename not provided.\""},{"line_number":78,"context_line":"    show_options"},{"line_number":79,"context_line":"    exit 1"},{"line_number":80,"context_line":"fi"}],"source_content_type":"application/x-shellscript","patch_set":3,"id":"fa98f980_a8eb0477","line":77,"updated":"2014-09-16 17:07:16.000000000","message":"Better to write error messages be written to stderr?","commit_id":"2fe9f711da43863de1f2e7d543eb36968943cf54"},{"author":{"_account_id":11076,"name":"Shivanand Tendulker","email":"stendulker@gmail.com","username":"stendulker"},"change_message_id":"24039f6ccf23b974caaf3ac4a701dff49beb3e11","unresolved":false,"context_lines":[{"line_number":80,"context_line":"fi"},{"line_number":81,"context_line":""},{"line_number":82,"context_line":"if [ -z \"$INITRD\" ]; then"},{"line_number":83,"context_line":"    echo \"ERROR: Initrd not provided.\""},{"line_number":84,"context_line":"    show_options"},{"line_number":85,"context_line":"    exit 1"},{"line_number":86,"context_line":"fi"}],"source_content_type":"application/x-shellscript","patch_set":3,"id":"fa98f980_48937800","line":83,"updated":"2014-09-16 17:07:16.000000000","message":"Same as above","commit_id":"2fe9f711da43863de1f2e7d543eb36968943cf54"},{"author":{"_account_id":11076,"name":"Shivanand Tendulker","email":"stendulker@gmail.com","username":"stendulker"},"change_message_id":"24039f6ccf23b974caaf3ac4a701dff49beb3e11","unresolved":false,"context_lines":[{"line_number":86,"context_line":"fi"},{"line_number":87,"context_line":""},{"line_number":88,"context_line":"if [ -z \"$KERNEL\" ]; then"},{"line_number":89,"context_line":"    echo \"ERROR: Kernel not provided.\""},{"line_number":90,"context_line":"    show_options"},{"line_number":91,"context_line":"    exit 1"},{"line_number":92,"context_line":"fi"}],"source_content_type":"application/x-shellscript","patch_set":3,"id":"fa98f980_c88688ba","line":89,"updated":"2014-09-16 17:07:16.000000000","message":"Same as above","commit_id":"2fe9f711da43863de1f2e7d543eb36968943cf54"}]}