diff mbox series

Documentation: rust: Fix arch support table

Message ID 20230213085920.19726-1-bagasdotme@gmail.com
State New
Headers
Series Documentation: rust: Fix arch support table |

Commit Message

Bagas Sanjaya Feb. 13, 2023, 8:59 a.m. UTC 
  Stephen Rothwell reported htmldocs warning when merging uml tree:

Documentation/rust/arch-support.rst:20: WARNING: Blank line required after table.

Fix the arch support table by removing extraneous simple table marker.

Link: https://lore.kernel.org/linux-next/20230213152714.78b844f4@canb.auug.org.au/
Fixes: 0438aadfa69a34 ("rust: arch/um: Add support for CONFIG_RUST under x86_64 UML")
Reported-by: Stephen Rothwell <sfr@canb.auug.org.au>
Signed-off-by: Bagas Sanjaya <bagasdotme@gmail.com>
---
 Documentation/rust/arch-support.rst | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)


base-commit: 990627acb87c5f6d502fa71baf7b985b8a4bf248

Comments

Richard Weinberger Feb. 13, 2023, 9:11 a.m. UTC | #1 
----- Ursprüngliche Mail -----
> Von: "Bagas Sanjaya" <bagasdotme@gmail.com>
> Stephen Rothwell reported htmldocs warning when merging uml tree:
> 
> Documentation/rust/arch-support.rst:20: WARNING: Blank line required after
> table.
> 
> Fix the arch support table by removing extraneous simple table marker.
> 
> Link:
> https://lore.kernel.org/linux-next/20230213152714.78b844f4@canb.auug.org.au/
> Fixes: 0438aadfa69a34 ("rust: arch/um: Add support for CONFIG_RUST under x86_64
> UML")
> Reported-by: Stephen Rothwell <sfr@canb.auug.org.au>
> Signed-off-by: Bagas Sanjaya <bagasdotme@gmail.com>
> ---
> Documentation/rust/arch-support.rst | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/Documentation/rust/arch-support.rst
> b/Documentation/rust/arch-support.rst
> index a526ca1c688be6..ed7f4f5b3cf157 100644
> --- a/Documentation/rust/arch-support.rst
> +++ b/Documentation/rust/arch-support.rst
> @@ -16,6 +16,6 @@ support corresponds to ``S`` values in the ``MAINTAINERS``
> file.
> Architecture  Level of support  Constraints
> ============  ================  ==============================================
> ``x86``       Maintained        ``x86_64`` only.
> -============  ================  ==============================================
> ``um``        Maintained        ``x86_64`` only.
> ============  ================  ==============================================
> +

Applied to linux-um next. :-)

Thanks,
//richard
Miguel Ojeda Feb. 13, 2023, 9:46 a.m. UTC | #2 
On Mon, Feb 13, 2023 at 10:11 AM Richard Weinberger <richard@nod.at> wrote:
>
> Applied to linux-um next. :-)

Ah, I didn't notice you picked the series (I think there was no
message) -- thanks Richard!

Since it is still unsorted, I am taking the chance to add a "good
first issue" for that:
https://github.com/Rust-for-Linux/linux/issues/965

Cheers,
Miguel
David Gow Feb. 14, 2023, 4:21 a.m. UTC | #3 
On Mon, 13 Feb 2023 at 17:00, Bagas Sanjaya <bagasdotme@gmail.com> wrote:
>
> Stephen Rothwell reported htmldocs warning when merging uml tree:
>
> Documentation/rust/arch-support.rst:20: WARNING: Blank line required after table.
>
> Fix the arch support table by removing extraneous simple table marker.
>
> Link: https://lore.kernel.org/linux-next/20230213152714.78b844f4@canb.auug.org.au/
> Fixes: 0438aadfa69a34 ("rust: arch/um: Add support for CONFIG_RUST under x86_64 UML")
> Reported-by: Stephen Rothwell <sfr@canb.auug.org.au>
> Signed-off-by: Bagas Sanjaya <bagasdotme@gmail.com>
> ---

Thanks for fixing this! I think I need to look into my sphinx version,
as this isn't the only warning it hasn't picked up...

Reviewed-by: David Gow <davidgow@google.com>

(And, as Miguel notes, next time we update this table, we should put
these in alphabetical order...)

Cheers,
-- David

>  Documentation/rust/arch-support.rst | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/Documentation/rust/arch-support.rst b/Documentation/rust/arch-support.rst
> index a526ca1c688be6..ed7f4f5b3cf157 100644
> --- a/Documentation/rust/arch-support.rst
> +++ b/Documentation/rust/arch-support.rst
> @@ -16,6 +16,6 @@ support corresponds to ``S`` values in the ``MAINTAINERS`` file.
>  Architecture  Level of support  Constraints
>  ============  ================  ==============================================
>  ``x86``       Maintained        ``x86_64`` only.
> -============  ================  ==============================================
>  ``um``        Maintained        ``x86_64`` only.
>  ============  ================  ==============================================
> +
>
> base-commit: 990627acb87c5f6d502fa71baf7b985b8a4bf248
> --
> An old man doll... just what I always wanted! - Clara
>
Vincenzo Palazzo Feb. 15, 2023, 10:31 p.m. UTC | #4 
> Stephen Rothwell reported htmldocs warning when merging uml tree:
>
> Documentation/rust/arch-support.rst:20: WARNING: Blank line required after table.
>
> Fix the arch support table by removing extraneous simple table marker.
>
> Link: https://lore.kernel.org/linux-next/20230213152714.78b844f4@canb.auug.org.au/
> Fixes: 0438aadfa69a34 ("rust: arch/um: Add support for CONFIG_RUST under x86_64 UML")
> Reported-by: Stephen Rothwell <sfr@canb.auug.org.au>
> Signed-off-by: Bagas Sanjaya <bagasdotme@gmail.com>
> ---

Reviewed-by: Vincenzo Palazzo <vincenzopalazzodev@gmail.com>
diff mbox series

Patch

diff --git a/Documentation/rust/arch-support.rst b/Documentation/rust/arch-support.rst
index a526ca1c688be6..ed7f4f5b3cf157 100644
--- a/Documentation/rust/arch-support.rst
+++ b/Documentation/rust/arch-support.rst
@@ -16,6 +16,6 @@  support corresponds to ``S`` values in the ``MAINTAINERS`` file.
 Architecture  Level of support  Constraints
 ============  ================  ==============================================
 ``x86``       Maintained        ``x86_64`` only.
-============  ================  ==============================================
 ``um``        Maintained        ``x86_64`` only.
 ============  ================  ==============================================
+