* [PATCH] tools/docs: sphinx-build-wrapper: -q is a boolean, not an integer
@ 2025-10-07 15:43 Mauro Carvalho Chehab
2025-10-07 15:58 ` Konstantin Ryabitsev
2025-10-08 7:20 ` Jonathan Corbet
0 siblings, 2 replies; 3+ messages in thread
From: Mauro Carvalho Chehab @ 2025-10-07 15:43 UTC (permalink / raw)
To: Linux Doc Mailing List, Jonathan Corbet
Cc: Mauro Carvalho Chehab, Akira Yokosawa, Konstantin Ryabitsev,
Mauro Carvalho Chehab, linux-kernel
As reported by Konstantin, sphinx-build -q is a boolean, not an integer.
Fix the code.
Reported-by: Konstantin Ryabitsev <konstantin@linuxfoundation.org>
Closes: https://lore.kernel.org/all/871pnepxfy.fsf@trenco.lwn.net/
Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
---
tools/docs/sphinx-build-wrapper | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tools/docs/sphinx-build-wrapper b/tools/docs/sphinx-build-wrapper
index a74998e8309a..3e6d166d4102 100755
--- a/tools/docs/sphinx-build-wrapper
+++ b/tools/docs/sphinx-build-wrapper
@@ -141,7 +141,7 @@ class SphinxBuilder:
#
parser = argparse.ArgumentParser()
parser.add_argument('-j', '--jobs', type=int)
- parser.add_argument('-q', '--quiet', type=int)
+ parser.add_argument('-q', '--quiet', action='store_true')
#
# Other sphinx-build arguments go as-is, so place them
--
2.51.0
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [PATCH] tools/docs: sphinx-build-wrapper: -q is a boolean, not an integer
2025-10-07 15:43 [PATCH] tools/docs: sphinx-build-wrapper: -q is a boolean, not an integer Mauro Carvalho Chehab
@ 2025-10-07 15:58 ` Konstantin Ryabitsev
2025-10-08 7:20 ` Jonathan Corbet
1 sibling, 0 replies; 3+ messages in thread
From: Konstantin Ryabitsev @ 2025-10-07 15:58 UTC (permalink / raw)
To: Mauro Carvalho Chehab
Cc: Linux Doc Mailing List, Jonathan Corbet, Akira Yokosawa,
Mauro Carvalho Chehab, linux-kernel
On Tue, Oct 07, 2025 at 05:43:12PM +0200, Mauro Carvalho Chehab wrote:
> As reported by Konstantin, sphinx-build -q is a boolean, not an integer.
>
> Fix the code.
>
> Reported-by: Konstantin Ryabitsev <konstantin@linuxfoundation.org>
Reviewed-by: Konstantin Ryabitsev <konstantin@linuxfoundation.org>
-K
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] tools/docs: sphinx-build-wrapper: -q is a boolean, not an integer
2025-10-07 15:43 [PATCH] tools/docs: sphinx-build-wrapper: -q is a boolean, not an integer Mauro Carvalho Chehab
2025-10-07 15:58 ` Konstantin Ryabitsev
@ 2025-10-08 7:20 ` Jonathan Corbet
1 sibling, 0 replies; 3+ messages in thread
From: Jonathan Corbet @ 2025-10-08 7:20 UTC (permalink / raw)
To: Mauro Carvalho Chehab, Linux Doc Mailing List
Cc: Mauro Carvalho Chehab, Akira Yokosawa, Konstantin Ryabitsev,
Mauro Carvalho Chehab, linux-kernel
Mauro Carvalho Chehab <mchehab+huawei@kernel.org> writes:
> As reported by Konstantin, sphinx-build -q is a boolean, not an integer.
>
> Fix the code.
>
> Reported-by: Konstantin Ryabitsev <konstantin@linuxfoundation.org>
> Closes: https://lore.kernel.org/all/871pnepxfy.fsf@trenco.lwn.net/
> Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
> ---
> tools/docs/sphinx-build-wrapper | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tools/docs/sphinx-build-wrapper b/tools/docs/sphinx-build-wrapper
> index a74998e8309a..3e6d166d4102 100755
> --- a/tools/docs/sphinx-build-wrapper
> +++ b/tools/docs/sphinx-build-wrapper
> @@ -141,7 +141,7 @@ class SphinxBuilder:
> #
> parser = argparse.ArgumentParser()
> parser.add_argument('-j', '--jobs', type=int)
> - parser.add_argument('-q', '--quiet', type=int)
> + parser.add_argument('-q', '--quiet', action='store_true')
Applied to the build-script branch, thanks.
jon
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2025-10-08 7:21 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2025-10-07 15:43 [PATCH] tools/docs: sphinx-build-wrapper: -q is a boolean, not an integer Mauro Carvalho Chehab
2025-10-07 15:58 ` Konstantin Ryabitsev
2025-10-08 7:20 ` Jonathan Corbet
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®