MAINTAINERS: add more xenstore files

[PATCH] MAINTAINERS: add more xenstore files

Posted by Juergen Gross 1 year ago

Xenstore consists of more files than just the tools/xenstore directory.

Add them to the XENSTORE block.

Suggested-by: Andrew Cooper <andrew.cooper3@citrix.com>
Signed-off-by: Juergen Gross <jgross@suse.com>
---
 MAINTAINERS | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index 0e5eba2312..f2f1881b32 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -653,6 +653,11 @@ M:	Wei Liu <wl@xen.org>
 M:	Juergen Gross <jgross@suse.com>
 R:	Julien Grall <julien@xen.org>
 S:	Supported
+F:	tools/helpers/init-xenstore-domain.c
+F:	tools/include/xenstore-compat/
+F:	tools/include/xenstore.h
+F:	tools/include/xenstore_lib.h
+F:	tools/libs/store/
 F:	tools/xenstore/
 
 XENTRACE
-- 
2.35.3

Re: [PATCH] MAINTAINERS: add more xenstore files

Posted by Jan Beulich 11 months, 3 weeks ago

On 28.04.2023 15:27, Juergen Gross wrote:
> Xenstore consists of more files than just the tools/xenstore directory.
> 
> Add them to the XENSTORE block.
> 
> Suggested-by: Andrew Cooper <andrew.cooper3@citrix.com>
> Signed-off-by: Juergen Gross <jgross@suse.com>
> ---
>  MAINTAINERS | 5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 0e5eba2312..f2f1881b32 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -653,6 +653,11 @@ M:	Wei Liu <wl@xen.org>
>  M:	Juergen Gross <jgross@suse.com>
>  R:	Julien Grall <julien@xen.org>
>  S:	Supported
> +F:	tools/helpers/init-xenstore-domain.c
> +F:	tools/include/xenstore-compat/
> +F:	tools/include/xenstore.h
> +F:	tools/include/xenstore_lib.h
> +F:	tools/libs/store/
>  F:	tools/xenstore/

I wonder if at the same time xenstore specific include files shouldn't
have been purged from LIBS.

Jan

Re: [PATCH] MAINTAINERS: add more xenstore files

Posted by Juergen Gross 11 months, 2 weeks ago

On 17.05.23 17:19, Jan Beulich wrote:
> On 28.04.2023 15:27, Juergen Gross wrote:
>> Xenstore consists of more files than just the tools/xenstore directory.
>>
>> Add them to the XENSTORE block.
>>
>> Suggested-by: Andrew Cooper <andrew.cooper3@citrix.com>
>> Signed-off-by: Juergen Gross <jgross@suse.com>
>> ---
>>   MAINTAINERS | 5 +++++
>>   1 file changed, 5 insertions(+)
>>
>> diff --git a/MAINTAINERS b/MAINTAINERS
>> index 0e5eba2312..f2f1881b32 100644
>> --- a/MAINTAINERS
>> +++ b/MAINTAINERS
>> @@ -653,6 +653,11 @@ M:	Wei Liu <wl@xen.org>
>>   M:	Juergen Gross <jgross@suse.com>
>>   R:	Julien Grall <julien@xen.org>
>>   S:	Supported
>> +F:	tools/helpers/init-xenstore-domain.c
>> +F:	tools/include/xenstore-compat/
>> +F:	tools/include/xenstore.h
>> +F:	tools/include/xenstore_lib.h
>> +F:	tools/libs/store/
>>   F:	tools/xenstore/
> 
> I wonder if at the same time xenstore specific include files shouldn't
> have been purged from LIBS.

Probably, yes. I'll send a patch.


Juergen

Re: [PATCH] MAINTAINERS: add more xenstore files

Posted by Andrew Cooper 1 year ago

On 28/04/2023 2:27 pm, Juergen Gross wrote:
> Xenstore consists of more files than just the tools/xenstore directory.
>
> Add them to the XENSTORE block.
>
> Suggested-by: Andrew Cooper <andrew.cooper3@citrix.com>
> Signed-off-by: Juergen Gross <jgross@suse.com>

Acked-by: Andrew Cooper <andrew.cooper3@citrix.com>