Re: [PATCH] Maintainers.txt: update OvmfPkg maintainership

[Ard Biesheuvel <ard.biesheuvel@linaro.org>]

On 16 August 2017 at 19:02, Laszlo Ersek <lersek@redhat.com> wrote:
> On 08/16/17 19:17, Leif Lindholm wrote:
>> Add Ard Biesheuvel as a maintainer on OvmfPkg.
>> Change Jordan Justen from maintainer to reviewer.
>>
>> Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
>> Cc: Jordan Justen <jordan.l.justen@intel.com>
>> Cc: Laszlo Ersek <lersek@redhat.com>
>> Cc: Andrew Fish <afish@apple.com>
>> Cc: Michael D Kinney <michael.d.kinney@intel.com>
>>
>> Contributed-under: TianoCore Contribution Agreement 1.1
>> Signed-off-by: Leif Lindholm <leif.lindholm@linaro.org>
>> ---
>>  Maintainers.txt | 3 ++-
>>  1 file changed, 2 insertions(+), 1 deletion(-)
>>
>> diff --git a/Maintainers.txt b/Maintainers.txt
>> index fd5c976056..82e864b0fd 100644
>> --- a/Maintainers.txt
>> +++ b/Maintainers.txt
>> @@ -192,8 +192,9 @@ M: Ruiyu Ni <ruiyu.ni@intel.com>
>>
>>  OvmfPkg
>>  W: http://www.tianocore.org/ovmf/
>> -M: Jordan Justen <jordan.l.justen@intel.com>
>> +R: Jordan Justen <jordan.l.justen@intel.com>
>>  M: Laszlo Ersek <lersek@redhat.com>
>> +M: Ard Biesheuvel <ard.biesheuvel@linaro.org>
>>  S: Maintained
>>
>>  PcAtChipsetPkg
>>
>
> For the record:
>
> I disagree with the two-maintainer limit set forth in commit
> d75b8ac278bc. (I recall that Leif found it strange too.) I would
> strongly prefer *all three of us* to co-maintain OvmfPkg, with push
> access -- "M" role -- tied specifically to OvmfPkg for *all* parties.
>

+1

> ... I guess the two-maintainer limit is a debate for another day.
>
> Acked-by: Laszlo Ersek <lersek@redhat.com>
>

Acked-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>