1. Patchew
  2. linux
  3. View series

[PATCH for-next v3] selftests/zram: gitignore output file

Li Zhijian posted 1 patch 1 year, 2 months ago
There is a newer version of this series
tools/testing/selftests/zram/.gitignore | 2 ++
1 file changed, 2 insertions(+)
create mode 100644 tools/testing/selftests/zram/.gitignore
[PATCH for-next v3] selftests/zram: gitignore output file
Posted by Li Zhijian 1 year, 2 months ago 
After `make run_tests`, the git status complains:
Untracked files:
    (use "git add <file>..." to include in what will be committed)
        zram/err.log

This file will be cleaned up when execute 'make clean'

Cc: Shuah Khan <shuah@kernel.org>
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:
  Add Copyright description
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/zram/.gitignore | 2 ++
 1 file changed, 2 insertions(+)
 create mode 100644 tools/testing/selftests/zram/.gitignore

diff --git a/tools/testing/selftests/zram/.gitignore b/tools/testing/selftests/zram/.gitignore
new file mode 100644
index 000000000000..088cd9bad87a
--- /dev/null
+++ b/tools/testing/selftests/zram/.gitignore
@@ -0,0 +1,2 @@
+# SPDX-License-Identifier: GPL-2.0-only
+err.log
-- 
2.44.0
Re: [PATCH for-next v3] selftests/zram: gitignore output file
Posted by Shuah Khan 1 year, 1 month ago 
On 11/22/24 00:49, Li Zhijian wrote:
> After `make run_tests`, the git status complains:
> Untracked files:
>      (use "git add <file>..." to include in what will be committed)
>          zram/err.log
> 
> This file will be cleaned up when execute 'make clean'
> 
> Cc: Shuah Khan <shuah@kernel.org>
> 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:
>    Add Copyright description
> 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/zram/.gitignore | 2 ++
>   1 file changed, 2 insertions(+)
>   create mode 100644 tools/testing/selftests/zram/.gitignore
> 
> diff --git a/tools/testing/selftests/zram/.gitignore b/tools/testing/selftests/zram/.gitignore
> new file mode 100644
> index 000000000000..088cd9bad87a
> --- /dev/null
> +++ b/tools/testing/selftests/zram/.gitignore
> @@ -0,0 +1,2 @@
> +# SPDX-License-Identifier: GPL-2.0-only
> +err.log


I am seeing duplicate signature warning on this patch. Please
fix and send a correct patch.

thanks,
-- Shuah
Re: [PATCH for-next v3] selftests/zram: gitignore output file
Posted by Zhijian Li (Fujitsu) 1 year, 1 month ago 

On 11/12/2024 01:09, Shuah Khan wrote:
> On 11/22/24 00:49, Li Zhijian wrote:
>> After `make run_tests`, the git status complains:
>> Untracked files:
>>      (use "git add <file>..." to include in what will be committed)
>>          zram/err.log
>>
>> This file will be cleaned up when execute 'make clean'
>>
>> Cc: Shuah Khan <shuah@kernel.org>
>> 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:
>>    Add Copyright description
>> 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/zram/.gitignore | 2 ++
>>   1 file changed, 2 insertions(+)
>>   create mode 100644 tools/testing/selftests/zram/.gitignore
>>
>> diff --git a/tools/testing/selftests/zram/.gitignore b/tools/testing/selftests/zram/.gitignore
>> new file mode 100644
>> index 000000000000..088cd9bad87a
>> --- /dev/null
>> +++ b/tools/testing/selftests/zram/.gitignore
>> @@ -0,0 +1,2 @@
>> +# SPDX-License-Identifier: GPL-2.0-only
>> +err.log
> 
> 
> 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
Re: [PATCH for-next v3] selftests/zram: gitignore output file
Posted by Shuah Khan 1 year, 2 months ago 
On 11/22/24 00:49, Li Zhijian wrote:
> After `make run_tests`, the git status complains:
> Untracked files:
>      (use "git add <file>..." to include in what will be committed)
>          zram/err.log
> 
> This file will be cleaned up when execute 'make clean'
> 
> Cc: Shuah Khan <shuah@kernel.org>
> 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:
>    Add Copyright description
> 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/zram/.gitignore | 2 ++
>   1 file changed, 2 insertions(+)
>   create mode 100644 tools/testing/selftests/zram/.gitignore
> 
> diff --git a/tools/testing/selftests/zram/.gitignore b/tools/testing/selftests/zram/.gitignore
> new file mode 100644
> index 000000000000..088cd9bad87a
> --- /dev/null
> +++ b/tools/testing/selftests/zram/.gitignore
> @@ -0,0 +1,2 @@
> +# SPDX-License-Identifier: GPL-2.0-only
> +err.log

This doesn't sound right. err.log is a generated file
when the test runs.

thanks,
-- Shuah
Re: [PATCH for-next v3] selftests/zram: gitignore output file
Posted by Zhijian Li (Fujitsu) 1 year, 2 months ago 

On 04/12/2024 09:21, Shuah Khan wrote:
> On 11/22/24 00:49, Li Zhijian wrote:
>> After `make run_tests`, the git status complains:
>> Untracked files:
>>      (use "git add <file>..." to include in what will be committed)
>>          zram/err.log
>>
>> This file will be cleaned up when execute 'make clean'
>>
>> Cc: Shuah Khan <shuah@kernel.org>
>> 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:
>>    Add Copyright description
>> 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/zram/.gitignore | 2 ++
>>   1 file changed, 2 insertions(+)
>>   create mode 100644 tools/testing/selftests/zram/.gitignore
>>
>> diff --git a/tools/testing/selftests/zram/.gitignore b/tools/testing/selftests/zram/.gitignore
>> new file mode 100644
>> index 000000000000..088cd9bad87a
>> --- /dev/null
>> +++ b/tools/testing/selftests/zram/.gitignore
>> @@ -0,0 +1,2 @@
>> +# SPDX-License-Identifier: GPL-2.0-only
>> +err.log
> 
> This doesn't sound right. err.log is a generated file
> when the test runs.


IIUC, generally, there are 2 ways to handle files generated by the tests
- 1. clean/remove it if it's a temporary file before the test exits.
- 2. add it to the gitignore if it's the test result or log that needs to be referred to later.

In this zram case, it seems it intended to log and print the error
only if some commands get something wrong.

And this file have been already tracked in the Makefile
tools/testing/selftests/zram/Makefile:EXTRA_CLEAN := err.log

Given that I also think it's fine to add it to gitignore.


Thanks
Zhijian

> 
> thanks,
> -- Shuah
>

Patchew 2.1-devel-b67e4c2

© 2016 - 2026 Red Hat, Inc.