)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":1812,"name":"p-draigbrady","email":"P@draigBrady.com","username":"p-draigbrady"},"change_message_id":"053862fb4a6d9f484cbedf817e38945b199a312f","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Refactor conf sample generator script"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Changed grep logic to catch files creating Opt instances."},{"line_number":10,"context_line":"Defined variables to enhance readability."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: Ic873f2c3b600a737ce54444dec973720c8813285"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"AAAAPX%2F%2FoMw%3D","line":9,"updated":"2013-06-13 23:57:46.000000000","message":"The line above is now incorrect. It should say:\n\n  Optimized grep calls to not create a grep process per file.","commit_id":"0ad92feb15449638ed1ce34d5ad9bf56d7aed39c"}],"etc/nova/nova.conf.sample":[{"author":{"_account_id":5652,"name":"David Ripton","email":"dripton@redhat.com","username":"dripton"},"change_message_id":"1147a345f359e8fc153ccd65ee87f41fb87e7aac","unresolved":false,"context_lines":[{"line_number":642,"context_line":"#"},{"line_number":643,"context_line":""},{"line_number":644,"context_line":"# default flavor to use, testing only (string value)"},{"line_number":645,"context_line":"#default_flavor\u003dm1.small"},{"line_number":646,"context_line":""},{"line_number":647,"context_line":""},{"line_number":648,"context_line":"#"}],"source_content_type":"application/octet-stream","patch_set":3,"id":"AAAAPX%2F%2Fsf0%3D","line":645,"updated":"2013-06-12 16:31:42.000000000","message":"another good but unrelated change","commit_id":"87cb9ebdac115671638e4fd3718ed0067030e58d"}],"tools/conf/generate_sample.sh":[{"author":{"_account_id":1812,"name":"p-draigbrady","email":"P@draigBrady.com","username":"p-draigbrady"},"change_message_id":"a27f52b9f85f8d256f7d77dfa626067e2137ffda","unresolved":false,"context_lines":[{"line_number":1,"context_line":"#!/usr/bin/env bash"},{"line_number":2,"context_line":"# vim: tabstop\u003d4 shiftwidth\u003d4 softtabstop\u003d4"},{"line_number":3,"context_line":""},{"line_number":4,"context_line":"# Copyright (c) 2013 Intel Corporation."},{"line_number":5,"context_line":"# Copyright 2012 SINA Corporation"},{"line_number":6,"context_line":"# All Rights Reserved."},{"line_number":7,"context_line":"#"}],"source_content_type":"text/x-sh","patch_set":3,"id":"AAAAPX%2F%2Fp8c%3D","line":4,"updated":"2013-06-13 11:22:59.000000000","message":"There was a thread recently which suggested that new copyright\nheaders should not be added, with a view to removing all copyright header info in future.","commit_id":"87cb9ebdac115671638e4fd3718ed0067030e58d"},{"author":{"_account_id":1812,"name":"p-draigbrady","email":"P@draigBrady.com","username":"p-draigbrady"},"change_message_id":"a27f52b9f85f8d256f7d77dfa626067e2137ffda","unresolved":false,"context_lines":[{"line_number":21,"context_line":"#"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"FILES\u003d$(find nova -type f -name \"*.py\" ! -path \"nova/tests/*\" -exec \\"},{"line_number":24,"context_line":"        grep -l \"register_.*opts\\?(\" {} \\; | sort -u)"},{"line_number":25,"context_line":"BINS\u003d$(echo bin/nova-* | grep -v nova-rootwrap)"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"export EVENTLET_NO_GREENDNS\u003dyes"}],"source_content_type":"text/x-sh","patch_set":3,"id":"AAAAPX%2F%2Fp8k%3D","line":24,"updated":"2013-06-13 11:22:59.000000000","message":"If optimizing, you might as well avoid a grep process per file.\nTo do that s/\\;/+/ in the command above","commit_id":"87cb9ebdac115671638e4fd3718ed0067030e58d"},{"author":{"_account_id":1812,"name":"p-draigbrady","email":"P@draigBrady.com","username":"p-draigbrady"},"change_message_id":"053862fb4a6d9f484cbedf817e38945b199a312f","unresolved":false,"context_lines":[{"line_number":20,"context_line":"#"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"FILES\u003d$(find nova -type f -name \"*.py\" ! -path \"nova/tests/*\" \\"},{"line_number":23,"context_line":"        -exec grep -l \"\\\\..\\+Opt(\" {} + | sort -u)"},{"line_number":24,"context_line":"BINS\u003d$(echo bin/nova-* | grep -v nova-rootwrap)"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"export EVENTLET_NO_GREENDNS\u003dyes"}],"source_content_type":"text/x-sh","patch_set":4,"id":"AAAAPX%2F%2FoM0%3D","line":23,"updated":"2013-06-13 23:57:46.000000000","message":"I compared the output from the original RE\n\n  \"Opt(\"\n\nand the newer more complex\n\n  \"\\\\..\\+Opt(\"\n\nand they\u0027re the same. So is this part of the change strictly necessary. As simple as possible and all that...","commit_id":"0ad92feb15449638ed1ce34d5ad9bf56d7aed39c"}]}