[dpdk-dev] [PATCH] doc: fix helloworld build on Windows

[dpdk-dev] [PATCH] doc: fix helloworld build on Windows

[adham]

From: Adham Masarwah <adham@mellanox.com>

The option -Dexamples=helloworld is missing.
The helloworld binary name was wrong.
Forcing clang may be required in some environments.

Fixes: 196c650b8b63 ("doc: add guide for Windows")

Signed-off-by: Adham Masarwah <adham@mellanox.com>
---
 doc/guides/windows_gsg/build_dpdk.rst | 16 ++++++++++++----
 1 file changed, 12 insertions(+), 4 deletions(-)

diff --git a/doc/guides/windows_gsg/build_dpdk.rst b/doc/guides/windows_gsg/build_dpdk.rst
index f48805236..6711e07e2 100644
--- a/doc/guides/windows_gsg/build_dpdk.rst
+++ b/doc/guides/windows_gsg/build_dpdk.rst
@@ -59,22 +59,30 @@ default.
 Using the ninja backend
 ~~~~~~~~~~~~~~~~~~~~~~~~
 
+Specifying the compiler might be required to complete the meson command.
+
+.. code-block:: console
+
+    set CC=clang
+
+To compile the examples, the flag ``-Dexamples`` is required.
+
 .. code-block:: console
 
     cd C:\Users\me\dpdk
-    meson build
+    meson -Dexamples=helloworld build
     cd build
     ninja
 
 Run the helloworld example
 ==========================
 
-Navigate to the build directory and run `dpdk-helloworld.exe`.
+Navigate to the examples in the build directory and run `dpdk-helloworld.exe`.
 
 .. code-block:: console
 
-    cd C:\Users\me\dpdk\build
-    helloworld.exe
+    cd C:\Users\me\dpdk\build\examples
+    dpdk-helloworld.exe
     hello from core 1
     hello from core 3
     hello from core 0
-- 
2.21.0.windows.1

Re: [dpdk-dev] [PATCH] doc: fix helloworld build on Windows

[Bruce Richardson]

On Mon, May 20, 2019 at 11:19:52AM +0300, adham@mellanox.com wrote:
> From: Adham Masarwah <adham@mellanox.com>
> 
> The option -Dexamples=helloworld is missing.
> The helloworld binary name was wrong.
> Forcing clang may be required in some environments.
> 
> Fixes: 196c650b8b63 ("doc: add guide for Windows")
> 
> Signed-off-by: Adham Masarwah <adham@mellanox.com>
> ---
Acked-by: Bruce Richardson <bruce.richardson@intel.com>

Re: [dpdk-dev] [PATCH] doc: fix helloworld build on Windows

[Thomas Monjalon]

20/05/2019 11:04, Bruce Richardson:
> On Mon, May 20, 2019 at 11:19:52AM +0300, adham@mellanox.com wrote:
> > From: Adham Masarwah <adham@mellanox.com>
> > 
> > The option -Dexamples=helloworld is missing.
> > The helloworld binary name was wrong.
> > Forcing clang may be required in some environments.
> > 
> > Fixes: 196c650b8b63 ("doc: add guide for Windows")
> > 
> > Signed-off-by: Adham Masarwah <adham@mellanox.com>
> > ---
> Acked-by: Bruce Richardson <bruce.richardson@intel.com>

Applied, thanks