[dpdk-dev] [PATCH] kvargs: fix building with meson outside source tree

[dpdk-dev] [PATCH] kvargs: fix building with meson outside source tree

[Bruce Richardson]

When building DPDK with meson with the <builddir> being outside the source
directory, an error was generated as the path to the EAL headers was not
found. The path specified for the includes backed out unnecessarily far and
so broke when the build directory was not in the expected location of
just inside the source dir. Changing the path to be shorter, just going
down one level and then into EAL builds for both cases - where builddir is
in source directory, and also when it's outside it.

Fixes: 092ee516496d ("kvargs: build before EAL")

Reported-by: Anatoly Burakov <anatoly.burakov@intel.com>
Signed-off-by: Bruce Richardson <bruce.richardson@intel.com>
---
 lib/librte_kvargs/meson.build | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lib/librte_kvargs/meson.build b/lib/librte_kvargs/meson.build
index a1c724961..acd3e5432 100644
--- a/lib/librte_kvargs/meson.build
+++ b/lib/librte_kvargs/meson.build
@@ -2,7 +2,7 @@
 # Copyright(c) 2017 Intel Corporation
 
 includes = [global_inc]
-includes += include_directories('../../../lib/librte_eal/common/include')
+includes += include_directories('../librte_eal/common/include')
 
 version = 1
 sources = files('rte_kvargs.c')
-- 
2.17.1

Re: [dpdk-dev] [PATCH] kvargs: fix building with meson outside source tree

[Burakov, Anatoly]

On 19-Jul-18 3:26 PM, Bruce Richardson wrote:
> When building DPDK with meson with the <builddir> being outside the source
> directory, an error was generated as the path to the EAL headers was not
> found. The path specified for the includes backed out unnecessarily far and
> so broke when the build directory was not in the expected location of
> just inside the source dir. Changing the path to be shorter, just going
> down one level and then into EAL builds for both cases - where builddir is
> in source directory, and also when it's outside it.
> 
> Fixes: 092ee516496d ("kvargs: build before EAL")
> 
> Reported-by: Anatoly Burakov <anatoly.burakov@intel.com>
> Signed-off-by: Bruce Richardson <bruce.richardson@intel.com>
> ---
>   lib/librte_kvargs/meson.build | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/lib/librte_kvargs/meson.build b/lib/librte_kvargs/meson.build
> index a1c724961..acd3e5432 100644
> --- a/lib/librte_kvargs/meson.build
> +++ b/lib/librte_kvargs/meson.build
> @@ -2,7 +2,7 @@
>   # Copyright(c) 2017 Intel Corporation
>   
>   includes = [global_inc]
> -includes += include_directories('../../../lib/librte_eal/common/include')
> +includes += include_directories('../librte_eal/common/include')
>   
>   version = 1
>   sources = files('rte_kvargs.c')
> 

Tested-by: Anatoly Burakov <anatoly.burakov@intel.com>

-- 
Thanks,
Anatoly

Re: [dpdk-dev] [PATCH] kvargs: fix building with meson outside source tree

[Thomas Monjalon]

19/07/2018 16:45, Burakov, Anatoly:
> On 19-Jul-18 3:26 PM, Bruce Richardson wrote:
> > When building DPDK with meson with the <builddir> being outside the source
> > directory, an error was generated as the path to the EAL headers was not
> > found. The path specified for the includes backed out unnecessarily far and
> > so broke when the build directory was not in the expected location of
> > just inside the source dir. Changing the path to be shorter, just going
> > down one level and then into EAL builds for both cases - where builddir is
> > in source directory, and also when it's outside it.
> > 
> > Fixes: 092ee516496d ("kvargs: build before EAL")
> > 
> > Reported-by: Anatoly Burakov <anatoly.burakov@intel.com>
> > Signed-off-by: Bruce Richardson <bruce.richardson@intel.com>
> 
> Tested-by: Anatoly Burakov <anatoly.burakov@intel.com>

Applied, thanks