)]}'
{
  "commit": "0cbe341c1a28037ee32d21b589030a7df0b9fdab",
  "tree": "8d13e5710580015d368cadb6e601785dc7d17506",
  "parents": [
    "cdb4ad4f61a32139a02a63d0ba2044ac3cb8705f"
  ],
  "author": {
    "name": "David \u0027Digit\u0027 Turner",
    "email": "digit@google.com",
    "time": "Tue Apr 12 12:20:10 2022 +0200"
  },
  "committer": {
    "name": "Commit Bot",
    "email": "commit-bot@chromium.org",
    "time": "Wed Apr 13 15:19:34 2022 +0000"
  },
  "message": "Fix NinjaCBinaryTargetWriter unit-test\n\nThe NinjaCBinaryTargetWriter.RustDepsOverDynamicLinking\nunit-test defined two Rust rlib targets with public\ndependencies to themselves!\n\nThis does not make any sense, and is not allowed by the\nGN parser itself, nor is it required by the test, so remove\nthese definitions from the code\n\nFor the record, the issue was done while working on some\nrefactor of target dependency computations, where this\nwould trigger a stack overflow / infinite recursion.\n\nBug: None\n\nChange-Id: I9f1c2fe31bc3ec881446c6a16ce2508c65e2fc03\nReviewed-on: https://gn-review.googlesource.com/c/gn/+/13601\nReviewed-by: danakj \u003cdanakj@chromium.org\u003e\nReviewed-by: Sylvain Defresne \u003csdefresne@chromium.org\u003e\nCommit-Queue: David Turner \u003cdigit@google.com\u003e\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "e4ec8006bd45f57bb246c0c6d790ee5b8911a978",
      "old_mode": 33188,
      "old_path": "src/gn/ninja_c_binary_target_writer_unittest.cc",
      "new_id": "ff51121f80bf42f4e9825c9da68a7e32b6f8e368",
      "new_mode": 33188,
      "new_path": "src/gn/ninja_c_binary_target_writer_unittest.cc"
    }
  ]
}
