Re: [PATCH v2] docs: Makefile: wrap SPHINXDIRS help text
From: Randy Dunlap
Date: Fri Dec 26 2025 - 16:38:14 EST
Hi,
On 12/26/25 12:56 AM, mustafaelrasheid@xxxxxxxxx wrote:
> From: Mustafa Elrasheid <mustafaelrasheid@xxxxxxxxx>
>
> When using `make help`, SPHINXDIR doesn't wrap around 80 characters,
> causing text to overflow or wrap in incorrect ways,
> which then makes the text difficult to read
>
> Signed-off-by: Mustafa Elrasheid <mustafaelrasheid@xxxxxxxxx>
> ---
> v2:
> - Added '-s' in fmt to prevent word segmenting at the end of lines.
> - Reduced width in `fmt -w` to account for the added indentation of
> `sed`
>
> Documentation/Makefile | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/Documentation/Makefile b/Documentation/Makefile
> index e96ac6dcac4f..e260df866f5e 100644
> --- a/Documentation/Makefile
> +++ b/Documentation/Makefile
> @@ -98,7 +98,7 @@ dochelp:
> @echo ' cleandocs - clean all generated files'
> @echo
> @echo ' make SPHINXDIRS="s1 s2" [target] Generate only docs of folder s1, s2'
> - @echo ' top level values for SPHINXDIRS are: $(_SPHINXDIRS)'
> + @echo ' top level values for SPHINXDIRS are: $(_SPHINXDIRS)' | fmt -s -w 72 | sed '1!s/^/ /'
> @echo ' you may also use a subdirectory like SPHINXDIRS=userspace-api/media,'
> @echo ' provided that there is an index.rst file at the subdirectory.'
> @echo
OK, that works.
I think that the output can be made a little nicer to look at like this:
@@ -98,7 +101,8 @@ dochelp:
@echo ' cleandocs - clean all generated files'
@echo
@echo ' make SPHINXDIRS="s1 s2" [target] Generate only docs of folder s1, s2'
- @echo ' top level values for SPHINXDIRS are: $(_SPHINXDIRS)'
+ @echo ' top level values for SPHINXDIRS are:'
+ @echo '$(_SPHINXDIRS)' | fmt -s -w 75 -g 75 | sed 's/^/ /'
@echo ' you may also use a subdirectory like SPHINXDIRS=userspace-api/media,'
@echo ' provided that there is an index.rst file at the subdirectory.'
@echo
although I would rather not assume a terminal width of 80.
thanks.
--
~Randy