* [PATCH] .gitignore: ignore smatch generated files
@ 2023-04-27 2:33 Brandon Pollack
2023-04-27 7:54 ` Miguel Ojeda
0 siblings, 1 reply; 7+ messages in thread
From: Brandon Pollack @ 2023-04-27 2:33 UTC (permalink / raw)
To: linux-kernel
Cc: masahiroy, ojeda, alex.gaynor, keescook, nathan, Brandon Pollack
ignore smatch generated files.
Signed-off-by: Brandon Pollack <brpol@chromium.org>
---
New to development and looking for tasks to work on to get familiar with
code, I noticed smatch generates some files at the root. Since this
tool seems frequent for newbies like myself I thought it would be worth
getting rid of the chance of accidentally committing these files.
---
.gitignore | 8 ++++++++
1 file changed, 8 insertions(+)
diff --git a/.gitignore b/.gitignore
index 70ec6037fa7a..04472c37b6df 100644
--- a/.gitignore
+++ b/.gitignore
@@ -173,3 +173,11 @@ sphinx_*/
# Rust analyzer configuration
/rust-project.json
+
+# Smatch tree outputs
+smatch_compile.warns
+smatch_db.sqlite
+smatch_warns.txt
+smatch_warns.txt.caller_info
+smatch_warns.txt.sql
+trinity_smatch.[ch]
--
2.40.1.495.gc816e09b53d-goog
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH] .gitignore: ignore smatch generated files
2023-04-27 2:33 [PATCH] .gitignore: ignore smatch generated files Brandon Pollack
@ 2023-04-27 7:54 ` Miguel Ojeda
2023-04-27 11:20 ` Dan Carpenter
0 siblings, 1 reply; 7+ messages in thread
From: Miguel Ojeda @ 2023-04-27 7:54 UTC (permalink / raw)
To: Brandon Pollack, Dan Carpenter
Cc: linux-kernel, masahiroy, ojeda, alex.gaynor, keescook, nathan
Hi Brandon,
On Thu, Apr 27, 2023 at 4:33 AM Brandon Pollack <brpol@chromium.org> wrote:
>
> ignore smatch generated files.
Cc'ing Dan who is the `smatch` author.
Also, when sending a v2 of a series, please mark it as such --
`git-format-patch` can do it for you with e.g. `-v2`.
Cheers,
Miguel
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] .gitignore: ignore smatch generated files
2023-04-27 7:54 ` Miguel Ojeda
@ 2023-04-27 11:20 ` Dan Carpenter
2023-04-28 2:55 ` Brandon Ross Pollack
0 siblings, 1 reply; 7+ messages in thread
From: Dan Carpenter @ 2023-04-27 11:20 UTC (permalink / raw)
To: Miguel Ojeda
Cc: Brandon Pollack, linux-kernel, masahiroy, ojeda, alex.gaynor,
keescook, nathan
Thanks for this. To be honest, I hadn't remembered that Smatch
still generates trinity_smatch.[ch]. I would prefer to just delete that
stuff. Another idea is maybe Smatch could put everything in a
smatch/ directory? That feels like maybe it would be nicer?
regards,
dan carpenter
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] .gitignore: ignore smatch generated files
2023-04-27 11:20 ` Dan Carpenter
@ 2023-04-28 2:55 ` Brandon Ross Pollack
2023-05-02 15:24 ` Dan Carpenter
0 siblings, 1 reply; 7+ messages in thread
From: Brandon Ross Pollack @ 2023-04-28 2:55 UTC (permalink / raw)
To: Dan Carpenter
Cc: Miguel Ojeda, linux-kernel, masahiroy, ojeda, alex.gaynor,
keescook, nathan
On Thu, Apr 27, 2023 at 8:20 PM Dan Carpenter <error27@gmail.com> wrote:
>
> Thanks for this. To be honest, I hadn't remembered that Smatch
> still generates trinity_smatch.[ch]. I would prefer to just delete that
> stuff. Another idea is maybe Smatch could put everything in a
> smatch/ directory? That feels like maybe it would be nicer?
Sure, that'd be great as well. I'm pretty sure that smatch outputs
these files from whatever directory you run from, that's why I did the
overall pattern rather than a path pattern.
Should we just patch smatch to not output trinity* files and output to
a directory and change this patch to mask out the directory?
>
> regards,
> dan carpenter
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] .gitignore: ignore smatch generated files
2023-04-28 2:55 ` Brandon Ross Pollack
@ 2023-05-02 15:24 ` Dan Carpenter
[not found] ` <CALWYx-ayGvg0iuRi15+JyGrThmiOqzwxPNfEfpqUpbsknkcpgg@mail.gmail.com>
0 siblings, 1 reply; 7+ messages in thread
From: Dan Carpenter @ 2023-05-02 15:24 UTC (permalink / raw)
To: Brandon Ross Pollack
Cc: Dan Carpenter, Miguel Ojeda, linux-kernel, masahiroy, ojeda,
alex.gaynor, keescook, nathan
On Fri, Apr 28, 2023 at 11:55:17AM +0900, Brandon Ross Pollack wrote:
> On Thu, Apr 27, 2023 at 8:20 PM Dan Carpenter <error27@gmail.com> wrote:
> >
> > Thanks for this. To be honest, I hadn't remembered that Smatch
> > still generates trinity_smatch.[ch]. I would prefer to just delete that
> > stuff. Another idea is maybe Smatch could put everything in a
> > smatch/ directory? That feels like maybe it would be nicer?
>
> Sure, that'd be great as well. I'm pretty sure that smatch outputs
> these files from whatever directory you run from, that's why I did the
> overall pattern rather than a path pattern.
>
> Should we just patch smatch to not output trinity* files and output to
> a directory and change this patch to mask out the directory?
I've deleted the trinity stuff. Let's just apply the rest as-is.
regards,
dan carpenter
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] .gitignore: ignore smatch generated files
[not found] ` <CALWYx-ayGvg0iuRi15+JyGrThmiOqzwxPNfEfpqUpbsknkcpgg@mail.gmail.com>
@ 2023-05-08 5:04 ` Dan Carpenter
0 siblings, 0 replies; 7+ messages in thread
From: Dan Carpenter @ 2023-05-08 5:04 UTC (permalink / raw)
To: Brandon Ross Pollack
Cc: Dan Carpenter, Miguel Ojeda, linux-kernel, masahiroy, ojeda,
alex.gaynor, keescook, nathan
On Mon, May 08, 2023 at 10:20:48AM +0900, Brandon Ross Pollack wrote:
> This is actually my first patch, is there anything else I need to do to
> apply it or should I let the maintainers handle it from here?
>
You need to send a v2 patch without the trinity stuff. (I removed it
from Smatch already).
https://staticthinking.wordpress.com/2022/07/27/how-to-send-a-v2-patch/
regards,
dan carpenter
^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH] .gitignore: ignore smatch generated files
@ 2023-04-26 3:46 Brandon Pollack
0 siblings, 0 replies; 7+ messages in thread
From: Brandon Pollack @ 2023-04-26 3:46 UTC (permalink / raw)
To: linux-kernel
Cc: masahiroy, ojeda, alex.gaynor, keescook, wedsonaf, Brandon Pollack
ignore smatch generated files.
Change-Id: Id4f2d5c4ea2b4819b4634a8db6fe352b5e38e1f2
---
New to development and looking for tasks to work on to get familiar with
code, I noticed smatch generates some files at the root. Since this
tool seems frequent for newbies like myself I thought it would be worth
getting rid of the chance of accidentally committing these files.
---
.gitignore | 8 ++++++++
1 file changed, 8 insertions(+)
diff --git a/.gitignore b/.gitignore
index 70ec6037fa7a..04472c37b6df 100644
--- a/.gitignore
+++ b/.gitignore
@@ -173,3 +173,11 @@ sphinx_*/
# Rust analyzer configuration
/rust-project.json
+
+# Smatch tree outputs
+smatch_compile.warns
+smatch_db.sqlite
+smatch_warns.txt
+smatch_warns.txt.caller_info
+smatch_warns.txt.sql
+trinity_smatch.[ch]
--
2.40.1.495.gc816e09b53d-goog
^ permalink raw reply related [flat|nested] 7+ messages in thread
end of thread, other threads:[~2023-05-08 5:15 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-04-27 2:33 [PATCH] .gitignore: ignore smatch generated files Brandon Pollack
2023-04-27 7:54 ` Miguel Ojeda
2023-04-27 11:20 ` Dan Carpenter
2023-04-28 2:55 ` Brandon Ross Pollack
2023-05-02 15:24 ` Dan Carpenter
[not found] ` <CALWYx-ayGvg0iuRi15+JyGrThmiOqzwxPNfEfpqUpbsknkcpgg@mail.gmail.com>
2023-05-08 5:04 ` Dan Carpenter
-- strict thread matches above, loose matches on Subject: below --
2023-04-26 3:46 Brandon Pollack
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.