On Tue, Jan 17, 2023 at 11:24:47PM +0900, Masahiro Yamada wrote: > On Tue, Jan 17, 2023 at 10:50 PM Thomas Weißschuh <linux@xxxxxxxxxxxxxx> wrote: > > > > If the kheaders archive generation is interrupted then this directory > > may be left. > > Ignore it, it will be deleted by the next run of kernel/gen_kheaders.sh. > > Also add it to clean-files so it can be cleaned up by make. > > > > Signed-off-by: Thomas Weißschuh <linux@xxxxxxxxxxxxxx> > > > Or, just rename the temporary directory. > For example, > > cpio_dir=$outdir/$(dirname $tarfile)/.tmp_cpio_dir > > > > Dot files are ignored. > 'make clean' removes the .tmp_* pattern. This sounds great. Unfortunately 'make clean' only cleans files and not directories based on this pattern. Should it be extended? > > --- > > Changes in v2: > > - Also add the directory to clean-files > > - Link to v1: https://lore.kernel.org/r/20230117-kernel-kheaders-gitignore-v1-1-2a3a070efd0d@xxxxxxxxxxxxxx > > --- > > kernel/.gitignore | 1 + > > kernel/Makefile | 2 +- > > 2 files changed, 2 insertions(+), 1 deletion(-) > > > > diff --git a/kernel/.gitignore b/kernel/.gitignore > > index c6b299a6b786..57ab1d703763 100644 > > --- a/kernel/.gitignore > > +++ b/kernel/.gitignore > > @@ -1,3 +1,4 @@ > > # SPDX-License-Identifier: GPL-2.0-only > > /config_data > > /kheaders.md5 > > +/kheaders_data.tar.xz.tmp/ > > diff --git a/kernel/Makefile b/kernel/Makefile > > index 10ef068f598d..f3535944f635 100644 > > --- a/kernel/Makefile > > +++ b/kernel/Makefile > > @@ -156,4 +156,4 @@ quiet_cmd_genikh = CHK $(obj)/kheaders_data.tar.xz > > $(obj)/kheaders_data.tar.xz: FORCE > > $(call cmd,genikh) > > > > -clean-files := kheaders_data.tar.xz kheaders.md5 > > +clean-files := kheaders_data.tar.xz kheaders.md5 kheaders_data.tar.xz.tmp > > > > --- > > base-commit: d532dd102151cc69fcd00b13e5a9689b23c0c8d9 > > change-id: 20230117-kernel-kheaders-gitignore-f0d73456cb6b > > > > Best regards, > > -- > > Thomas Weißschuh <linux@xxxxxxxxxxxxxx>
