Re: [PATCH dpdk] meson: properly disable docs

2024-10-22 Thread Thomas Monjalon
22/10/2024 17:49, Bruce Richardson: > On Tue, Oct 22, 2024 at 05:31:47PM +0200, Robin Jarry wrote: > > Bruce Richardson, Oct 22, 2024 at 17:25: > > > No strong objection to this. Just FYI, the reason it is currently done as > > > it is in the build, is that even if you configure a build without the

Re: [PATCH dpdk] meson: properly disable docs

2024-10-22 Thread Bruce Richardson
On Tue, Oct 22, 2024 at 05:31:47PM +0200, Robin Jarry wrote: > Bruce Richardson, Oct 22, 2024 at 17:25: > > No strong objection to this. Just FYI, the reason it is currently done as > > it is in the build, is that even if you configure a build without the docs > > enabled, you can just build the do

Re: [PATCH dpdk] meson: properly disable docs

2024-10-22 Thread Stephen Hemminger
On Tue, 22 Oct 2024 17:11:42 +0200 Robin Jarry wrote: > Fix the following meson warning when -Denable_docs=false: > > $ meson setup build -Denable_docs=false -Denable_drivers=net/null \ > -Denable_libs=rib -Ddisable_apps=* --wipe --fatal-meson-warnings > ... > Program doxygen found: YES

Re: [PATCH dpdk] meson: properly disable docs

2024-10-22 Thread Robin Jarry
Bruce Richardson, Oct 22, 2024 at 17:25: No strong objection to this. Just FYI, the reason it is currently done as it is in the build, is that even if you configure a build without the docs enabled, you can just build the docs explicitly by doing "ninja -C build doc". That is so that one can do a

Re: [PATCH dpdk] meson: properly disable docs

2024-10-22 Thread Bruce Richardson
On Tue, Oct 22, 2024 at 05:11:42PM +0200, Robin Jarry wrote: > Fix the following meson warning when -Denable_docs=false: > > $ meson setup build -Denable_docs=false -Denable_drivers=net/null \ > -Denable_libs=rib -Ddisable_apps=* --wipe --fatal-meson-warnings > ... > Program doxygen found

[PATCH dpdk] meson: properly disable docs

2024-10-22 Thread Robin Jarry
Fix the following meson warning when -Denable_docs=false: $ meson setup build -Denable_docs=false -Denable_drivers=net/null \ -Denable_libs=rib -Ddisable_apps=* --wipe --fatal-meson-warnings ... Program doxygen found: YES (/usr/bin/doxygen) Configuring doxy-api-html.conf using configurat