| Message ID | 20241004111127.897937-1-frediano.ziglio@cloud.com (mailing list archive) |
|---|---|
| State | Superseded |
| Headers | show |
| Series | [v2] MAINTAINERS: minor file line update | expand |
On 04.10.2024 13:11, Frediano Ziglio wrote: > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -517,7 +517,7 @@ F: stubdom/ > TEE MEDIATORS > M: Volodymyr Babchuk <volodymyr_babchuk@epam.com> > S: Supported > -F: xen/arch/arm/include/asm/tee > +F: xen/arch/arm/include/asm/tee/ > F: xen/arch/arm/tee/ I think this is a change which actually wants ack-ing by the maintainer named here, not THE REST. Cc-ing Volodymyr. Jan
Hi Jan, On 08/10/2024 12:48, Jan Beulich wrote: > On 04.10.2024 13:11, Frediano Ziglio wrote: >> --- a/MAINTAINERS >> +++ b/MAINTAINERS >> @@ -517,7 +517,7 @@ F: stubdom/ >> TEE MEDIATORS >> M: Volodymyr Babchuk <volodymyr_babchuk@epam.com> >> S: Supported >> -F: xen/arch/arm/include/asm/tee >> +F: xen/arch/arm/include/asm/tee/ >> F: xen/arch/arm/tee/ > > I think this is a change which actually wants ack-ing by the maintainer > named here, not THE REST. Cc-ing Volodymyr. I would agree if this is significantly changing the list of files maintained. However, from testing, it doesn't change anything. So I don't think it is necessary to require the actual maintainer named in this section... Also Bertrand is now a co-maintainer. So CCing him. I will give both of them another day to answer. If there are no answer, I will commit by end of my day tomorrow with: Acked-by: Julien Grall <jgrall@amazon.com> Cheers,
Jan Beulich <jbeulich@suse.com> writes: Hi > On 04.10.2024 13:11, Frediano Ziglio wrote: >> --- a/MAINTAINERS >> +++ b/MAINTAINERS >> @@ -517,7 +517,7 @@ F: stubdom/ >> TEE MEDIATORS >> M: Volodymyr Babchuk <volodymyr_babchuk@epam.com> >> S: Supported >> -F: xen/arch/arm/include/asm/tee >> +F: xen/arch/arm/include/asm/tee/ >> F: xen/arch/arm/tee/ > Acked-by: Volodymyr Babchuk <volodymyr_babchuk@epam.com> > I think this is a change which actually wants ack-ing by the maintainer > named here, not THE REST. Cc-ing Volodymyr. > > Jan
Hi, > On 17 Oct 2024, at 18:53, Volodymyr Babchuk <Volodymyr_Babchuk@epam.com> wrote: > > Jan Beulich <jbeulich@suse.com> writes: > > Hi > >> On 04.10.2024 13:11, Frediano Ziglio wrote: >>> --- a/MAINTAINERS >>> +++ b/MAINTAINERS >>> @@ -517,7 +517,7 @@ F: stubdom/ >>> TEE MEDIATORS >>> M: Volodymyr Babchuk <volodymyr_babchuk@epam.com> >>> S: Supported >>> -F: xen/arch/arm/include/asm/tee >>> +F: xen/arch/arm/include/asm/tee/ >>> F: xen/arch/arm/tee/ >> > > Acked-by: Volodymyr Babchuk <volodymyr_babchuk@epam.com> Acked-by: Bertrand Marquis <bertrand.marquis@arm.com> Cheers Bertrand > >> I think this is a change which actually wants ack-ing by the maintainer >> named here, not THE REST. Cc-ing Volodymyr. >> >> Jan > > > -- > WBR, Volodymyr
diff --git a/MAINTAINERS b/MAINTAINERS index fb0ebf0939..a2994c7421 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -517,7 +517,7 @@ F: stubdom/ TEE MEDIATORS M: Volodymyr Babchuk <volodymyr_babchuk@epam.com> S: Supported -F: xen/arch/arm/include/asm/tee +F: xen/arch/arm/include/asm/tee/ F: xen/arch/arm/tee/ TOOLSTACK
"xen/arch/arm/include/asm/tee" is a directory and should be terminated by a slash ("/"). Signed-off-by: Frediano Ziglio <frediano.ziglio@cloud.com> --- MAINTAINERS | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) --- Changes since v1: - removed second change, it was intentional.