public inbox for dev@dpdk.org
 help / color / mirror / Atom feed
* [PATCH] doc: raise an error on warnings for dts
@ 2026-01-21 10:01 David Marchand
  2026-01-21 10:31 ` Thomas Monjalon
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: David Marchand @ 2026-01-21 10:01 UTC (permalink / raw)
  To: dev; +Cc: probb, bruce.richardson, thomas

Similarly to commit 6572fc926037 ("doc: make sphinx comply with ..."),
pass -W to sphinx to catch warnings.

Signed-off-by: David Marchand <david.marchand@redhat.com>
---
 doc/api/dts/meson.build | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/doc/api/dts/meson.build b/doc/api/dts/meson.build
index 8a6d061879..fc147f13f3 100644
--- a/doc/api/dts/meson.build
+++ b/doc/api/dts/meson.build
@@ -12,6 +12,9 @@ if python_ver_satisfied != 0
 endif
 
 extra_sphinx_args = ['-E', '-c', join_paths(doc_source_dir, 'guides')]
+if get_option('werror')
+    extra_sphinx_args += '-W'
+endif
 
 # DTS docs are generated inside main Doxygen docs build directory
 # DTS docs relies on main docs to install it so does not define install dir
-- 
2.52.0


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

* Re: [PATCH] doc: raise an error on warnings for dts
  2026-01-21 10:01 [PATCH] doc: raise an error on warnings for dts David Marchand
@ 2026-01-21 10:31 ` Thomas Monjalon
  2026-01-21 11:03 ` Marat Khalili
  2026-01-21 14:40 ` David Marchand
  2 siblings, 0 replies; 4+ messages in thread
From: Thomas Monjalon @ 2026-01-21 10:31 UTC (permalink / raw)
  To: David Marchand; +Cc: dev, probb, bruce.richardson

21/01/2026 11:01, David Marchand:
> Similarly to commit 6572fc926037 ("doc: make sphinx comply with ..."),
> pass -W to sphinx to catch warnings.
> 
> Signed-off-by: David Marchand <david.marchand@redhat.com>

Acked-by: Thomas Monjalon <thomas@monjalon.net>

Note: "dts" could be uppercased in the title



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

* RE: [PATCH] doc: raise an error on warnings for dts
  2026-01-21 10:01 [PATCH] doc: raise an error on warnings for dts David Marchand
  2026-01-21 10:31 ` Thomas Monjalon
@ 2026-01-21 11:03 ` Marat Khalili
  2026-01-21 14:40 ` David Marchand
  2 siblings, 0 replies; 4+ messages in thread
From: Marat Khalili @ 2026-01-21 11:03 UTC (permalink / raw)
  To: David Marchand, dev@dpdk.org
  Cc: probb@iol.unh.edu, bruce.richardson@intel.com,
	thomas@monjalon.net

> -----Original Message-----
> From: David Marchand <david.marchand@redhat.com>
> Sent: Wednesday 21 January 2026 10:01
> To: dev@dpdk.org
> Cc: probb@iol.unh.edu; bruce.richardson@intel.com; thomas@monjalon.net
> Subject: [PATCH] doc: raise an error on warnings for dts
> 
> Similarly to commit 6572fc926037 ("doc: make sphinx comply with ..."),
> pass -W to sphinx to catch warnings.
> 
> Signed-off-by: David Marchand <david.marchand@redhat.com>
> ---
>  doc/api/dts/meson.build | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/doc/api/dts/meson.build b/doc/api/dts/meson.build
> index 8a6d061879..fc147f13f3 100644
> --- a/doc/api/dts/meson.build
> +++ b/doc/api/dts/meson.build
> @@ -12,6 +12,9 @@ if python_ver_satisfied != 0
>  endif
> 
>  extra_sphinx_args = ['-E', '-c', join_paths(doc_source_dir, 'guides')]
> +if get_option('werror')
> +    extra_sphinx_args += '-W'
> +endif
> 
>  # DTS docs are generated inside main Doxygen docs build directory
>  # DTS docs relies on main docs to install it so does not define install dir
> --
> 2.52.0

Acked-by: Marat Khalili <marat.khalili@huawei.com>
Tested-by: Marat Khalili <marat.khalili@huawei.com>

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

* Re: [PATCH] doc: raise an error on warnings for dts
  2026-01-21 10:01 [PATCH] doc: raise an error on warnings for dts David Marchand
  2026-01-21 10:31 ` Thomas Monjalon
  2026-01-21 11:03 ` Marat Khalili
@ 2026-01-21 14:40 ` David Marchand
  2 siblings, 0 replies; 4+ messages in thread
From: David Marchand @ 2026-01-21 14:40 UTC (permalink / raw)
  To: David Marchand; +Cc: dev, probb, bruce.richardson, thomas, Marat Khalili

On Wed, 21 Jan 2026 at 11:01, David Marchand <david.marchand@redhat.com> wrote:
>
> Similarly to commit 6572fc926037 ("doc: make sphinx comply with ..."),
> pass -W to sphinx to catch warnings.
>
> Signed-off-by: David Marchand <david.marchand@redhat.com>
> ---
>  doc/api/dts/meson.build | 3 +++
>  1 file changed, 3 insertions(+)
>
> diff --git a/doc/api/dts/meson.build b/doc/api/dts/meson.build
> index 8a6d061879..fc147f13f3 100644
> --- a/doc/api/dts/meson.build
> +++ b/doc/api/dts/meson.build
> @@ -12,6 +12,9 @@ if python_ver_satisfied != 0
>  endif
>
>  extra_sphinx_args = ['-E', '-c', join_paths(doc_source_dir, 'guides')]
> +if get_option('werror')
> +    extra_sphinx_args += '-W'
> +endif
>
>  # DTS docs are generated inside main Doxygen docs build directory
>  # DTS docs relies on main docs to install it so does not define install dir

Applied, thanks.


-- 
David Marchand


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

end of thread, other threads:[~2026-01-21 14:40 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-01-21 10:01 [PATCH] doc: raise an error on warnings for dts David Marchand
2026-01-21 10:31 ` Thomas Monjalon
2026-01-21 11:03 ` Marat Khalili
2026-01-21 14:40 ` David Marchand

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox