Re: [PATCH] Documentation: kunit: eliminate code-block warnings

From: Randy Dunlap
Date: Sat Jun 25 2022 - 11:12:09 EST




On 6/24/22 12:27, Jonathan Corbet wrote:
> Randy Dunlap <rdunlap@xxxxxxxxxxxxx> writes:
>
>> Fix Sphinx complaints about code-block directive missing an argument.
>> For start.rst, add "none" since that is already heavily used in that
>> file. For run_wrapper.rst, use the simpler "::" literal block instead.
>
> [Sorry that this fell through the cracks; I'm never quite sure who is
> going to handle kunit patches]
>
>> dev-tools/kunit/start.rst:83: WARNING: Error in "code-block" directive:
>> 1 argument(s) required, 0 supplied.
>>
>> dev-tools/kunit/run_wrapper.rst:17: WARNING: Error in "code-block" directive:
>> 1 argument(s) required, 0 supplied.
>
> So which version of Sphinx are you using? The language argument became
> optional in 2.0, so you'd need to be running something pretty ancient to
> see this.
>
> Ah, I see 1.8.5 in your later message...how wedded are you to that
> version?

It's what ships with OpenSuse Leap 15.3, which I have been using for quite
a long time.

I see that there is now OpenSuse Leap 15.4, so I could upgrade to that,
but I don't know what version on Sphinx it uses.

> Ostensibly we support back to 1.7, so I guess we should stick by its
> rules. But the better solution, I think, is to raise our minimum
> version; I think I'll look into that shortly.
>

>>
>> Run command:
>>
>> -.. code-block::
>> +::
>
> A much nicer fix for these would have been just:
>
> Run command::

Yeah, that is nicer.

>
> Oh well, I've applied it.

thanks.

--
~Randy