diff mbox series

MAINTAINERS: add more xenstore files

Message ID 20230428132756.8763-1-jgross@suse.com (mailing list archive)
State New, archived
Headers show
Series MAINTAINERS: add more xenstore files | expand

Commit Message

Jürgen Groß April 28, 2023, 1:27 p.m. UTC
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(+)

Comments

Andrew Cooper April 28, 2023, 1:29 p.m. UTC | #1
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>
Jan Beulich May 17, 2023, 3:19 p.m. UTC | #2
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
Jürgen Groß May 22, 2023, 2:22 p.m. UTC | #3
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
diff mbox series

Patch

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