diff mbox series

maintainers: add Windows exports script

Message ID 20190613141155.13573-1-thomas@monjalon.net (mailing list archive)
State Accepted, archived
Delegated to: Thomas Monjalon
Headers
Series maintainers: add Windows exports script |

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/intel-Performance-Testing success Performance Testing PASS
ci/mellanox-Performance-Testing success Performance Testing PASS
ci/Intel-compilation fail Compilation issues

Commit Message

Thomas Monjalon June 13, 2019, 2:11 p.m. UTC 
  A script used with meson was missing in the list of files.

Fixes: 4c773788e05a ("build: generate Windows exports file")
Cc: bruce.richardson@intel.com

Signed-off-by: Thomas Monjalon <thomas@monjalon.net>
---
 MAINTAINERS | 1 +
 1 file changed, 1 insertion(+)

Comments

David Marchand June 13, 2019, 2:17 p.m. UTC | #1 
On Thu, Jun 13, 2019 at 4:12 PM Thomas Monjalon <thomas@monjalon.net> wrote:

> A script used with meson was missing in the list of files.
>
> Fixes: 4c773788e05a ("build: generate Windows exports file")
> Cc: bruce.richardson@intel.com
>
> Signed-off-by: Thomas Monjalon <thomas@monjalon.net>
> ---
>  MAINTAINERS | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 0212fe6d0..fdc083db9 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -117,6 +117,7 @@ F: lib/librte_eal/freebsd/BSDmakefile.meson
>  F: meson_options.txt
>  F: config/rte_config.h
>  F: buildtools/gen-pmdinfo-cfile.sh
> +F: buildtools/map_to_def.py
>  F: buildtools/symlink-drivers-solibs.sh
>
>  Public CI
>

Reviewed-by: David Marchand <david.marchand@redhat.com>
Thomas Monjalon June 17, 2019, 9:33 a.m. UTC | #2 
13/06/2019 17:11, Thomas Monjalon:
> A script used with meson was missing in the list of files.
> 
> Fixes: 4c773788e05a ("build: generate Windows exports file")
> Cc: bruce.richardson@intel.com
> 
> Signed-off-by: Thomas Monjalon <thomas@monjalon.net>


Bruce, do you agree with following addition?

> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -117,6 +117,7 @@ F: lib/librte_eal/freebsd/BSDmakefile.meson
>  F: meson_options.txt
>  F: config/rte_config.h
>  F: buildtools/gen-pmdinfo-cfile.sh
> +F: buildtools/map_to_def.py
>  F: buildtools/symlink-drivers-solibs.sh
Bruce Richardson June 17, 2019, 10:02 a.m. UTC | #3 
On Mon, Jun 17, 2019 at 12:33:58PM +0300, Thomas Monjalon wrote:
> 13/06/2019 17:11, Thomas Monjalon:
> > A script used with meson was missing in the list of files.
> > 
> > Fixes: 4c773788e05a ("build: generate Windows exports file")
> > Cc: bruce.richardson@intel.com
> > 
> > Signed-off-by: Thomas Monjalon <thomas@monjalon.net>
> 
> 
> Bruce, do you agree with following addition?
> 

No objections. Sorry for not replying sooner.

Acked-by: Bruce Richardson <bruce.richardson@intel.com>

> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -117,6 +117,7 @@ F: lib/librte_eal/freebsd/BSDmakefile.meson
> >  F: meson_options.txt
> >  F: config/rte_config.h
> >  F: buildtools/gen-pmdinfo-cfile.sh
> > +F: buildtools/map_to_def.py
> >  F: buildtools/symlink-drivers-solibs.sh
> 
> 
> 
>
Thomas Monjalon June 17, 2019, 11:28 a.m. UTC | #4 
17/06/2019 13:02, Bruce Richardson:
> On Mon, Jun 17, 2019 at 12:33:58PM +0300, Thomas Monjalon wrote:
> > 13/06/2019 17:11, Thomas Monjalon:
> > > A script used with meson was missing in the list of files.
> > > 
> > > Fixes: 4c773788e05a ("build: generate Windows exports file")
> > > Cc: bruce.richardson@intel.com
> > > 
> > > Signed-off-by: Thomas Monjalon <thomas@monjalon.net>
> > 
> > 
> > Bruce, do you agree with following addition?
> > 
> 
> No objections. Sorry for not replying sooner.
> 
> Acked-by: Bruce Richardson <bruce.richardson@intel.com>

Applied, thanks
diff mbox series

Patch

diff --git a/MAINTAINERS b/MAINTAINERS
index 0212fe6d0..fdc083db9 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -117,6 +117,7 @@  F: lib/librte_eal/freebsd/BSDmakefile.meson
 F: meson_options.txt
 F: config/rte_config.h
 F: buildtools/gen-pmdinfo-cfile.sh
+F: buildtools/map_to_def.py
 F: buildtools/symlink-drivers-solibs.sh
 
 Public CI