On Wed, May 16, 2018 at 09:18:01AM +0000, Iremonger, Bernard wrote: > Hi Adrien, > > > -----Original Message----- > > From: Adrien Mazarguil [mailto:adrien.mazarg...@6wind.com] > > Sent: Tuesday, May 15, 2018 5:23 PM > > To: dev@dpdk.org > > Cc: sta...@dpdk.org; Iremonger, Bernard <bernard.iremon...@intel.com> > > Subject: [PATCH] maintainers: fix responsibility of flow API bits > > > > The following commits lack MAINTAINERS entries for this mess. > > > > Fixes: 4d73b6fb9907 ("doc: add generic flow API guide") > > Fixes: 19c90af6285c ("app/testpmd: add flow command") > > Cc: sta...@dpdk.org > > > > Signed-off-by: Adrien Mazarguil <adrien.mazarg...@6wind.com> > > Cc: Bernard Iremonger <bernard.iremon...@intel.com> > > --- > > MAINTAINERS | 2 ++ > > 1 file changed, 2 insertions(+) > > > > diff --git a/MAINTAINERS b/MAINTAINERS > > index bd08d4292..187817fff 100644 > > --- a/MAINTAINERS > > +++ b/MAINTAINERS > > @@ -303,6 +303,8 @@ F: devtools/test-null.sh Flow API > > M: Adrien Mazarguil <adrien.mazarg...@6wind.com> > > T: git://dpdk.org/next/dpdk-next-net > > +F: app/test-pmd/cmdline_flow.c > > +F: doc/guides/prog_guide/rte_flow.rst > > doc/guides/testpmd_app_ug/testpmd_funcs.rst > Should be added to the list of flow related files.
I did not add it because there is no way to split responsibility on a documentation section basis AFAIK, and only a fraction of this file contains information about rte_flow-related stuff. -- Adrien Mazarguil 6WIND
