Skip to content

docs(how-to): Add upstream patch submission document from old wiki - #119

Open
RossPorter506 wants to merge 1 commit into
canonical:mainfrom
RossPorter506:submit-upstream-patch
Open

RossPorter506 wants to merge 1 commit into
canonical:mainfrom
RossPorter506:submit-upstream-patch

Conversation

@RossPorter506

@RossPorter506 RossPorter506 commented Jul 29, 2026

Copy link
Copy Markdown

Add a document explaining how to go about submitting a patch to the upstream Linux kernel mailing-list.
Also add an explainer and some differentiation between the Linux kernel mailing-list and the Ubuntu mailing-list pages.

  • Added or updated meta description (if this PR includes documentation changes)
  • Reviewed all GitHub Actions / CI results; any unresolved failures are explained in a comment

@RossPorter506
RossPorter506 requested a review from AnneCYH as a code owner July 29, 2026 05:46
@RossPorter506

Copy link
Copy Markdown
Author

Link check is a false positive (it's a weird Matrix link so I'm not surprised), spell check is also a false positive (unless I've forgotten how to spell).
Removed URLs test seems broken.

@bjdean-canonical
bjdean-canonical self-requested a review September 8, 2026 02:35

@bjdean-canonical bjdean-canonical left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @RossPorter506

Link check is a false positive (it's a weird Matrix link so I'm not surprised)

You can add r"https://matrix.to/" to docs/conf.py linkcheck_anchors_ignore_for_url to fix this check.

spell check is also a false positive (unless I've forgotten how to spell).

You can add "maintainer's" to .custom_wordlist.txt where words are ok, or use the spellexception" in-line role if it's not a common word

Removed URLs test seems broken.

This test succeeded on re-running.

@RossPorter506

Copy link
Copy Markdown
Author

Thanks, I added those changes and squashed them into the original commit.

@bjdean-canonical bjdean-canonical left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perfect thanks

@bjdean-canonical

Copy link
Copy Markdown
Collaborator

@RossPorter506 apologies - I merged a conflict fix earlier on (in send-patches.rst) which has now caused another conflict. Could you please rebase and resolve that (should just be choosing the new title and note below:

-How to send patches to the mailing list
-#######################################
+How to send patches to the Ubuntu mailing-list
+##############################################
+
+The Ubuntu mailing-list is the place to submit Ubuntu-specific changes, such as Ubuntu kernel configs. 
+
+.. note::
+    If your patch otherwise applies more generally to the Linux kernel, you should :doc:`submit your patches upstream <./send-patches-upstream>` instead.

Also add an explainer and some differentiation between the Linux kernel mailing-list and the Ubuntu mailing-list pages.

Signed-off-by: Ross Porter <ross.porter@canonical.com>
(cherry picked from commit 58e3da7)

# Conflicts:
#	docs/how-to/source-code/send-patches.rst
@RossPorter506

Copy link
Copy Markdown
Author

Let me know if there's anything else, otherwise it would be good to get this merged before there's more conflicts with main.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants