On 25.07.2017 07:48, Thomas Huth wrote: > On 25.07.2017 00:40, Philippe Mathieu-Daudé wrote: >> Hi perl gurus :) >> >> after changing the ./configure I wanted to figure to whom email and got >> this: >> >> $ ./scripts/get_maintainer.pl -f configure >> Kamil Rytarowski <ka...@netbsd.org> (maintainer:NETBSD) >> qemu-devel@nongnu.org (open list:All patches CC here) >> >> $ fgrep -A5 NETBSD MAINTAINERS >> NETBSD >> L: qemu-devel@nongnu.org >> M: Kamil Rytarowski <ka...@netbsd.org> >> S: Maintained >> K: (?i)NetBSD >> >> Descriptions of section entries: >> >> K: Keyword perl extended regex pattern to match >> content in a patch or file. >> >> $ fgrep -inA1 NetBSD configure >> 547:elif check_define __NetBSD__; then >> 548: targetos='NetBSD' >> -- >> 709:NetBSD) >> 710- bsd="yes" >> -- >> 715: HOST_VARIANT_DIR="netbsd" >> 716- supported_os="yes" >> >> since ./configure check for NetBSD host/target, the string appears in >> the ./configure and the K regex is matched... >> >> I tried to add a 'F: configure' in NETBSD entry but it doesn't change. >> The easier is to kill the K entry :) > > But without the K: entry there is no more pattern for this entry at all > anymore - since there are also no F: entries here. > > I'd suggest to do it the same way as the "Trivial patches" entry: > > K: ^Subject:.*(?i)NetBSD > > ... then it should hopefully only trigger on patch titles. > > Philippe, Kamil, if one of you has got some spare minutes, could you > please send a patch? > > Thomas >
I don't have preference here. I'm fine either way.
signature.asc
Description: OpenPGP digital signature
