)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":1000001,"name":"Lorenz Brun","display_name":"Lorenz","email":"lorenz@monogon.tech","username":"lorenz","avatars":[{"url":"https://www.gravatar.com/avatar/75c04f6e9881c24ee621fba80667eed8.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d32","height":32},{"url":"https://www.gravatar.com/avatar/75c04f6e9881c24ee621fba80667eed8.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d56","height":56},{"url":"https://www.gravatar.com/avatar/75c04f6e9881c24ee621fba80667eed8.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d100","height":100},{"url":"https://www.gravatar.com/avatar/75c04f6e9881c24ee621fba80667eed8.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d120","height":120}]},"change_message_id":"32bb35971d75326c00295d75dc1d06378953d0cf","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"740c553c_94bc6a37","updated":"2021-08-19 10:21:27.000000000","message":"Nice! One nit, otherwise LGTM.","commit_id":"63b8cf7e9e5d6307ec73f2cb43c534970c592c28"}],"third_party/chrony/chrony.bzl":[{"author":{"_account_id":1000001,"name":"Lorenz Brun","display_name":"Lorenz","email":"lorenz@monogon.tech","username":"lorenz","avatars":[{"url":"https://www.gravatar.com/avatar/75c04f6e9881c24ee621fba80667eed8.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d32","height":32},{"url":"https://www.gravatar.com/avatar/75c04f6e9881c24ee621fba80667eed8.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d56","height":56},{"url":"https://www.gravatar.com/avatar/75c04f6e9881c24ee621fba80667eed8.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d100","height":100},{"url":"https://www.gravatar.com/avatar/75c04f6e9881c24ee621fba80667eed8.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d120","height":120}]},"change_message_id":"32bb35971d75326c00295d75dc1d06378953d0cf","unresolved":true,"context_lines":[{"line_number":5,"context_line":"    name \u003d \"config.h\","},{"line_number":6,"context_line":"    src \u003d \"@dev_source_monogon//third_party/chrony:config.h.in\","},{"line_number":7,"context_line":"    substitutions \u003d {"},{"line_number":8,"context_line":"        \"%CHRONY_VERSION%\": \"4.1-monogon\","},{"line_number":9,"context_line":"    }"},{"line_number":10,"context_line":")"},{"line_number":11,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"f613a6aa_8bfb3bf0","line":8,"updated":"2021-08-19 10:21:27.000000000","message":"Is there any possibility to link this to the version in the external file? I\u0027m a bit concerned this will very easily become stale. If not, we may want to make this a generic version to encourage people to look into the WORKSPACE file instead of making wrong assumptions.","commit_id":"63b8cf7e9e5d6307ec73f2cb43c534970c592c28"},{"author":{"_account_id":1000002,"name":"Serge Bazanski","display_name":"Serge","email":"serge@monogon.tech","username":"serge","avatars":[{"url":"https://www.gravatar.com/avatar/52c41428b6369f2c02b9717425216f7d.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d32","height":32},{"url":"https://www.gravatar.com/avatar/52c41428b6369f2c02b9717425216f7d.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d56","height":56},{"url":"https://www.gravatar.com/avatar/52c41428b6369f2c02b9717425216f7d.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d100","height":100},{"url":"https://www.gravatar.com/avatar/52c41428b6369f2c02b9717425216f7d.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d120","height":120}]},"change_message_id":"1b9eeb6c2ab8940feb4f9bacbf07c81bf000269f","unresolved":false,"context_lines":[{"line_number":5,"context_line":"    name \u003d \"config.h\","},{"line_number":6,"context_line":"    src \u003d \"@dev_source_monogon//third_party/chrony:config.h.in\","},{"line_number":7,"context_line":"    substitutions \u003d {"},{"line_number":8,"context_line":"        \"%CHRONY_VERSION%\": \"4.1-monogon\","},{"line_number":9,"context_line":"    }"},{"line_number":10,"context_line":")"},{"line_number":11,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"03715709_40758214","line":8,"in_reply_to":"54499e14_b9141f72","updated":"2021-08-19 11:53:28.000000000","message":"Done","commit_id":"63b8cf7e9e5d6307ec73f2cb43c534970c592c28"},{"author":{"_account_id":1000002,"name":"Serge Bazanski","display_name":"Serge","email":"serge@monogon.tech","username":"serge","avatars":[{"url":"https://www.gravatar.com/avatar/52c41428b6369f2c02b9717425216f7d.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d32","height":32},{"url":"https://www.gravatar.com/avatar/52c41428b6369f2c02b9717425216f7d.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d56","height":56},{"url":"https://www.gravatar.com/avatar/52c41428b6369f2c02b9717425216f7d.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d100","height":100},{"url":"https://www.gravatar.com/avatar/52c41428b6369f2c02b9717425216f7d.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d120","height":120}]},"change_message_id":"ef97a7498868ef62afc9cd6a44104fc31ca2755f","unresolved":true,"context_lines":[{"line_number":5,"context_line":"    name \u003d \"config.h\","},{"line_number":6,"context_line":"    src \u003d \"@dev_source_monogon//third_party/chrony:config.h.in\","},{"line_number":7,"context_line":"    substitutions \u003d {"},{"line_number":8,"context_line":"        \"%CHRONY_VERSION%\": \"4.1-monogon\","},{"line_number":9,"context_line":"    }"},{"line_number":10,"context_line":")"},{"line_number":11,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"54499e14_b9141f72","line":8,"in_reply_to":"f613a6aa_8bfb3bf0","updated":"2021-08-19 11:31:22.000000000","message":"Keeping this in sync automagically is possible but IMO isn\u0027t worth the complexity (you\u0027d have to use build_file_contents and template them, and that IMO isn\u0027t worth the readability penalty).\n\nInstead I\u0027ve removed the version argument from chrony_external and added ONCHANGE comments to encourage this to be kept in sync.","commit_id":"63b8cf7e9e5d6307ec73f2cb43c534970c592c28"}]}
