public inbox for llvm@lists.linux.dev
 help / color / mirror / Atom feed
* [PATCH] scripts/clang-tools: remove unused module
@ 2022-09-13  4:07 yangxingwu
  2022-09-13  4:07 ` yangxingwu
  2022-09-13  9:50 ` Nathan Chancellor
  0 siblings, 2 replies; 4+ messages in thread
From: yangxingwu @ 2022-09-13  4:07 UTC (permalink / raw)
  To: nathan; +Cc: ndesaulniers, trix, llvm, linux-kernel, yangxingwu

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


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* [PATCH] scripts/clang-tools: remove unused module
  2022-09-13  4:07 [PATCH] scripts/clang-tools: remove unused module yangxingwu
@ 2022-09-13  4:07 ` yangxingwu
  2022-09-13  9:50 ` Nathan Chancellor
  1 sibling, 0 replies; 4+ messages in thread
From: yangxingwu @ 2022-09-13  4:07 UTC (permalink / raw)
  To: nathan; +Cc: ndesaulniers, trix, llvm, linux-kernel, yangxingwu

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


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH] scripts/clang-tools: remove unused module
  2022-09-13  4:07 [PATCH] scripts/clang-tools: remove unused module yangxingwu
  2022-09-13  4:07 ` yangxingwu
@ 2022-09-13  9:50 ` Nathan Chancellor
  2022-09-13 14:04   ` Masahiro Yamada
  1 sibling, 1 reply; 4+ messages in thread
From: Nathan Chancellor @ 2022-09-13  9:50 UTC (permalink / raw)
  To: yangxingwu, Masahiro Yamada
  Cc: ndesaulniers, trix, llvm, linux-kernel, linux-kbuild

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
> 

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] scripts/clang-tools: remove unused module
  2022-09-13  9:50 ` Nathan Chancellor
@ 2022-09-13 14:04   ` Masahiro Yamada
  0 siblings, 0 replies; 4+ messages in thread
From: Masahiro Yamada @ 2022-09-13 14:04 UTC (permalink / raw)
  To: Nathan Chancellor
  Cc: yangxingwu, Nick Desaulniers, Tom Rix, clang-built-linux,
	Linux Kernel Mailing List, Linux Kbuild mailing list

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

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2022-09-13 14:05 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2022-09-13  4:07 [PATCH] scripts/clang-tools: remove unused module yangxingwu
2022-09-13  4:07 ` yangxingwu
2022-09-13  9:50 ` Nathan Chancellor
2022-09-13 14:04   ` Masahiro Yamada

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox