[PATCH] scripts: gen_compile_commands: fix invalid escape sequence warning

João Marcos Costa jmcosta944 at gmail.com
Fri May 3 18:03:36 CEST 2024


Hello,
thanks for this fix!

Em sex., 3 de mai. de 2024 às 15:20, Caleb Connolly <
caleb.connolly at linaro.org> escreveu:

> Since Python 3.12 unrecognised escape sequences trigger a SyntaxWarning.
> Convert the '\#' string to a raw string so the backslash is correctly
> used as a literal.
>
> Ported from Linux commit dae4a0171e25 ("gen_compile_commands: fix invalid
> escape sequence warning").
>
> This updates the script to be in-line with Linux 6.9-rc6.
>
> Signed-off-by: Caleb Connolly <caleb.connolly at linaro.org>
> ---
>  scripts/gen_compile_commands.py | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/scripts/gen_compile_commands.py
> b/scripts/gen_compile_commands.py
> index fec513e55474..e746adddd929 100755
> --- a/scripts/gen_compile_commands.py
> +++ b/scripts/gen_compile_commands.py
> @@ -171,9 +171,9 @@ def process_line(root_directory, command_prefix,
> file_path):
>      # The .cmd files are intended to be included directly by Make, so they
>      # escape the pound sign '#', either as '\#' or '$(pound)' (depending
> on the
>      # kernel version). The compile_commands.json file is not interepreted
>      # by Make, so this code replaces the escaped version with '#'.
> -    prefix = command_prefix.replace('\#', '#').replace('$(pound)', '#')
> +    prefix = command_prefix.replace(r'\#', '#').replace('$(pound)', '#')
>
>      # 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):
> --
> 2.45.0
>
>
Reviewed-by: João Marcos Costa <jmcosta944 at gmail.com>

-- 
Atenciosamente,
João Marcos Costa


More information about the U-Boot mailing list