Message ID | 20241122073725.1531483-1-lizhijian@fujitsu.com |
---|---|
State | Superseded |
Headers | show |
Series | [for-next,v3] selftests/filesystems: Add missing gitignore file | expand |
On Fri, Nov 22, 2024 at 03:37:25PM +0800, Li Zhijian wrote: > Compiled binary files should be added to .gitignore > > 'git status' complains: > Untracked files: > (use "git add <file>..." to include in what will be committed) > filesystems/statmount/statmount_test_ns > > Cc: Shuah Khan <shuah@kernel.org> > Cc: Christian Brauner <brauner@kernel.org> > Cc: Miklos Szeredi <mszeredi@redhat.com> > Cc: Josef Bacik <josef@toxicpanda.com> > Signed-off-by: Li Zhijian <lizhijian@fujitsu.com> > --- > Hello, > Cover letter is here. > > This patch set aims to make 'git status' clear after 'make' and 'make > run_tests' for kselftests. > --- > V3: > sorted the ignored files > V2: > split as a separate patch from a small one [0] > [0] https://lore.kernel.org/linux-kselftest/20241015010817.453539-1-lizhijian@fujitsu.com/ > > Signed-off-by: Li Zhijian <lizhijian@fujitsu.com> > --- > tools/testing/selftests/filesystems/statmount/.gitignore | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/tools/testing/selftests/filesystems/statmount/.gitignore b/tools/testing/selftests/filesystems/statmount/.gitignore > index 82a4846cbc4b..973363ad66a2 100644 > --- a/tools/testing/selftests/filesystems/statmount/.gitignore > +++ b/tools/testing/selftests/filesystems/statmount/.gitignore > @@ -1,2 +1,3 @@ > # SPDX-License-Identifier: GPL-2.0-only > +statmount_test_ns Thank you! This is still an issue in 6.13-rc1. Reviewed-by: Charlie Jenkins <charlie@rivosinc.com> Tested-by: Charlie Jenkins <charlie@rivosinc.com> > /*_test > -- > 2.44.0 > >
On 11/22/24 00:37, Li Zhijian wrote: > Compiled binary files should be added to .gitignore > > 'git status' complains: > Untracked files: > (use "git add <file>..." to include in what will be committed) > filesystems/statmount/statmount_test_ns > > Cc: Shuah Khan <shuah@kernel.org> > Cc: Christian Brauner <brauner@kernel.org> > Cc: Miklos Szeredi <mszeredi@redhat.com> > Cc: Josef Bacik <josef@toxicpanda.com> > Signed-off-by: Li Zhijian <lizhijian@fujitsu.com> > --- > Hello, > Cover letter is here. > > This patch set aims to make 'git status' clear after 'make' and 'make > run_tests' for kselftests. > --- > V3: > sorted the ignored files > V2: > split as a separate patch from a small one [0] > [0] https://lore.kernel.org/linux-kselftest/20241015010817.453539-1-lizhijian@fujitsu.com/ > > Signed-off-by: Li Zhijian <lizhijian@fujitsu.com> > --- > tools/testing/selftests/filesystems/statmount/.gitignore | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/tools/testing/selftests/filesystems/statmount/.gitignore b/tools/testing/selftests/filesystems/statmount/.gitignore > index 82a4846cbc4b..973363ad66a2 100644 > --- a/tools/testing/selftests/filesystems/statmount/.gitignore > +++ b/tools/testing/selftests/filesystems/statmount/.gitignore > @@ -1,2 +1,3 @@ > # SPDX-License-Identifier: GPL-2.0-only > +statmount_test_ns > /*_test ' I am seeing duplicate signature warning on this patch. Please fix and send a correct patch. thanks, -- Shuah
On 11/12/2024 01:10, Shuah Khan wrote: > On 11/22/24 00:37, Li Zhijian wrote: >> Compiled binary files should be added to .gitignore >> >> 'git status' complains: >> Untracked files: >> (use "git add <file>..." to include in what will be committed) >> filesystems/statmount/statmount_test_ns >> >> Cc: Shuah Khan <shuah@kernel.org> >> Cc: Christian Brauner <brauner@kernel.org> >> Cc: Miklos Szeredi <mszeredi@redhat.com> >> Cc: Josef Bacik <josef@toxicpanda.com> >> Signed-off-by: Li Zhijian <lizhijian@fujitsu.com> >> --- >> Hello, >> Cover letter is here. >> >> This patch set aims to make 'git status' clear after 'make' and 'make >> run_tests' for kselftests. >> --- >> V3: >> sorted the ignored files >> V2: >> split as a separate patch from a small one [0] >> [0] https://lore.kernel.org/linux-kselftest/20241015010817.453539-1-lizhijian@fujitsu.com/ >> >> Signed-off-by: Li Zhijian <lizhijian@fujitsu.com> >> --- >> tools/testing/selftests/filesystems/statmount/.gitignore | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/tools/testing/selftests/filesystems/statmount/.gitignore b/tools/testing/selftests/filesystems/statmount/.gitignore >> index 82a4846cbc4b..973363ad66a2 100644 >> --- a/tools/testing/selftests/filesystems/statmount/.gitignore >> +++ b/tools/testing/selftests/filesystems/statmount/.gitignore >> @@ -1,2 +1,3 @@ >> # SPDX-License-Identifier: GPL-2.0-only >> +statmount_test_ns >> /*_test > ' > I am seeing duplicate signature warning on this patch. Please > fix and send a correct patch. Good catch, fixed and posted an updated revision. Thanks Zhijian > > thanks, > -- Shuah
diff --git a/tools/testing/selftests/filesystems/statmount/.gitignore b/tools/testing/selftests/filesystems/statmount/.gitignore index 82a4846cbc4b..973363ad66a2 100644 --- a/tools/testing/selftests/filesystems/statmount/.gitignore +++ b/tools/testing/selftests/filesystems/statmount/.gitignore @@ -1,2 +1,3 @@ # SPDX-License-Identifier: GPL-2.0-only +statmount_test_ns /*_test