[PATCH] scripts/clang-tools: remove unused module

yangxingwu posted 1 patch 3 years, 6 months ago
scripts/clang-tools/run-clang-tools.py | 1 -
1 file changed, 1 deletion(-)
[PATCH] scripts/clang-tools: remove unused module
Posted by yangxingwu 3 years, 6 months ago
remove unused imported 'os' module

Signed-off-by: yangxingwu <xingwu.yang@gmail.com>
---
 scripts/clang-tools/run-clang-tools.py | 1 -
 1 file changed, 1 deletion(-)

diff --git a/scripts/clang-tools/run-clang-tools.py b/scripts/clang-tools/run-clang-tools.py
index 1337cedca..bb78c9bde 100755
--- a/scripts/clang-tools/run-clang-tools.py
+++ b/scripts/clang-tools/run-clang-tools.py
@@ -12,7 +12,6 @@ compile_commands.json.
 import argparse
 import json
 import multiprocessing
-import os
 import subprocess
 import sys

--
2.37.2
Re: [PATCH] scripts/clang-tools: remove unused module
Posted by Nathan Chancellor 3 years, 6 months ago
On Tue, Sep 13, 2022 at 04:07:52AM +0000, yangxingwu wrote:
> remove unused imported 'os' module
> 
> Signed-off-by: yangxingwu <xingwu.yang@gmail.com>

Sure.

Reviewed-by: Nathan Chancellor <nathan@kernel.org>

Masahiro, are you able to pick this up? The original was not cc'd to
linux-kbuild but it is on lore:

https://lore.kernel.org/20220913040753.2198-1-xingwu.yang@gmail.com/

> ---
>  scripts/clang-tools/run-clang-tools.py | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/scripts/clang-tools/run-clang-tools.py b/scripts/clang-tools/run-clang-tools.py
> index 1337cedca..bb78c9bde 100755
> --- a/scripts/clang-tools/run-clang-tools.py
> +++ b/scripts/clang-tools/run-clang-tools.py
> @@ -12,7 +12,6 @@ compile_commands.json.
>  import argparse
>  import json
>  import multiprocessing
> -import os
>  import subprocess
>  import sys
> 
> --
> 2.37.2
>
Re: [PATCH] scripts/clang-tools: remove unused module
Posted by Masahiro Yamada 3 years, 6 months ago
On Tue, Sep 13, 2022 at 6:50 PM Nathan Chancellor <nathan@kernel.org> wrote:
>
> On Tue, Sep 13, 2022 at 04:07:52AM +0000, yangxingwu wrote:
> > remove unused imported 'os' module
> >
> > Signed-off-by: yangxingwu <xingwu.yang@gmail.com>
>
> Sure.
>
> Reviewed-by: Nathan Chancellor <nathan@kernel.org>
>
> Masahiro, are you able to pick this up? The original was not cc'd to
> linux-kbuild but it is on lore:
>
> https://lore.kernel.org/20220913040753.2198-1-xingwu.yang@gmail.com/


Sure, applied to linux-kbuild. Thanks.


> > ---
> >  scripts/clang-tools/run-clang-tools.py | 1 -
> >  1 file changed, 1 deletion(-)
> >
> > diff --git a/scripts/clang-tools/run-clang-tools.py b/scripts/clang-tools/run-clang-tools.py
> > index 1337cedca..bb78c9bde 100755
> > --- a/scripts/clang-tools/run-clang-tools.py
> > +++ b/scripts/clang-tools/run-clang-tools.py
> > @@ -12,7 +12,6 @@ compile_commands.json.
> >  import argparse
> >  import json
> >  import multiprocessing
> > -import os
> >  import subprocess
> >  import sys
> >
> > --
> > 2.37.2
> >



-- 
Best Regards
Masahiro Yamada
[PATCH] scripts/clang-tools: remove unused module
Posted by yangxingwu 3 years, 6 months ago
remove unused imported 'os' module

Signed-off-by: yangxingwu <xingwu.yang@gmail.com>
---
 scripts/clang-tools/run-clang-tools.py | 1 -
 1 file changed, 1 deletion(-)

diff --git a/scripts/clang-tools/run-clang-tools.py b/scripts/clang-tools/run-clang-tools.py
index 1337cedca..bb78c9bde 100755
--- a/scripts/clang-tools/run-clang-tools.py
+++ b/scripts/clang-tools/run-clang-tools.py
@@ -12,7 +12,6 @@ compile_commands.json.
 import argparse
 import json
 import multiprocessing
-import os
 import subprocess
 import sys

--
2.37.2