)]}'
{
  "commit": "3c2c821e2e7d0bee29f1ae4c13aaa37626d0d76c",
  "tree": "29261b240ba0c1843b57779097a98ed1f2b5c965",
  "parents": [
    "4876aa484e4a38931b46f70196665db099e7ef59"
  ],
  "author": {
    "name": "lberki",
    "email": "lberki@google.com",
    "time": "Wed Oct 21 04:00:04 2020 -0700"
  },
  "committer": {
    "name": "Copybara-Service",
    "email": "copybara-worker@google.com",
    "time": "Wed Oct 21 04:01:20 2020 -0700"
  },
  "message": "Remove the distinction between visible and non-visible aspects in AspectCollection.\n\nThis has not been used in the code since unknown commit. I don\u0027t claim I understood this completely when I authored that change, but it turns out that the distinction wasn\u0027t necessary in the first place.\n\nAspectFunction can have incoming dependencies in two ways: first, when traversing down the configured target graph, second, when traversing the graph of aspects attached to a single configured target.\n\nIn the first case, for reasons of consistency, if another aspect is visible on the configured target attached the aspect is attached to (in the \"target\" argument of ctx), it should also be visible on the direct dependencies, too.\n\nIn the second case, When multiple aspects are attached to a configured target and an outer aspect is not interested in an inner one, they are already filtered out when the AspectCollection is computed when traversing down the configured target graph.\n\nThe test cases that needed to be updated were those when AspectCollection has a different set of propagated and visible aspects, which is not possible anymore.\n\nAlso update comments with the newly acquired knowledge.\n\nRELNOTES: None.\nPiperOrigin-RevId: 338235579\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "0966464e874a3bd6e2230fbb5cb2ce8b7abf816a",
      "old_mode": 33188,
      "old_path": "src/main/java/com/google/devtools/build/lib/analysis/AspectCollection.java",
      "new_id": "0cbc338b22697f436003fe383129dbfd5c8f5a7d",
      "new_mode": 33188,
      "new_path": "src/main/java/com/google/devtools/build/lib/analysis/AspectCollection.java"
    },
    {
      "type": "modify",
      "old_id": "015ac7109671e4eed72eea4cb31596469332b5a8",
      "old_mode": 33188,
      "old_path": "src/main/java/com/google/devtools/build/lib/analysis/AspectResolver.java",
      "new_id": "201442b96102a1cc7c9e1c87cfa6d0761e15fbab",
      "new_mode": 33188,
      "new_path": "src/main/java/com/google/devtools/build/lib/analysis/AspectResolver.java"
    },
    {
      "type": "modify",
      "old_id": "317635614ef8a7214bc1ec576edc2b56810c71c9",
      "old_mode": 33188,
      "old_path": "src/main/java/com/google/devtools/build/lib/analysis/BUILD",
      "new_id": "1b80335f6235875ed7ddafa904623e4b11fef634",
      "new_mode": 33188,
      "new_path": "src/main/java/com/google/devtools/build/lib/analysis/BUILD"
    },
    {
      "type": "modify",
      "old_id": "61d6120179680a9a92f787113cc2897c5a3315ff",
      "old_mode": 33188,
      "old_path": "src/main/java/com/google/devtools/build/lib/analysis/DependencyResolver.java",
      "new_id": "faf4109885eca6004fc18d7f937d1c665e96dc70",
      "new_mode": 33188,
      "new_path": "src/main/java/com/google/devtools/build/lib/analysis/DependencyResolver.java"
    },
    {
      "type": "modify",
      "old_id": "4732ca22f59fe9dc36dca50123ab322e5bd79718",
      "old_mode": 33188,
      "old_path": "src/main/java/com/google/devtools/build/lib/skyframe/BUILD",
      "new_id": "16261b97be986b052fd7cf205404964b08aaa644",
      "new_mode": 33188,
      "new_path": "src/main/java/com/google/devtools/build/lib/skyframe/BUILD"
    },
    {
      "type": "modify",
      "old_id": "59ce404ed20dccd766eb957f0e3fd3f16d9d19d7",
      "old_mode": 33188,
      "old_path": "src/main/java/com/google/devtools/build/lib/skyframe/SkyframeExecutor.java",
      "new_id": "10d7b1165a50d9401068ecab1adc1ab44cf098ed",
      "new_mode": 33188,
      "new_path": "src/main/java/com/google/devtools/build/lib/skyframe/SkyframeExecutor.java"
    },
    {
      "type": "modify",
      "old_id": "c0ab5dda05728799c5d26fe01524ce6f6689b60f",
      "old_mode": 33188,
      "old_path": "src/test/java/com/google/devtools/build/lib/analysis/AspectCollectionTest.java",
      "new_id": "8b20ec5a2c22449f3185d8847fa9b059e9787ff1",
      "new_mode": 33188,
      "new_path": "src/test/java/com/google/devtools/build/lib/analysis/AspectCollectionTest.java"
    },
    {
      "type": "modify",
      "old_id": "e47551fb03fae3a7d69c608d8c616ce81fcdf0ce",
      "old_mode": 33188,
      "old_path": "src/test/java/com/google/devtools/build/lib/analysis/DependencyKeyTest.java",
      "new_id": "6df917aa1579702b2a31a567cecfe336435c350c",
      "new_mode": 33188,
      "new_path": "src/test/java/com/google/devtools/build/lib/analysis/DependencyKeyTest.java"
    },
    {
      "type": "modify",
      "old_id": "9ccac7ddceebca66aa68a45c2f7fcf1d0613878b",
      "old_mode": 33188,
      "old_path": "src/test/java/com/google/devtools/build/lib/analysis/DependencyResolverTest.java",
      "new_id": "6f1dc0fa8508d4107b4cd920a054d200e8f6df89",
      "new_mode": 33188,
      "new_path": "src/test/java/com/google/devtools/build/lib/analysis/DependencyResolverTest.java"
    },
    {
      "type": "modify",
      "old_id": "9e0688ea1bb5977850be2ee727e34415fb49eadf",
      "old_mode": 33188,
      "old_path": "src/test/java/com/google/devtools/build/lib/analysis/DependencyTest.java",
      "new_id": "96946c52d40043942f931410826fa318a7db6560",
      "new_mode": 33188,
      "new_path": "src/test/java/com/google/devtools/build/lib/analysis/DependencyTest.java"
    }
  ]
}