[dpdk-dev] [PATCH] test: remove hack for private header inclusion

[dpdk-dev] [PATCH] test: remove hack for private header inclusion

[David Marchand]

This hack was needed with the make build system.
With meson, any private header from a library is visible as long as a
dependency to this library is expressed.

Signed-off-by: David Marchand <david.marchand@redhat.com>
---
 app/test/test_eal_fs.c         | 3 +--
 app/test/test_memzone.c        | 3 ++-
 app/test/test_telemetry_json.c | 3 ++-
 3 files changed, 5 insertions(+), 4 deletions(-)

diff --git a/app/test/test_eal_fs.c b/app/test/test_eal_fs.c
index bb93b82a43..39ac6961b3 100644
--- a/app/test/test_eal_fs.c
+++ b/app/test/test_eal_fs.c
@@ -8,8 +8,7 @@
 #include <string.h>
 #include <errno.h>
 
-/* eal_filesystem.h is not a public header file, so use relative path */
-#include "../../lib/eal/common/eal_filesystem.h"
+#include "eal_filesystem.h"
 
 static int
 test_parse_sysfs_value(void)
diff --git a/app/test/test_memzone.c b/app/test/test_memzone.c
index 03a9d1d3bb..27b8b52fcd 100644
--- a/app/test/test_memzone.c
+++ b/app/test/test_memzone.c
@@ -18,7 +18,8 @@
 #include <rte_string_fns.h>
 #include <rte_errno.h>
 #include <rte_malloc.h>
-#include "../../lib/eal/common/malloc_elem.h"
+
+#include "malloc_elem.h"
 
 #include "test.h"
 
diff --git a/app/test/test_telemetry_json.c b/app/test/test_telemetry_json.c
index 3171ab12ec..790181d316 100644
--- a/app/test/test_telemetry_json.c
+++ b/app/test/test_telemetry_json.c
@@ -4,7 +4,8 @@
 
 #include <string.h>
 
-#include "../../lib/telemetry/telemetry_json.h"
+#include "telemetry_json.h"
+
 #include "test.h"
 
 static int
-- 
2.23.0

Re: [dpdk-dev] [PATCH] test: remove hack for private header inclusion

[Bruce Richardson]

On Thu, Jun 24, 2021 at 01:53:19PM +0200, David Marchand wrote:
> This hack was needed with the make build system.
> With meson, any private header from a library is visible as long as a
> dependency to this library is expressed.
> 
> Signed-off-by: David Marchand <david.marchand@redhat.com>
> ---
Acked-by: Bruce Richardson <bruce.richardson@intel.com>

Re: [dpdk-dev] [PATCH] test: remove hack for private header inclusion

[Tyler Retzlaff]

On Thu, Jun 24, 2021 at 01:53:19PM +0200, David Marchand wrote:
> This hack was needed with the make build system.
> With meson, any private header from a library is visible as long as a
> dependency to this library is expressed.
> 
> Signed-off-by: David Marchand <david.marchand@redhat.com>
> ---

Acked-by: Tyler Retzlaff <roretzla@linux.microsoft.com>

Re: [dpdk-dev] [PATCH] test: remove hack for private header inclusion

[David Marchand]

On Thu, Jun 24, 2021 at 1:53 PM David Marchand
<david.marchand@redhat.com> wrote:
>
> This hack was needed with the make build system.
> With meson, any private header from a library is visible as long as a
> dependency to this library is expressed.
>
> Signed-off-by: David Marchand <david.marchand@redhat.com>

Acked-by: Bruce Richardson <bruce.richardson@intel.com>
Acked-by: Tyler Retzlaff <roretzla@linux.microsoft.com>

Applied.

-- 
David Marchand