)]}'
{
  "commit": "231e86c10c674235cf28447a8486f7955d5f4dd9",
  "tree": "29a698f58cb2e00a6d0192bb56826082a104b9b6",
  "parents": [
    "1e3f26542a6ecd3006c2c0d5ccc0bae4a700f7e5"
  ],
  "author": {
    "name": "Phillip Wood",
    "email": "phillip.wood@dunelm.org.uk",
    "time": "Thu Aug 10 14:33:13 2023 +0000"
  },
  "committer": {
    "name": "Junio C Hamano",
    "email": "gitster@pobox.com",
    "time": "Thu Aug 10 09:16:27 2023 -0700"
  },
  "message": "t4053: avoid race when killing background processes\n\nThe test \u0027diff --no-index reads from pipes\u0027 starts a couple of\nbackground processes that write to the pipes that are passed to \"diff\n--no-index\". If the test passes then we expect these processes to exit\nas all their output will have been read. However if the test fails\nthen we want to make sure they do not hang about on the users machine\nand the test remembers they should be killed by calling\n\n      test_when_finished  \"! kill $!\"\n\nafter each background process is created. Unfortunately there is a\nrace where test_when_finished may run before the background process\nexits even when all its output has been read resulting in the kill\ncommand succeeding which causes the test to fail. Fix this by ignoring\nthe exit status of the kill command. If the diff is successful we\ncould instead wait for the background process to exit and check their\nstatus but that feels like it is testing the platform\u0027s printf\nimplementation rather than git\u0027s code.\n\nReported-by: Jeff King \u003cpeff@peff.net\u003e\nSigned-off-by: Phillip Wood \u003cphillip.wood@dunelm.org.uk\u003e\nSigned-off-by: Junio C Hamano \u003cgitster@pobox.com\u003e\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "a28b9ff243404ff426fbbb38f9be41ddb5528fb1",
      "old_mode": 33261,
      "old_path": "t/t4053-diff-no-index.sh",
      "new_id": "1fb7d334620767406225df21ede0b05697d4e64f",
      "new_mode": 33261,
      "new_path": "t/t4053-diff-no-index.sh"
    }
  ]
}