[PATCH v3 13/16] docs: conf.py: add include_pattern to speedup

Mauro Carvalho Chehab posted 16 patches 3 months, 4 weeks ago
There is a newer version of this series
[PATCH v3 13/16] docs: conf.py: add include_pattern to speedup
Posted by Mauro Carvalho Chehab 3 months, 4 weeks ago
Now that we have a parser for yaml, use include_pattern, adding
just yaml files from the only directory we currently process.

Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
---
 Documentation/conf.py | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/Documentation/conf.py b/Documentation/conf.py
index b8668bcaf090..60e6998e49e1 100644
--- a/Documentation/conf.py
+++ b/Documentation/conf.py
@@ -222,10 +222,13 @@ language = 'en'
 
 # List of patterns, relative to source directory, that match files and
 # directories to ignore when looking for source files.
+include_patterns = [
+	'**.rst',
+	'netlink/specs/*.yaml',
+]
+
 exclude_patterns = [
 	'output',
-	'devicetree/bindings/**.yaml',
-	'netlink/*.yaml',
 ]
 
 # The reST default role (used for this markup: `text`) to use for all
-- 
2.49.0
Re: [PATCH v3 13/16] docs: conf.py: add include_pattern to speedup
Posted by Donald Hunter 3 months, 4 weeks ago
Mauro Carvalho Chehab <mchehab+huawei@kernel.org> writes:

> Now that we have a parser for yaml, use include_pattern, adding
> just yaml files from the only directory we currently process.
>
> Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
> ---
>  Documentation/conf.py | 7 +++++--
>  1 file changed, 5 insertions(+), 2 deletions(-)
>
> diff --git a/Documentation/conf.py b/Documentation/conf.py
> index b8668bcaf090..60e6998e49e1 100644
> --- a/Documentation/conf.py
> +++ b/Documentation/conf.py
> @@ -222,10 +222,13 @@ language = 'en'
>  
>  # List of patterns, relative to source directory, that match files and
>  # directories to ignore when looking for source files.
> +include_patterns = [
> +	'**.rst',
> +	'netlink/specs/*.yaml',
> +]

It's unfortunate where you added the include patterns because the
adjacent comment is specific to exclude patterns. I suggest rewording
the comment to be relevant to both.

> +
>  exclude_patterns = [
>  	'output',
> -	'devicetree/bindings/**.yaml',
> -	'netlink/*.yaml',
>  ]
>  
>  # The reST default role (used for this markup: `text`) to use for all