)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9664,"name":"amrith","email":"amrith.kumar@gmail.com","username":"amrith"},"change_message_id":"5d01cd6e618d759898bc2627cae1c18bb56555be","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix enable on boot when working with systemd and symlinked units"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"mysql on Fedora20 is actually mariadb and /usr/lib/systemd/system/mysqld.service"},{"line_number":10,"context_line":"is a symbolic link to /usr/lib/systemd/system/mariadb.service. systemctl \"enable\""},{"line_number":11,"context_line":"doesn\u0027t work on symlinked inits like described in:"},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"1ae5cdf2_e8f8d14b","line":9,"updated":"2014-06-21 22:57:13.000000000","message":"I\u0027m surprised that this line worked, it is 73 characters long and the next one is 74 ...","commit_id":"3482d5f4e6175e9e99e78eeaef6eeb730d6c239a"}],"trove/guestagent/common/operating_system.py":[{"author":{"_account_id":8415,"name":"Denis M.","email":"lildee1991@gmail.com","username":"dmakogon"},"change_message_id":"859ecb565e8f3cae7ef879ac1c5791051db67370","unresolved":false,"context_lines":[{"line_number":70,"context_line":"            result[\u0027cmd_stop\u0027] \u003d \"sudo systemctl stop %s\" % service"},{"line_number":71,"context_line":""},{"line_number":72,"context_line":"            # currently \"systemctl enable\" doesn\u0027t work for symlinked units as described"},{"line_number":73,"context_line":"            # in https://bugzilla.redhat.com/1014311, therefore replacing a symlink with its real"},{"line_number":74,"context_line":"            # path"},{"line_number":75,"context_line":"            if os.path.islink(service_path):"},{"line_number":76,"context_line":"                real_path \u003d os.path.realpath(service_path)"}],"source_content_type":"text/x-python","patch_set":1,"id":"1ae5cdf2_bf430a6b","line":73,"updated":"2014-06-17 13:35:39.000000000","message":"too long lines, should be less than 79 symbols","commit_id":"3015156cc2bae0efb73baca270611ddb015f430c"},{"author":{"_account_id":10725,"name":"Sergey Gotliv","email":"sgotliv@redhat.com","username":"sgotliv"},"change_message_id":"5f26e2e857aaa2f96c6898b2ff2fb334517a0f01","unresolved":false,"context_lines":[{"line_number":70,"context_line":"            result[\u0027cmd_stop\u0027] \u003d \"sudo systemctl stop %s\" % service"},{"line_number":71,"context_line":""},{"line_number":72,"context_line":"            # currently \"systemctl enable\" doesn\u0027t work for symlinked units as described"},{"line_number":73,"context_line":"            # in https://bugzilla.redhat.com/1014311, therefore replacing a symlink with its real"},{"line_number":74,"context_line":"            # path"},{"line_number":75,"context_line":"            if os.path.islink(service_path):"},{"line_number":76,"context_line":"                real_path \u003d os.path.realpath(service_path)"}],"source_content_type":"text/x-python","patch_set":1,"id":"1ae5cdf2_72e8d916","line":73,"in_reply_to":"1ae5cdf2_bf430a6b","updated":"2014-06-17 13:42:43.000000000","message":"Will do that.","commit_id":"3015156cc2bae0efb73baca270611ddb015f430c"},{"author":{"_account_id":9746,"name":"Doug Shelley","email":"dougshelley66@gmail.com","username":"dougshelley"},"change_message_id":"f326e7cda11d722b346595cf20b3953ef14aa914","unresolved":false,"context_lines":[{"line_number":75,"context_line":"            if os.path.islink(service_path):"},{"line_number":76,"context_line":"                real_path \u003d os.path.realpath(service_path)"},{"line_number":77,"context_line":"                result[\u0027cmd_enable\u0027] \u003d \"sudo systemctl enable %s\" % real_path"},{"line_number":78,"context_line":"                result[\u0027cmd_disable\u0027] \u003d \"sudo systemctl disable %s\" % \\"},{"line_number":79,"context_line":"                                        real_path"},{"line_number":80,"context_line":"            else:"},{"line_number":81,"context_line":"                result[\u0027cmd_enable\u0027] \u003d \"sudo systemctl enable %s\" % service"}],"source_content_type":"text/x-python","patch_set":2,"id":"1ae5cdf2_e549618f","line":78,"updated":"2014-06-18 02:19:47.000000000","message":"Sergey - the OS hacking guidelines [1] say not to continue a line with a backslash\n\n[1] http://docs.openstack.org/developer/hacking/#general","commit_id":"8f7c7d95c9c7a49851041335087c3e067c76186c"},{"author":{"_account_id":10725,"name":"Sergey Gotliv","email":"sgotliv@redhat.com","username":"sgotliv"},"change_message_id":"12795d97f66360cb142670ea3a8acc083830f29b","unresolved":false,"context_lines":[{"line_number":75,"context_line":"            if os.path.islink(service_path):"},{"line_number":76,"context_line":"                real_path \u003d os.path.realpath(service_path)"},{"line_number":77,"context_line":"                result[\u0027cmd_enable\u0027] \u003d \"sudo systemctl enable %s\" % real_path"},{"line_number":78,"context_line":"                result[\u0027cmd_disable\u0027] \u003d \"sudo systemctl disable %s\" % \\"},{"line_number":79,"context_line":"                                        real_path"},{"line_number":80,"context_line":"            else:"},{"line_number":81,"context_line":"                result[\u0027cmd_enable\u0027] \u003d \"sudo systemctl enable %s\" % service"}],"source_content_type":"text/x-python","patch_set":2,"id":"1ae5cdf2_721fd070","line":78,"in_reply_to":"1ae5cdf2_e549618f","updated":"2014-06-18 06:47:52.000000000","message":"First, thanks for review.\n\nI have no problem to change it, but \n\n1. It seems like a backslash appears in many places in code, you even can see it in this module (line 59).\n\n2. This code have passed a pep8 gate.","commit_id":"8f7c7d95c9c7a49851041335087c3e067c76186c"}]}