)]}'
{
  "commit": "64d9713637ab98e2b65c9c4317a50ddba0ba0dbc",
  "tree": "47d9559f8758b99148574bd559fd8f0df7ade0c5",
  "parents": [
    "8fc3a7974701f12f46f3f7c1f967001b0cb22847"
  ],
  "author": {
    "name": "kadir çetinkaya",
    "email": "kadircet@google.com",
    "time": "Tue Aug 13 12:51:52 2024 +0200"
  },
  "committer": {
    "name": "GitHub",
    "email": "noreply@github.com",
    "time": "Tue Aug 13 12:51:52 2024 +0200"
  },
  "message": "[include-cleaner] Unify symlink handling (#102615)\n\nWe were using tryGetRealPathName in certain places, which resolves\r\nsymlinks (sometimes). This was resulting in discrepancies in behavior,\r\ndepending on how a file was first reached.\r\n\r\nThis path migrates all usages of tryGetRealPathName to regular getName\r\ninstead.\r\n\r\nThis implies one backward incompatible change for header-filtering. Our\r\nignore-header option used to filter against suffixes of absolute paths,\r\nwhereas now filter can receive working-directory relative paths in some\r\ncases, possibly braking existing filters.\r\nChances of really braking users is pretty low:\r\n- We\u0027ll still filter against absolute paths when header is outside the\r\n  working directory (e.g. /usr/bin/include/some/linux/header.h.)\r\n- Most projects run builds in a working directory that\u0027s nested inside\r\n  the repository, hence relative paths still contain all the segments\r\n  relative to repository root and anything else is unlikely to be\r\n  meaningful. e.g. if a header is in\r\n  `$HOME/work/llvm-project/clang-tools-extra/header.h` with builds being\r\n  run in `$home/work/llvm-project/build`, we\u0027ll still filter against\r\n  `../clang-tools-extra/header.h` which has all the useful segments as a\r\n  suffix.\r\n- This is also a change in how we handle symlinks, but this is aligned\r\n  with what we do in rest of our tools (clangd, tidy checks etc.). We\r\n  tend to not resolve any symlinks for the file.",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "7d5c19872d5a851b1405f2ebdb43a5d2eca9e509",
      "old_mode": 33188,
      "old_path": "clang-tools-extra/include-cleaner/include/clang-include-cleaner/Types.h",
      "new_id": "2888e252267556109fd33b4f77a44778cfcf197a",
      "new_mode": 33188,
      "new_path": "clang-tools-extra/include-cleaner/include/clang-include-cleaner/Types.h"
    },
    {
      "type": "modify",
      "old_id": "68fe79d6929f6bdc1a790375c22fe0b5f43c75d4",
      "old_mode": 33188,
      "old_path": "clang-tools-extra/include-cleaner/lib/Analysis.cpp",
      "new_id": "05e9d14734a95ff4c4951d3baab4d7002c2e90e6",
      "new_mode": 33188,
      "new_path": "clang-tools-extra/include-cleaner/lib/Analysis.cpp"
    },
    {
      "type": "modify",
      "old_id": "195f658a0af9208684862276ff5c51e9d332a5a9",
      "old_mode": 33188,
      "old_path": "clang-tools-extra/include-cleaner/lib/HTMLReport.cpp",
      "new_id": "bbe8bc230c6e2016acd2b46600909ef68f1be098",
      "new_mode": 33188,
      "new_path": "clang-tools-extra/include-cleaner/lib/HTMLReport.cpp"
    },
    {
      "type": "modify",
      "old_id": "cb8a55ed13e5d00a0ecd989bbe665d1202e2ad81",
      "old_mode": 33188,
      "old_path": "clang-tools-extra/include-cleaner/lib/Types.cpp",
      "new_id": "7a637639edf8b5a139d3dc780e57f49dcf20eeca",
      "new_mode": 33188,
      "new_path": "clang-tools-extra/include-cleaner/lib/Types.cpp"
    },
    {
      "type": "modify",
      "old_id": "3bc449b0152bba1a1eb4b174c09b1e4d43422272",
      "old_mode": 33188,
      "old_path": "clang-tools-extra/include-cleaner/tool/IncludeCleaner.cpp",
      "new_id": "d8a44ab9b6e12ec88147104e2d968d51624f1130",
      "new_mode": 33188,
      "new_path": "clang-tools-extra/include-cleaner/tool/IncludeCleaner.cpp"
    },
    {
      "type": "modify",
      "old_id": "5696c380758f850a6fa2beae841c85c40799b87c",
      "old_mode": 33188,
      "old_path": "clang-tools-extra/include-cleaner/unittests/AnalysisTest.cpp",
      "new_id": "43634ee8f2d8031eade4e9a5c5726a181af822ea",
      "new_mode": 33188,
      "new_path": "clang-tools-extra/include-cleaner/unittests/AnalysisTest.cpp"
    },
    {
      "type": "modify",
      "old_id": "8f6ad09c46cc4a5120d09bf5850eeacffe053321",
      "old_mode": 33188,
      "old_path": "clang-tools-extra/include-cleaner/unittests/IncludeSpellerTest.cpp",
      "new_id": "a27e83a434372fdfe9f5329acc6e9887225645af",
      "new_mode": 33188,
      "new_path": "clang-tools-extra/include-cleaner/unittests/IncludeSpellerTest.cpp"
    },
    {
      "type": "modify",
      "old_id": "1a5996e5df284b504483eef657a2acd8b755a8a2",
      "old_mode": 33188,
      "old_path": "clang-tools-extra/include-cleaner/unittests/RecordTest.cpp",
      "new_id": "bcb9f8982c461a1f5a5ef87728c8c020af7fe064",
      "new_mode": 33188,
      "new_path": "clang-tools-extra/include-cleaner/unittests/RecordTest.cpp"
    }
  ]
}