Re: [Xen-devel] [PATCH] get_maintainers.pl: Avoid THE_REST when files are added or removed

2018-02-22 Thread Ian Jackson
Jan Beulich writes ("Re: [PATCH] get_maintainers.pl: Avoid THE_REST when files are added or removed"): > On 22.02.18 at 10:27, wrote: > Oh, I see - it's the file name collection code you change, not the > consuming side, I'm sorry for the noise. However, isn't >

Re: [Xen-devel] [PATCH] get_maintainers.pl: Avoid THE_REST when files are added or removed

2018-02-22 Thread Alan Robinson
Hi Jan, On Thu, Feb 22, 2018 at 02:45:48AM -0700, Jan Beulich wrote: > Oh, I see - it's the file name collection code you change, not the > consuming side, I'm sorry for the noise. However, isn't > omitting the leading / a potential problem then? The leading / is stripped away several lines

Re: [Xen-devel] [PATCH] get_maintainers.pl: Avoid THE_REST when files are added or removed

2018-02-22 Thread Jan Beulich
>>> On 22.02.18 at 10:27, wrote: > On Thu, Feb 22, 2018 at 02:16:23AM -0700, Jan Beulich wrote: >> From: Jan Beulich >> Subject: Re: [PATCH] get_maintainers.pl: Avoid THE_REST when files are >> added or removed >> >> >>> On 22.02.18 at 10:09,

Re: [Xen-devel] [PATCH] get_maintainers.pl: Avoid THE_REST when files are added or removed

2018-02-22 Thread Alan Robinson
Hi Jan, On Thu, Feb 22, 2018 at 02:16:23AM -0700, Jan Beulich wrote: > From: Jan Beulich > Subject: Re: [PATCH] get_maintainers.pl: Avoid THE_REST when files are > added or removed > > >>> On 22.02.18 at 10:09, wrote: > > When files are added

Re: [Xen-devel] [PATCH] get_maintainers.pl: Avoid THE_REST when files are added or removed

2018-02-22 Thread Jan Beulich
>>> On 22.02.18 at 10:09, wrote: > When files are added or removed /dev/null is used as a place > holder name in the patch for the absent file. Don't try and > find a MAINTAINER for this place holder, it only ever flags > and then spams THE REST. When a file is

[Xen-devel] [PATCH] get_maintainers.pl: Avoid THE_REST when files are added or removed

2018-02-22 Thread Alan Robinson
When files are added or removed /dev/null is used as a place holder name in the patch for the absent file. Don't try and find a MAINTAINER for this place holder, it only ever flags and then spams THE REST. Signed-off-by: Alan Robinson --- Tested by pretending to