[PATCH v2] kbuild: clean `*.long-type-*.txt` files

Joel Kamminga posted 1 patch 1 week, 2 days ago
Makefile | 1 +
1 file changed, 1 insertion(+)
[PATCH v2] kbuild: clean `*.long-type-*.txt` files
Posted by Joel Kamminga 1 week, 2 days ago
This cleans up files generated by rustc compiler in the case of an
error containing an excessively long type name that doesn't fit in
a single line. Such types appear relatively frequently so the risk
of generating these files certainly exists. These files are purely
compiler artifacts and are not created intentionally by the build
system. They should be added to the `clean` target to stop from
cluttering up the source tree.

Suggested-by: Miguel Ojeda <ojeda@kernel.org>
Link: https://github.com/Rust-for-Linux/linux/issues/1236
Signed-off-by: Joel Kamminga <contact@jkam.dev>
---
 Makefile | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Makefile b/Makefile
index 9f59598d3a08..3a265e7e3347 100644
--- a/Makefile
+++ b/Makefile
@@ -2164,6 +2164,7 @@ clean: $(clean-dirs)
 		-o -name '*.c.[012]*.*' \
 		-o -name '*.ll' \
 		-o -name '*.gcno' \
+		-o -name '*.long-type-*.txt' \
 		\) -type f -print \
 		-o -name '.tmp_*' -print \
 		| xargs rm -rf
-- 
2.54.0
Re: [PATCH v2] kbuild: clean `*.long-type-*.txt` files
Posted by Miguel Ojeda 1 week, 2 days ago
On Fri, May 29, 2026 at 10:55 PM Joel Kamminga <contact@jkam.dev> wrote:
>
> This cleans up files generated by rustc compiler in the case of an
> error containing an excessively long type name that doesn't fit in
> a single line. Such types appear relatively frequently so the risk
> of generating these files certainly exists. These files are purely
> compiler artifacts and are not created intentionally by the build
> system. They should be added to the `clean` target to stop from
> cluttering up the source tree.

Normally, we try to use the imperative for the actual change in commit
messages, i.e. "as if you are giving orders to the codebase to change
its behaviour":

  https://docs.kernel.org/process/submitting-patches.html#describe-your-changes

I would also probably add "rust: " after "kbuild: " in the title to
scope it a bit more, since these files are `rustc`'s ones.

Otherwise, it looks good, thanks! :)

I can take this through Rust since I have Manos' there too, unless
others prefer otherwise.

Cheers,
Miguel