)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":1000001,"name":"tazjin","email":"tazjin@tvl.su","username":"tazjin"},"change_message_id":"4ea276678a33e0966099fa3fd38d734b7774192e","unresolved":true,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"In principle we don\u0027t want to build any (later) pipeline target during"},{"line_number":10,"context_line":"pipeline evaluation insofar they appear in extraSteps. For this reason,"},{"line_number":11,"context_line":"we have the needsOutput mechanism which specifically allows prevents the"},{"line_number":12,"context_line":"parent target of an extraStep from being built in 🦙."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Unfortunately, this mechanism is not general purpose enough, as we use"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"701d5129_1c27b550","line":11,"range":{"start_line":11,"start_character":53,"end_line":11,"end_character":68},"updated":"2024-03-02 23:03:23.000000000","message":"?","commit_id":"4f2a7527ac4eb92918c744e341d016ab2938a0d8"},{"author":{"_account_id":1000034,"name":"sterni","email":"sternenseemann@systemli.org","username":"sterni"},"change_message_id":"7a2f3d3da498c8fce73c6342752b5caa9a907450","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"In principle we don\u0027t want to build any (later) pipeline target during"},{"line_number":10,"context_line":"pipeline evaluation insofar they appear in extraSteps. For this reason,"},{"line_number":11,"context_line":"we have the needsOutput mechanism which specifically allows prevents the"},{"line_number":12,"context_line":"parent target of an extraStep from being built in 🦙."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Unfortunately, this mechanism is not general purpose enough, as we use"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"b4ebfa39_61acb51f","line":11,"range":{"start_line":11,"start_character":53,"end_line":11,"end_character":68},"in_reply_to":"701d5129_1c27b550","updated":"2024-03-02 23:38:56.000000000","message":"Done","commit_id":"4f2a7527ac4eb92918c744e341d016ab2938a0d8"}],"nix/buildkite/default.nix":[{"author":{"_account_id":1000001,"name":"tazjin","email":"tazjin@tvl.su","username":"tazjin"},"change_message_id":"c03ff48ade37b58fd656e1ece4174ec3b8c386de","unresolved":true,"context_lines":[{"line_number":418,"context_line":"          echo \u0027--- Building extra step script\u0027"},{"line_number":419,"context_line":"          # Make sure the script will be ignored by git(1), as some extra steps"},{"line_number":420,"context_line":"          # check whether the tree is dirty or not."},{"line_number":421,"context_line":"          echo \u0027/${commandScriptLink}\u0027 \u003e\u003e .git/info/exclude"},{"line_number":422,"context_line":"          command_script\u003d\"$(${"},{"line_number":423,"context_line":"            # Using command substitution in this way assumes the script drv only has one output"},{"line_number":424,"context_line":"            assert builtins.length cfg.command.outputs \u003d\u003d 1;"}],"source_content_type":"text/x-nix","patch_set":8,"id":"a7292301_0857af14","line":421,"range":{"start_line":421,"start_character":0,"end_line":421,"end_character":2},"updated":"2024-03-03 09:03:50.000000000","message":"Clever! I\u0027m curious whether Buildkite\u0027s cleanup commands get rid of this, though, or if we\u0027re accumulating these in the existing depot clones.\n\nThis seems to be a static string - can\u0027t we add it to the actual depot gitignore?","commit_id":"f219cf0b05b7899687b5507928a1d78089395312"},{"author":{"_account_id":1000034,"name":"sterni","email":"sternenseemann@systemli.org","username":"sterni"},"change_message_id":"55c131075af9d8a088a76f07433aca426694b1fe","unresolved":false,"context_lines":[{"line_number":418,"context_line":"          echo \u0027--- Building extra step script\u0027"},{"line_number":419,"context_line":"          # Make sure the script will be ignored by git(1), as some extra steps"},{"line_number":420,"context_line":"          # check whether the tree is dirty or not."},{"line_number":421,"context_line":"          echo \u0027/${commandScriptLink}\u0027 \u003e\u003e .git/info/exclude"},{"line_number":422,"context_line":"          command_script\u003d\"$(${"},{"line_number":423,"context_line":"            # Using command substitution in this way assumes the script drv only has one output"},{"line_number":424,"context_line":"            assert builtins.length cfg.command.outputs \u003d\u003d 1;"}],"source_content_type":"text/x-nix","patch_set":8,"id":"848c3b97_a0954e90","line":421,"range":{"start_line":421,"start_character":0,"end_line":421,"end_character":2},"in_reply_to":"6cb1539f_c85cc771","updated":"2024-03-03 15:19:11.000000000","message":"https://buildkite.com/docs/agent/v3/configuration#git-clean-flags\n\nSeems like the agent runs `git clean -fdxq` which would remove these symlinks.","commit_id":"f219cf0b05b7899687b5507928a1d78089395312"},{"author":{"_account_id":1000034,"name":"sterni","email":"sternenseemann@systemli.org","username":"sterni"},"change_message_id":"9c3e26df2a4535dca80ea4d24c36ac3d75ac446d","unresolved":true,"context_lines":[{"line_number":418,"context_line":"          echo \u0027--- Building extra step script\u0027"},{"line_number":419,"context_line":"          # Make sure the script will be ignored by git(1), as some extra steps"},{"line_number":420,"context_line":"          # check whether the tree is dirty or not."},{"line_number":421,"context_line":"          echo \u0027/${commandScriptLink}\u0027 \u003e\u003e .git/info/exclude"},{"line_number":422,"context_line":"          command_script\u003d\"$(${"},{"line_number":423,"context_line":"            # Using command substitution in this way assumes the script drv only has one output"},{"line_number":424,"context_line":"            assert builtins.length cfg.command.outputs \u003d\u003d 1;"}],"source_content_type":"text/x-nix","patch_set":8,"id":"6cb1539f_c85cc771","line":421,"range":{"start_line":421,"start_character":0,"end_line":421,"end_character":2},"in_reply_to":"a7292301_0857af14","updated":"2024-03-03 15:14:25.000000000","message":"Added to .gitignore.\n\nNo clue what buildkite does here, should look it up. If it doesn\u0027t delete these we have the same problem with `needsOutput`.","commit_id":"f219cf0b05b7899687b5507928a1d78089395312"}]}
