Re: [Xen-devel] [PATCH] Intel TXT: add reviewer, move to Odd Fixes state

[Jan Beulich <JBeulich@xxxxxxxx>]

On 30.07.2019 10:54, Julien Grall wrote:
> On 7/30/19 9:29 AM, Jan Beulich wrote:
>> On 30.07.2019 08:56, Lukasz Hawrylko wrote:
>>> Support for Intel TXT has orphaned status right now because
>>> no active maintainter is listed. Adding myself as reviewer
>>> and moving it to Odd Fixes state.
>>>
>>> Signed-off-by: Lukasz Hawrylko <lukasz.hawrylko@xxxxxxxxx>
>>> ---
>>>    MAINTAINERS | 3 ++-
>>>    1 file changed, 2 insertions(+), 1 deletion(-)
>>>
>>> diff --git a/MAINTAINERS b/MAINTAINERS
>>> index 89a01b710b..ca300e87c8 100644
>>> --- a/MAINTAINERS
>>> +++ b/MAINTAINERS
>>> @@ -240,7 +240,8 @@ S:    Maintained
>>>  F:    tools/golang
>>>  INTEL(R) TRUSTED EXECUTION TECHNOLOGY (TXT)
>>> -S:    Orphaned
>>> +R:    Lukasz Hawrylko <lukasz.hawrylko@xxxxxxxxx>
>>> +S:    Odd Fixes
>>
>> I guess we should give it a few days for objections to be raised
>> against this slightly inconsistent state, but I think that's the
>> best way to express the current state of things (hence my
>> suggestion to this effect). If no objections turn up, I've queued
>> this onto my to-be-committed list.
> 
> I have some objections regarding the process itself... On the first
> version of this patch, it was pointed out that the e-mail shouldn't
> be sent with disclaimer. This is now the third version and the
> disclaimer is still present.

Okay, I must have missed both earlier requests to this effect. I've
gone back to the list archives though, and I couldn't find any such
request either from July or June. Therefore ...

> Technically, no patch should be applied when there are a disclaimer.

... I'd also like to ask for the background of this. It would never
have occurred to me that I should pay attention to possible
disclaimers or alike on patch submissions.

Jan
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel