Skip to content

Conversation

@cfriedt
Copy link
Member

@cfriedt cfriedt commented May 9, 2025

Add documentation for the west patch command.

Closes #89215

Doc Preview

@cfriedt cfriedt force-pushed the doc-for-west-patch branch from ad74139 to 50f0476 Compare May 9, 2025 18:11
Copy link
Contributor

@pdgendt pdgendt left a comment

Choose a reason for hiding this comment

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

Nice work, thanks!

@cfriedt cfriedt force-pushed the doc-for-west-patch branch 9 times, most recently from d288dd1 to 726bb86 Compare May 11, 2025 11:51
@cfriedt
Copy link
Member Author

cfriedt commented May 11, 2025

  • made some stylistic / wording improvements

@cfriedt cfriedt requested review from JarmouniA and pdgendt May 11, 2025 12:07
pdgendt
pdgendt previously approved these changes May 11, 2025
@matt-wood-ct
Copy link
Contributor

Thanks for doing this, this command is a important workflow tool which many people were likely unaware of

Comment on lines 290 to 301
The ``patch`` command allows users to apply patches to Zephyr or Zephyr modules
in a controlled manner that makes automation and tracking easier for external applications that
use the :ref:`T2 star topology <west-t2>`.

There are several sub-commands available:

* ``apply``: apply a patch to the Zephyr module or a module in the workspace.
* ``clean``: remove all patches that have been applied to the Zephyr module or a module in the
workspace.
* ``list``: list all patches that have been applied to the Zephyr module or a module in the
workspace.
* ``gh-fetch``: fetch patches from GitHub.
Copy link
Member Author

Choose a reason for hiding this comment

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

Ooo... I just noticed that this could probably be worded better.

Suggested change
The ``patch`` command allows users to apply patches to Zephyr or Zephyr modules
in a controlled manner that makes automation and tracking easier for external applications that
use the :ref:`T2 star topology <west-t2>`.
There are several sub-commands available:
* ``apply``: apply a patch to the Zephyr module or a module in the workspace.
* ``clean``: remove all patches that have been applied to the Zephyr module or a module in the
workspace.
* ``list``: list all patches that have been applied to the Zephyr module or a module in the
workspace.
* ``gh-fetch``: fetch patches from GitHub.
The ``patch`` command allows users to apply patches to Zephyr or Zephyr modules
in a controlled manner that makes automation and tracking easier for external applications that
use the :ref:`T2 star topology <west-t2>`.
There are several sub-commands available to manage patches for Zephyr or other modules in the workspace.
* ``apply``: apply patches listed in ``patches.yml``
* ``clean``: remove all patches that have been applied, and reset to the manifest checkout state
* ``list``: list all patches in ``patches.yml``
* ``gh-fetch``: fetch patches from a GitHub pull request

Copy link
Member Author

Choose a reason for hiding this comment

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

Fixed!

@cfriedt cfriedt force-pushed the doc-for-west-patch branch from 05d8580 to 26bc584 Compare May 14, 2025 10:41
JarmouniA
JarmouniA previously approved these changes May 14, 2025
pdgendt
pdgendt previously approved these changes May 14, 2025
@cfriedt cfriedt dismissed stale reviews from pdgendt and JarmouniA via 81a9bd7 May 14, 2025 15:49
@cfriedt cfriedt force-pushed the doc-for-west-patch branch from 26bc584 to 81a9bd7 Compare May 14, 2025 15:49
@cfriedt cfriedt requested review from JarmouniA, kartben and pdgendt May 14, 2025 15:50
@cfriedt cfriedt force-pushed the doc-for-west-patch branch from 81a9bd7 to 7a2ccf8 Compare May 14, 2025 15:51
kartben
kartben previously approved these changes May 14, 2025
Add documentation for the `west patch` command.

Signed-off-by: Chris Friedt <cfriedt@tenstorrent.com>
@cfriedt
Copy link
Member Author

cfriedt commented May 14, 2025

  • recalculated sha256 for zephyr patch rather than mcuboot patch (which was deleted in the example)

Sorry for the last minute change @kartben 😅

@cfriedt cfriedt requested a review from kartben May 14, 2025 16:12
@sonarqubecloud
Copy link

@kartben kartben merged commit 1fc7ccb into zephyrproject-rtos:main May 14, 2025
21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

West patch lacks documentation

6 participants