-
Notifications
You must be signed in to change notification settings - Fork 11
how-to: source-code: import sending-patches #73
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
Link to docs build: https://canonical-kernel-docs--73.com.readthedocs.build/en/73/ |
fafc8ee to
b8f8284
Compare
docs/how-to/source-code/index.md
Outdated
| Obtain kernel source using Git <obtain-kernel-source-git> | ||
| Enable kernel source package repositories <enable-source-repositories> | ||
| Sending patches <sending-patches> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| Sending patches <sending-patches> | |
| Send patches to the mailing list <send-patches> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @gagath , thanks for this.
I will be using the same scope to review the update.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please rename the file to "send-patches.rst".
| @@ -0,0 +1,55 @@ | |||
| .. _sending-patches: | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| .. _sending-patches: | |
| .. _how-to-send-patches: |
| Sending patches to the mailing-list | ||
| ################################### |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| Sending patches to the mailing-list | |
| ################################### | |
| How to send patches to the mailing-list | |
| ####################################### |
| Sending patches to the mailing-list | ||
| ################################### | ||
|
|
||
| To send patches to the mailing-list, you should use the ``git send-email`` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| To send patches to the mailing-list, you should use the ``git send-email`` | |
| To send kernel patches to the mailing-list, you should use the ``git send-email`` |
To make it clearer that this is for sending kernel patches especially if users are trying to get here from search engines.
| Specifying series | ||
| ================= |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| Specifying series | |
| ================= | |
| Specify series | |
| ============== |
| Since patches must be targeted at some series (unstable, noble, ...) | ||
| you need to also specify this. The ``--subject-prefix`` option can be used | ||
| in this regard: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| Since patches must be targeted at some series (unstable, noble, ...) | |
| you need to also specify this. The ``--subject-prefix`` option can be used | |
| in this regard: | |
| All patches must be targeted at some series (unstable, noble, ...). | |
| Specify the targeted series with the ``--subject-prefix`` option: |
It is common to explain to partners how they can send patches on our mailing-list. While this guide was previously internal, shipping it into the public documentation makes more sense as it can help anyone who wants to contribute to the Ubuntu kernel development. A separate commit in the private documentation will be created to redirect to this new page once accepted and live. Signed-off-by: Agathe Porte <[email protected]>
b8f8284 to
6126782
Compare
|
Thanks for the review, I believe I fixed all of the open items. |
Signed-off-by: AnneCYH <[email protected]>
It is common to explain to partners how they can send patches on our mailing-list. While this guide was previously internal, shipping it into the public documentation makes more sense as it can help anyone who wants to contribute to the Ubuntu kernel development.
A separate commit in the private documentation will be created to redirect to this new page once accepted and live.