2023-03-20 12:43:43

by Bagas Sanjaya

[permalink] [raw]
Subject: [PATCH] Documentation: maintainer-tip: Rectify link to "Describe your changes" section of submitting-patches.rst

The general changelog rules for the tip tree refers to "Describe your
changes" section of submitting patches guide. However, the internal link
reference targets to non-existent "submittingpatches" label, which
brings reader to the top of the linked doc.

Correct the target. No changes to submitting-patches.rst since the
required label is already there.

Fixes: 31c9d7c8297558 ("Documentation/process: Add tip tree handbook")
Signed-off-by: Bagas Sanjaya <[email protected]>
---
This patch is based on core/urgent branch of tip tree.

Documentation/process/maintainer-tip.rst | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/Documentation/process/maintainer-tip.rst b/Documentation/process/maintainer-tip.rst
index 572a3289c9cbf3..178c95fd17dcad 100644
--- a/Documentation/process/maintainer-tip.rst
+++ b/Documentation/process/maintainer-tip.rst
@@ -128,8 +128,8 @@ uppercase letter and should be written in imperative tone.
Changelog
^^^^^^^^^

-The general rules about changelogs in the process documentation, see
-:ref:`Documentation/process/ <submittingpatches>`, apply.
+The general rules about changelogs in the :ref:`Submitting patches guide
+<describe_changes>`, apply.

The tip tree maintainers set value on following these rules, especially on
the request to write changelogs in imperative mood and not impersonating

base-commit: 3e2619c4ebba2cab8414c55b131b7a28f628de3b
--
An old man doll... just what I always wanted! - Clara



2023-03-22 10:19:23

by Thomas Gleixner

[permalink] [raw]
Subject: Re: [PATCH] Documentation: maintainer-tip: Rectify link to "Describe your changes" section of submitting-patches.rst

On Mon, Mar 20 2023 at 19:43, Bagas Sanjaya wrote:
> The general changelog rules for the tip tree refers to "Describe your
> changes" section of submitting patches guide. However, the internal link
> reference targets to non-existent "submittingpatches" label, which
> brings reader to the top of the linked doc.
>
> Correct the target. No changes to submitting-patches.rst since the
> required label is already there.
>
> Fixes: 31c9d7c8297558 ("Documentation/process: Add tip tree handbook")
> Signed-off-by: Bagas Sanjaya <[email protected]>

In cass Jonathan is picking this up:

Reviewed-by: Thomas Gleixner <[email protected]>

> ---
> This patch is based on core/urgent branch of tip tree.

Why? Picking a random branch to base patches on is not really
helpful. It has zero dependencies on that branch.

Thanks,

tglx

2023-03-23 17:27:41

by Jonathan Corbet

[permalink] [raw]
Subject: Re: [PATCH] Documentation: maintainer-tip: Rectify link to "Describe your changes" section of submitting-patches.rst

Thomas Gleixner <[email protected]> writes:

> On Mon, Mar 20 2023 at 19:43, Bagas Sanjaya wrote:
>> The general changelog rules for the tip tree refers to "Describe your
>> changes" section of submitting patches guide. However, the internal link
>> reference targets to non-existent "submittingpatches" label, which
>> brings reader to the top of the linked doc.
>>
>> Correct the target. No changes to submitting-patches.rst since the
>> required label is already there.
>>
>> Fixes: 31c9d7c8297558 ("Documentation/process: Add tip tree handbook")
>> Signed-off-by: Bagas Sanjaya <[email protected]>
>
> In cass Jonathan is picking this up:
>
> Reviewed-by: Thomas Gleixner <[email protected]>

I've applied it, thanks.

jon