)]}'
{
  "commit": "43d7563d1ac5b8d9e1f05d94c48b9eb9aecb5faa",
  "tree": "f68c2fd0e25722f92029b9dde90c945cf4c37211",
  "parents": [
    "3cf7368a05f49c800102d8bb3f8fb4e3cb2f04b9"
  ],
  "author": {
    "name": "waltl",
    "email": "waltl@google.com",
    "time": "Fri Mar 19 10:54:13 2021 -0700"
  },
  "committer": {
    "name": "Copybara-Service",
    "email": "copybara-worker@google.com",
    "time": "Fri Mar 19 10:56:42 2021 -0700"
  },
  "message": "Don\u0027t generate some module maps that are not needed\n\nThis is effectively an NFC.\n\nThe motivation is to not generate module maps when they are not used,\nand also to minimize the number of actions created.\n\nThe two things I did are:\n1. Don\u0027t unconditionally enable the module_maps feature.  This allows\n   the crosstool to turn off the feature when it\u0027s not needed.\n2. Restructure the module map creating logic to minimize the number of\n   actions created:\n   - If module_map feature is off, generate a swift module map in the\n     main CcCompilationHelper.compile() call.\n   - If module_map feature is on, generate a layering check and a\n     swift module map.  In this case, the layering check module map\n     must be the primary one integrated into the main\n     CcCompilationHelper.compile() call that generates the CcInfo.\n\nPiperOrigin-RevId: 363925797\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "3f503fa89e805f8b554d6ae3a9cd3bb32e683457",
      "old_mode": 33188,
      "old_path": "src/main/java/com/google/devtools/build/lib/rules/objc/CompilationSupport.java",
      "new_id": "34a33b1f8f9beb578085e34e0d74e3cd2d92828b",
      "new_mode": 33188,
      "new_path": "src/main/java/com/google/devtools/build/lib/rules/objc/CompilationSupport.java"
    },
    {
      "type": "modify",
      "old_id": "168e4286948bcf65bd999629401a17e104dfd793",
      "old_mode": 33188,
      "old_path": "src/main/java/com/google/devtools/build/lib/rules/objc/IntermediateArtifacts.java",
      "new_id": "410ecb5065f2b0408b31ace41e877e48fe78a5da",
      "new_mode": 33188,
      "new_path": "src/main/java/com/google/devtools/build/lib/rules/objc/IntermediateArtifacts.java"
    },
    {
      "type": "modify",
      "old_id": "2653625bf33b9137099d62af098fe82145ebe51e",
      "old_mode": 33188,
      "old_path": "src/main/java/com/google/devtools/build/lib/rules/objc/ObjcCommon.java",
      "new_id": "42054e6d7e9a90587cbf3e8dc456b7a5c191d279",
      "new_mode": 33188,
      "new_path": "src/main/java/com/google/devtools/build/lib/rules/objc/ObjcCommon.java"
    },
    {
      "type": "modify",
      "old_id": "7b1c8b6c78d652a601922feb456a9d0a73159db2",
      "old_mode": 33188,
      "old_path": "src/main/java/com/google/devtools/build/lib/rules/objc/ObjcVariablesExtension.java",
      "new_id": "d2926469c8b9f4a34ac6113e0c7bb63571144d45",
      "new_mode": 33188,
      "new_path": "src/main/java/com/google/devtools/build/lib/rules/objc/ObjcVariablesExtension.java"
    }
  ]
}