All of lore.kernel.org
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH] doc: fix helloworld build on Windows
@ 2019-05-20  8:19 adham
  2019-05-20  9:04 ` Bruce Richardson
  0 siblings, 1 reply; 3+ messages in thread
From: adham @ 2019-05-20  8:19 UTC (permalink / raw)
  To: dev; +Cc: Adham Masarwah

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


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [dpdk-dev] [PATCH] doc: fix helloworld build on Windows
  2019-05-20  8:19 [dpdk-dev] [PATCH] doc: fix helloworld build on Windows adham
@ 2019-05-20  9:04 ` Bruce Richardson
  2019-05-29 10:55   ` Thomas Monjalon
  0 siblings, 1 reply; 3+ messages in thread
From: Bruce Richardson @ 2019-05-20  9:04 UTC (permalink / raw)
  To: adham; +Cc: dev

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>

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [dpdk-dev] [PATCH] doc: fix helloworld build on Windows
  2019-05-20  9:04 ` Bruce Richardson
@ 2019-05-29 10:55   ` Thomas Monjalon
  0 siblings, 0 replies; 3+ messages in thread
From: Thomas Monjalon @ 2019-05-29 10:55 UTC (permalink / raw)
  To: adham; +Cc: dev, Bruce Richardson

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




^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2019-05-29 10:56 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-05-20  8:19 [dpdk-dev] [PATCH] doc: fix helloworld build on Windows adham
2019-05-20  9:04 ` Bruce Richardson
2019-05-29 10:55   ` Thomas Monjalon

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.