[AMD Official Use Only]
> -----Original Message-----
> From: Dmitry Kozlyuk <dmitry.kozl...@gmail.com>
> Sent: Tuesday, March 29, 2022 4:21 AM
> To: Varghese, Vipin <vipin.vargh...@amd.com>
> Cc: Thomas Monjalon <tho...@monjalon.net>;
> david.march...@redhat.com; Tummala, Sivaprasad
> <sivaprasad.tumm...@amd.com>; dev@dpdk.org
> Subject: Re: [PATCH] meson: update doc logic for Windows
>
> [CAUTION: External Email]
>
> 2022-03-28 12:29 (UTC+0000), Varghese, Vipin:
> [...]
> > > We did get error from `doc/meson` stating ` echo command not
> > > available on windows` for both cmd and powershell for the line ```
> > > run_target('doc', command: [echo, message, doc_target_names],
> > > depends: doc_targets)
> > > ```
> >
> > This line never searches for "echo" command, e8c90926bdde ("doc: fix
> build on Windows with Meson 0.58") introduced "echo" variable precisely to
> avoid this.
> > Can you give the full log?
> >
> > > > Steps to reproduce the error:
> > > > - Install dependencies doxygen & sphinix build on Windwos server
> 2019.
> > > > - Build DPDK with option enable_docs=true for API or User Guide.
> > > >
> > > > This produces error
> > > > ```
> > > > FAILED: doc/api/examples.dox
> > > > sh -e dpdk/doc/api/generate_examples.sh dpdk/examples
> > > > doc/api/examples.dox ```
>
> Thanks for confirming that the "echo" issue did not reappear.
>
> [...]
> > Are you recommending
> >
> > Windows libraries and binaries should be cross build from linux (where all
> dependencies and shell script works)?
> >
> > Or
> >
> > let us enhance the check for windows native build, if yes for option `-
> Denable_docs=true, we can stop the build for docs`.
> >
> > If the latter is suggestion I will share v2 patch to check the same.
>
> I recommend enhancing the Windows native build by enabling full-fledged
> docs generation :) I'm reluctant to accept this patch, even corrected, because
> it effectively works against better Windows support:
> - it makes less meson code reachable on Windows,
> allowing it to accumulate incompatible changes;
> - it treats Windows as special,
> while we want it to be a platform supported on par with the others.
> That being said, the latter option is acceptable for sure, because it fixes a
> build failure in certain configuration.
As stated in patch with Windows native build using meson `-Denable_docs=true`
result in build error
```
FAILED: doc/api/examples.dox
sh -e dpdk/doc/api/generate_examples.sh dpdk/examples doc/api/examples.dox ```
```
The reason is because shell script is not supported on Windows CMD. The scope
of the patch is addressing the issue `by continuing the build processing with a
warning to user`.
IMO the current patch can be deprecated or removed from mainline one `once full
fledge docs generation is supported by native Windows build` as suggested by
Dmitry.
