diff mbox series

[4/4] selftests: exec: update gitignore for load_address

Message ID 20240924-selftests-gitignore-v1-4-9755ac883388@gmail.com
State Superseded
Headers show
Series selftsets: gitignore and clean target file additions | expand

Commit Message

Javier Carrasco Sept. 24, 2024, 12:49 p.m. UTC
The name of the "load_address" objects has been modified, but the
corresponding entry in the gitignore file must be updated.

Update the load_address entry in the gitignore file to account for
the new names.

Signed-off-by: Javier Carrasco <javier.carrasco.cruz@gmail.com>
---
 tools/testing/selftests/exec/.gitignore | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

kernel test robot Sept. 25, 2024, noon UTC | #1
Hi Javier,

kernel test robot noticed the following build warnings:

[auto build test WARNING on 4d0326b60bb753627437fff0f76bf1525bcda422]

url:    https://github.com/intel-lab-lkp/linux/commits/Javier-Carrasco/selftests-add-unshare_test-and-msg_oob-to-gitignore/20240924-205133
base:   4d0326b60bb753627437fff0f76bf1525bcda422
patch link:    https://lore.kernel.org/r/20240924-selftests-gitignore-v1-4-9755ac883388%40gmail.com
patch subject: [PATCH 4/4] selftests: exec: update gitignore for load_address
config: x86_64-allnoconfig (https://download.01.org/0day-ci/archive/20240925/202409251828.dMx8LnrF-lkp@intel.com/config)
compiler: clang version 18.1.8 (https://github.com/llvm/llvm-project 3b5b5c1ec4a3095ab096dd780e84d7ab81f3d7ff)
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20240925/202409251828.dMx8LnrF-lkp@intel.com/reproduce)

If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202409251828.dMx8LnrF-lkp@intel.com/

All warnings (new ones prefixed by >>):

   tools/testing/selftests/arm64/tags/.gitignore: warning: ignored by one of the .gitignore files
   tools/testing/selftests/arm64/tags/Makefile: warning: ignored by one of the .gitignore files
   tools/testing/selftests/arm64/tags/tags_test.c: warning: ignored by one of the .gitignore files
>> tools/testing/selftests/exec/load_address.c: warning: ignored by one of the .gitignore files
Shuah Khan Sept. 25, 2024, 3:46 p.m. UTC | #2
On 9/24/24 06:49, Javier Carrasco wrote:
> The name of the "load_address" objects has been modified, but the
> corresponding entry in the gitignore file must be updated.
> 
> Update the load_address entry in the gitignore file to account for
> the new names.
> 
> Signed-off-by: Javier Carrasco <javier.carrasco.cruz@gmail.com>
> ---
>   tools/testing/selftests/exec/.gitignore | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/tools/testing/selftests/exec/.gitignore b/tools/testing/selftests/exec/.gitignore
> index 90c238ba6a4b..4d9fb7b20ea7 100644
> --- a/tools/testing/selftests/exec/.gitignore
> +++ b/tools/testing/selftests/exec/.gitignore
> @@ -9,7 +9,7 @@ execveat.ephemeral
>   execveat.denatured
>   non-regular
>   null-argv
> -/load_address_*
> +/load_address.*

Hmm. This will include the load_address.c which shouldn't
be included in the .gitignore?

>   /recursion-depth
>   xxxxxxxx*
>   pipe
> 

thanks,
-- Shuah
Shuah Khan Sept. 25, 2024, 5:44 p.m. UTC | #3
On 9/25/24 10:25, Javier Carrasco wrote:
> On 25/09/2024 17:46, Shuah Khan wrote:
>> On 9/24/24 06:49, Javier Carrasco wrote:
>>> The name of the "load_address" objects has been modified, but the
>>> corresponding entry in the gitignore file must be updated.
>>>
>>> Update the load_address entry in the gitignore file to account for
>>> the new names.
>>>
>>> Signed-off-by: Javier Carrasco <javier.carrasco.cruz@gmail.com>
>>> ---
>>>    tools/testing/selftests/exec/.gitignore | 2 +-
>>>    1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>> diff --git a/tools/testing/selftests/exec/.gitignore b/tools/testing/
>>> selftests/exec/.gitignore
>>> index 90c238ba6a4b..4d9fb7b20ea7 100644
>>> --- a/tools/testing/selftests/exec/.gitignore
>>> +++ b/tools/testing/selftests/exec/.gitignore
>>> @@ -9,7 +9,7 @@ execveat.ephemeral
>>>    execveat.denatured
>>>    non-regular
>>>    null-argv
>>> -/load_address_*
>>> +/load_address.*
>>
>> Hmm. This will include the load_address.c which shouldn't
>> be included in the .gitignore?
>>
>>>    /recursion-depth
>>>    xxxxxxxx*
>>>    pipe
>>>
>>
>> thanks,
>> -- Shuah
> 
> 
> Hi, the kernel test robot already notified me about that issue, and I
> sent a v2 to fix it shortly after. Please take a look at the newer
> version where I added the exception for load_address.c.
> 

Thanks. I saw your v2 after sending this email. I have a comment
on v2 to split core and net patch

thanks,
-- Shuah
diff mbox series

Patch

diff --git a/tools/testing/selftests/exec/.gitignore b/tools/testing/selftests/exec/.gitignore
index 90c238ba6a4b..4d9fb7b20ea7 100644
--- a/tools/testing/selftests/exec/.gitignore
+++ b/tools/testing/selftests/exec/.gitignore
@@ -9,7 +9,7 @@  execveat.ephemeral
 execveat.denatured
 non-regular
 null-argv
-/load_address_*
+/load_address.*
 /recursion-depth
 xxxxxxxx*
 pipe