Re: [PATCH] gen_compile_commands: add missing sys import

From: Nick Desaulniers
Date: Tue Sep 14 2021 - 12:01:46 EST


On Tue, Sep 14, 2021 at 8:44 AM <andreas@xxxxxxxxxxx> wrote:
>
> From: Andreas Rammhold <andreas@xxxxxxxxxxx>
>
> The sys.exit was being used at multiple locations within the script but
> never imported. This lead to the script exiting with a error instead of
> with the nicely formatted (useful) error output.
>
> Signed-off-by: Andreas Rammhold <andreas@xxxxxxxxxxx>

Hi Andreas,
Thanks for the patch. Someone beat you to the punch though:
https://lore.kernel.org/lkml/20210908032847.18683-1-kortanzh@xxxxxxxxx/.

This is good; it means folks are playing with this. :)

> ---
> scripts/clang-tools/gen_compile_commands.py | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/scripts/clang-tools/gen_compile_commands.py b/scripts/clang-tools/gen_compile_commands.py
> index 0033eedce003e..1d1bde1fd45eb 100755
> --- a/scripts/clang-tools/gen_compile_commands.py
> +++ b/scripts/clang-tools/gen_compile_commands.py
> @@ -13,6 +13,7 @@ import logging
> import os
> import re
> import subprocess
> +import sys
>
> _DEFAULT_OUTPUT = 'compile_commands.json'
> _DEFAULT_LOG_LEVEL = 'WARNING'
> --
> 2.32.0
>


--
Thanks,
~Nick Desaulniers