)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":1000001,"name":"tazjin","email":"tazjin@tvl.su","username":"tazjin"},"change_message_id":"ed5666957c54a9d6d88a9cdb19b871909e0aa557","unresolved":true,"context_lines":[{"line_number":27,"context_line":"* Introduce the Owners-Override label, settable by depot-interventions"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"The enclosed patch adds two extra pieces of functionality that"},{"line_number":30,"context_line":"we need in tvldepot but aren\u0027t upstream:"},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"1. The ability to just specify usernames rather than email addresses"},{"line_number":33,"context_line":"2. The ability to specify `group:GROUPNAME`, _as long as_ that group is"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"1f2e061a_fe537edb","line":30,"range":{"start_line":30,"start_character":0,"end_line":30,"end_character":40},"updated":"2022-09-19 09:25:45.000000000","message":"I think the first thing should be able to be upstreamed, no?","commit_id":"4e580ab30caf80bab4276d81120a40785b03c713"},{"author":{"_account_id":1000000,"name":"lukegb","email":"lukegb@tvl.fyi","username":"lukegb","status":"zzz"},"change_message_id":"32a4d1cb22f353b6beae56274d64634390204620","unresolved":false,"context_lines":[{"line_number":27,"context_line":"* Introduce the Owners-Override label, settable by depot-interventions"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"The enclosed patch adds two extra pieces of functionality that"},{"line_number":30,"context_line":"we need in tvldepot but aren\u0027t upstream:"},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"1. The ability to just specify usernames rather than email addresses"},{"line_number":33,"context_line":"2. The ability to specify `group:GROUPNAME`, _as long as_ that group is"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"1f7675ba_5d97e45f","line":30,"range":{"start_line":30,"start_character":0,"end_line":30,"end_character":40},"in_reply_to":"1f2e061a_fe537edb","updated":"2022-09-19 10:53:07.000000000","message":"In _theory_ yes, but in practice I just pretend that usernames are email addresses rather than fixing their rusted-in-place abstraction. To upstream it properly I think I\u0027d need to put a bit more work into it and replace it with a tagged-union AutoOneOf thing.","commit_id":"4e580ab30caf80bab4276d81120a40785b03c713"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":1000001,"name":"tazjin","email":"tazjin@tvl.su","username":"tazjin"},"change_message_id":"ed5666957c54a9d6d88a9cdb19b871909e0aa557","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"01d543d4_01a3acad","updated":"2022-09-19 09:25:45.000000000","message":"Excellent as always","commit_id":"4e580ab30caf80bab4276d81120a40785b03c713"}],"third_party/gerrit_plugins/builder.nix":[{"author":{"_account_id":1000034,"name":"sterni","email":"sternenseemann@systemli.org","username":"sterni"},"change_message_id":"9fda5d4a294f9dcb9d184ac818ba60aa45375c24","unresolved":true,"context_lines":[{"line_number":29,"context_line":"      postPatch \u003d \u0027\u0027"},{"line_number":30,"context_line":"        ${super.postPatch or \"\"}"},{"line_number":31,"context_line":"        pushd \"plugins/${name}\""},{"line_number":32,"context_line":"        ${lib.concatMapStringsSep \"\\n\" (patch: \u0027\u0027"},{"line_number":33,"context_line":"          patch -p1 \u003c ${patch}"},{"line_number":34,"context_line":"        \u0027\u0027) patches}"},{"line_number":35,"context_line":"        popd"}],"source_content_type":"text/x-nix","patch_set":2,"id":"5de89932_98ab1296","line":32,"updated":"2022-09-19 09:44:27.000000000","message":"has the underlying thing a custom patchPhase or why can\u0027t we append to the normal patches list?","commit_id":"4e580ab30caf80bab4276d81120a40785b03c713"},{"author":{"_account_id":1000000,"name":"lukegb","email":"lukegb@tvl.fyi","username":"lukegb","status":"zzz"},"change_message_id":"32a4d1cb22f353b6beae56274d64634390204620","unresolved":false,"context_lines":[{"line_number":29,"context_line":"      postPatch \u003d \u0027\u0027"},{"line_number":30,"context_line":"        ${super.postPatch or \"\"}"},{"line_number":31,"context_line":"        pushd \"plugins/${name}\""},{"line_number":32,"context_line":"        ${lib.concatMapStringsSep \"\\n\" (patch: \u0027\u0027"},{"line_number":33,"context_line":"          patch -p1 \u003c ${patch}"},{"line_number":34,"context_line":"        \u0027\u0027) patches}"},{"line_number":35,"context_line":"        popd"}],"source_content_type":"text/x-nix","patch_set":2,"id":"cc20221c_0d9bc38e","line":32,"in_reply_to":"5de89932_98ab1296","updated":"2022-09-19 10:53:07.000000000","message":"We can\u0027t append to the normal patches list because the patches here are expected to be in the context of the plugin directory rather than the source root and I couldn\u0027t be bothered to figure out how to do this better at the time.","commit_id":"4e580ab30caf80bab4276d81120a40785b03c713"}]}