Hi Quentin, Am Dienstag, dem 01.04.2025 um 12:07 +0200 schrieb Quentin Schulz: > Hi Leonard, > > On 4/1/25 11:49 AM, Leonard Anderweit wrote: > > Remove double : before code-block. While at it, use code-block for > > all > > Why?
I just found it odd that the two styles are mixed here. > > https://www.sphinx-doc.org/en/master/usage/restructuredtext/basics.html#literal-blocks > > is valid. > > We could think about changing the default highlighting language to > bash > maybe, I don't know. > > If you use ..code-block instead of :: for syntax highlighting, then > say > that in the commit log :) > > Can be two commits though, one for fixing the :: following by a > ..code-block, and another one for switching the :: to ..code-block:: > with syntax highlighting. Ok, I'll split the changes into two commits. Leonard > > > bash commands. > > > > Signed-off-by: Leonard Anderweit <[email protected]> > > --- > > doc/build/docker.rst | 10 +++++++--- > > 1 file changed, 7 insertions(+), 3 deletions(-) > > > > diff --git a/doc/build/docker.rst b/doc/build/docker.rst > > index 01ed35050908..4974a98d4af5 100644 > > --- a/doc/build/docker.rst > > +++ b/doc/build/docker.rst > > @@ -12,7 +12,9 @@ You will need a multi-platform container, > > otherwise this error is shown:: > > ERROR: Multi-platform build is not supported for the docker > > driver. > > Switch to a different driver, or turn on the containerd image > > store, and try again. > > > > -You can add a simple one with:: > > +You can add a simple one with: > > + > > +.. code-block:: bash > > > > sudo docker buildx create --name multiarch --driver docker- > > container --use > > > > @@ -20,7 +22,9 @@ This will result in a builder that will use QEMU > > for the non-native > > architectures request in a build. While both amd64 and arm64 > > happen in > > parallel, the non-native part will take considerably longer as it > > must use QEMU > > to emulate the foreign code. An alternative, if you have accesss > > to reasonably > > -fast amd64 (i.e. 64-bit x86) and arm64 machines is:: > > +fast amd64 (i.e. 64-bit x86) and arm64 machines is: > > + > > +.. code-block:: bash > > > > sudo docker buildx create --name multiarch-multinode --node > > localNode --bootstrap --use > > sudo docker buildx create --name multiarch-multinode --append > > --node remoteNode --bootstrap ssh://user@host > > @@ -28,7 +32,7 @@ fast amd64 (i.e. 64-bit x86) and arm64 machines > > is:: > > And this will result in a builder named multiarch-multinode that > > will build > > each platform natively on each node. > > > > -To build the image yourself:: > > +To build the image yourself: > > This one change is fine though (or removing the next line). > > > > > .. code-block:: bash > > > > Cheers, > Quentin
