)]}'
{
  "commit": "e8efc031bf75fc2be2a595b4790a43b6b6cd36ea",
  "tree": "ea32ad251b2d9c1cacfe484bfd7a6205eb3e049e",
  "parents": [
    "7b0abdc591b9b0b0dbdfd62bf9d98928b0efe6a2"
  ],
  "author": {
    "name": "Jon Shea",
    "email": "1385+jonshea@users.noreply.github.com",
    "time": "Fri May 03 07:33:50 2024 +0200"
  },
  "committer": {
    "name": "GitHub",
    "email": "noreply@github.com",
    "time": "Fri May 03 13:33:50 2024 +0800"
  },
  "message": "Restore Target Uniqueness Guard in dependency_tree_parser.bzl (#1122)\n\nI noticed the refactor of `dependency_tree_parser.bzl` in\r\nhttps://github.com/bazelbuild/rules_jvm_external/pull/994 might have\r\nunintentionally omitted a line that added target labels to the set of\r\n`seen_imports`. This line is present in\r\n[v5.3](https://github.com/bazelbuild/rules_jvm_external/blob/5.3/private/dependency_tree_parser.bzl#L155-L156),\r\nbut then removed in\r\n[v6.1](https://github.com/bazelbuild/rules_jvm_external/blob/6.1/private/dependency_tree_parser.bzl#L457).\r\n\r\nI believe this removal was accidental, as it is now possible for\r\n`maven_install` to create a generated BUILD file which contains\r\nmultiple copies of a target with the same name, which is an invalid\r\nstate.\r\n\r\nThis PR restores the missing call to `seen_imports[target_label] \u003d True`.",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "0dcce37ee79a3f90c571a000b9df2df3c36ee46f",
      "old_mode": 33188,
      "old_path": "private/dependency_tree_parser.bzl",
      "new_id": "d6faf4cdc940bcfdce70ecccd69278f61881219d",
      "new_mode": 33188,
      "new_path": "private/dependency_tree_parser.bzl"
    }
  ]
}