Re: [Qemu-devel] [PULL 37/40] Add David Gibson for sPAPR in MAINTAINERS file

[Eric Blake]

On 11/18/2015 01:46 PM, Andreas Färber wrote:

>> +++ b/MAINTAINERS
>> @@ -486,7 +486,8 @@ F: hw/ppc/prep.c
>>  F: hw/pci-host/prep.[hc]
>>  F: hw/isa/pc87312.[hc]
>>  
>> -sPAPR
>> +sPAPR (pseries)
> 
> This change leads to get_maintainer.pl using "address@hidden (open
> list:sPAPR (pseries))", which results in:
> 
> 5.1.3 Bad recipient address syntax
> 
>> +M: David Gibson <address@hidden>
>>  M: Alexander Graf <address@hidden>
>>  L: address@hidden
> 
> Changing this to
> 
> L: qemu-ppc <address@hidden>
> 
> I am instead asked:
> 
> (cc-cmd) Adding cc: (open list:sPAPR (pseries)) from:
> 'scripts/get_maintainer.pl --nogit-fallback'
> (cc-cmd) Adding cc: address@hidden (open list:PowerPC) from:
> 'scripts/get_maintainer.pl --nogit-fallback'
> error: unable to extract a valid address from: ")"  (open list:sPAPR
> (pseries)
> What to do with this address? ([q]uit|[d]rop|[e]dit):
> 
>>  S: Supported
> 
> While this is not the only MAINTAINERS section with parenthesis in the
> title, it is the only one to have an L: entry in that section.

Does using a section title of "sPAPR (pseries)" (with the quotes present
in the file) make a difference?

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

signature.asc
Description: OpenPGP digital signature