[PATCH] MAINTAINERS: drop XSM maintainer

Jan Beulich posted 1 patch 2 years, 5 months ago
Failed in applying to current master (apply log)
[PATCH] MAINTAINERS: drop XSM maintainer
Posted by Jan Beulich 2 years, 5 months ago
While mail hasn't been bouncing, Daniel has not been responding to patch
submissions or otherwise interacting with the community for several
years. Move maintainership to THE REST in kind of an unusual way, with
the goal to avoid
- orphaning the component,
- repeating all THE REST members here,
- removing the entry altogether.

Signed-off-by: Jan Beulich <jbeulich@suse.com>
---
We hope this to be transient, with a new maintainer to be established
rather sooner than later.

I realize the way I'm expressing this may upset scripts/*_maintainer*.pl,
so I'd welcome any better alternative suggestion.

--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -648,7 +648,7 @@ F:	xen/common/trace.c
 F:	xen/include/xen/trace.h
 
 XSM/FLASK
-M:	Daniel De Graaf <dgdegra@tycho.nsa.gov>
+M:	THE REST (see below)
 R:	Daniel P. Smith <dpsmith@apertussolutions.com>
 S:	Supported
 F:	tools/flask/
Re: [PATCH] MAINTAINERS: drop XSM maintainer
Posted by Jan Beulich 2 years, 5 months ago
On 09.06.2022 17:33, Jan Beulich wrote:
> While mail hasn't been bouncing, Daniel has not been responding to patch
> submissions or otherwise interacting with the community for several
> years. Move maintainership to THE REST in kind of an unusual way, with
> the goal to avoid
> - orphaning the component,
> - repeating all THE REST members here,
> - removing the entry altogether.
> 
> Signed-off-by: Jan Beulich <jbeulich@suse.com>
> ---
> We hope this to be transient, with a new maintainer to be established
> rather sooner than later.
> 
> I realize the way I'm expressing this may upset scripts/*_maintainer*.pl,
> so I'd welcome any better alternative suggestion.

Two weeks have passed. May I ask for an ack so this can go in?

Thanks, Jan

> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -648,7 +648,7 @@ F:	xen/common/trace.c
>  F:	xen/include/xen/trace.h
>  
>  XSM/FLASK
> -M:	Daniel De Graaf <dgdegra@tycho.nsa.gov>
> +M:	THE REST (see below)
>  R:	Daniel P. Smith <dpsmith@apertussolutions.com>
>  S:	Supported
>  F:	tools/flask/
>
Re: [PATCH] MAINTAINERS: drop XSM maintainer
Posted by George Dunlap 2 years, 5 months ago

> On 23 Jun 2022, at 08:43, Jan Beulich <jbeulich@suse.com> wrote:
> 
> On 09.06.2022 17:33, Jan Beulich wrote:
>> While mail hasn't been bouncing, Daniel has not been responding to patch
>> submissions or otherwise interacting with the community for several
>> years. Move maintainership to THE REST in kind of an unusual way, with
>> the goal to avoid
>> - orphaning the component,
>> - repeating all THE REST members here,
>> - removing the entry altogether.
>> 
>> Signed-off-by: Jan Beulich <jbeulich@suse.com>
>> ---
>> We hope this to be transient, with a new maintainer to be established
>> rather sooner than later.
>> 
>> I realize the way I'm expressing this may upset scripts/*_maintainer*.pl,
>> so I'd welcome any better alternative suggestion.
> 
> Two weeks have passed. May I ask for an ack so this can go in?

I’m happy to give you my Ack re the maintainership change, but I’m not qualified to comment on whether it will screw up the get_maintainer script.  Perhaps it would be better to send a v2 patch, proposing Daniel Smith as a replacement, to avoid the question of the script entirely.

Acked-by: George Dunlap <george.dunlap@citrix.com <mailto:george.dunlap@citrix.com>> # Removing old maintainer

 -George

Re: [PATCH] MAINTAINERS: drop XSM maintainer
Posted by Roger Pau Monné 2 years, 5 months ago
On Thu, Jun 23, 2022 at 09:56:34AM +0000, George Dunlap wrote:
> 
> 
> > On 23 Jun 2022, at 08:43, Jan Beulich <jbeulich@suse.com> wrote:
> > 
> > On 09.06.2022 17:33, Jan Beulich wrote:
> >> While mail hasn't been bouncing, Daniel has not been responding to patch
> >> submissions or otherwise interacting with the community for several
> >> years. Move maintainership to THE REST in kind of an unusual way, with
> >> the goal to avoid
> >> - orphaning the component,
> >> - repeating all THE REST members here,
> >> - removing the entry altogether.
> >> 
> >> Signed-off-by: Jan Beulich <jbeulich@suse.com>
> >> ---
> >> We hope this to be transient, with a new maintainer to be established
> >> rather sooner than later.
> >> 
> >> I realize the way I'm expressing this may upset scripts/*_maintainer*.pl,
> >> so I'd welcome any better alternative suggestion.
> > 
> > Two weeks have passed. May I ask for an ack so this can go in?
> 
> I’m happy to give you my Ack re the maintainership change, but I’m not qualified to comment on whether it will screw up the get_maintainer script.  Perhaps it would be better to send a v2 patch, proposing Daniel Smith as a replacement, to avoid the question of the script entirely.

Maybe we should modify the script so that any sections that don't have
any Maintainer assigned (no M:) fallback into the rest if that's not
the case already?

Thanks, Roger.

Re: [PATCH] MAINTAINERS: drop XSM maintainer
Posted by Roger Pau Monné 2 years, 5 months ago
On Thu, Jun 23, 2022 at 09:43:30AM +0200, Jan Beulich wrote:
> On 09.06.2022 17:33, Jan Beulich wrote:
> > While mail hasn't been bouncing, Daniel has not been responding to patch
> > submissions or otherwise interacting with the community for several
> > years. Move maintainership to THE REST in kind of an unusual way, with
> > the goal to avoid
> > - orphaning the component,
> > - repeating all THE REST members here,
> > - removing the entry altogether.
> > 
> > Signed-off-by: Jan Beulich <jbeulich@suse.com>
> > ---
> > We hope this to be transient, with a new maintainer to be established
> > rather sooner than later.
> > 
> > I realize the way I'm expressing this may upset scripts/*_maintainer*.pl,
> > so I'd welcome any better alternative suggestion.
> 
> Two weeks have passed. May I ask for an ack so this can go in?

I don't think mine suffices, but just in case:

Acked-by: Roger Pau Monné <roger.pau@citrix.com>

Thanks, Roger.