ANDROID: Reland "gen_compile_commands.py: fix path resolve with symlinks in it"

Ulises Mendez Martinez · Treehugger Robot · commit b41110b01310 · 2024-02-08T10:16:39.000Z
This reverts commit 2285818. Reason for revert: Re-landing after hacking Kleaf script. Bug: 320449031 Change-Id: I8aa2783b77f13859798cf7b075e3f1656038315e Signed-off-by: Ulises Mendez Martinez <umendez@google.com>
diff --git a/scripts/clang-tools/gen_compile_commands.py b/scripts/clang-tools/gen_compile_commands.py
@@ -64,7 +64,7 @@ def parse_arguments():
     args = parser.parse_args()
 
     return (args.log_level,
-            os.path.abspath(args.directory),
+            os.path.realpath(args.directory),
             args.output,
             args.ar,
             args.paths if len(args.paths) > 0 else [args.directory])
@@ -172,8 +172,8 @@ def process_line(root_directory, command_prefix, file_path):
     # by Make, so this code replaces the escaped version with '#'.
     prefix = command_prefix.replace('\#', '#').replace('$(pound)', '#')
 
-    # Use os.path.abspath() to normalize the path resolving '.' and '..' .
-    abs_path = os.path.abspath(os.path.join(root_directory, file_path))
+    # Return the canonical path, eliminating any symbolic links encountered in the path.
+    abs_path = os.path.realpath(os.path.join(root_directory, file_path))
     if not os.path.exists(abs_path):
         raise ValueError('File %s not found' % abs_path)
     return {
