Skip to content

Commit

Permalink
scripts: drop downstream 'west patch' command
Browse files Browse the repository at this point in the history
Zephyr upstream has its own implementation of 'west patch' subcommand,
which will basically allow the same kind of functionality.

Drop downstream 'west patch', since it is not used currently by the SDK.
That way there won't be any conflict after bumping Zephyr version.

Signed-off-by: Marcin Niestroj <[email protected]>
  • Loading branch information
mniestroj committed Jan 24, 2025
1 parent 9c89fd7 commit 76f4104
Show file tree
Hide file tree
Showing 12 changed files with 0 additions and 97 deletions.
3 changes: 0 additions & 3 deletions .ci-west-ncs-twister.yml
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,3 @@ manifest:
name-allowlist:
- nrf
- zephyr
userdata:
patches_dirs:
- patches/west-ncs
3 changes: 0 additions & 3 deletions .ci-west-ncs.yml
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,3 @@ manifest:
- trusted-firmware-m
- zcbor
- zephyr
userdata:
patches_dirs:
- patches/west-ncs
3 changes: 0 additions & 3 deletions .ci-west-zephyr-twister.yml
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,3 @@ manifest:
file: west-zephyr.yml
name-allowlist:
- zephyr
userdata:
patches_dirs:
- patches/west-zephyr
3 changes: 0 additions & 3 deletions .ci-west-zephyr.yml
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,3 @@ manifest:
- tinycrypt
- zcbor
- zephyr
userdata:
patches_dirs:
- patches/west-zephyr
1 change: 0 additions & 1 deletion .github/workflows/hil-integration-nsim.yml
Original file line number Diff line number Diff line change
Expand Up @@ -93,7 +93,6 @@ jobs:
west update -o=--depth=1 -n
git config --global user.email [email protected]
git config --global user.name "Git User"
west patch
pip install \
-r zephyr/scripts/requirements-base.txt \
Expand Down
1 change: 0 additions & 1 deletion .github/workflows/hil-sample-nsim.yml
Original file line number Diff line number Diff line change
Expand Up @@ -64,7 +64,6 @@ jobs:
west update -o=--depth=1 -n
git config --global user.email [email protected]
git config --global user.name "Git User"
west patch
pip3 install \
-r zephyr/scripts/requirements-base.txt \
Expand Down
2 changes: 0 additions & 2 deletions .github/workflows/hil-sample-zephyr.yml
Original file line number Diff line number Diff line change
Expand Up @@ -91,7 +91,6 @@ jobs:
if west config manifest.file | grep -q '.ci-west-zephyr.yml'; then
git config --global user.email [email protected]
git config --global user.name "Git User"
west patch
fi
pip3 install \
Expand Down Expand Up @@ -173,7 +172,6 @@ jobs:
west update -o=--depth=1 -n
git config --global user.email [email protected]
git config --global user.name "Git User"
west patch
pip3 install \
-r zephyr/scripts/requirements-base.txt \
Expand Down
1 change: 0 additions & 1 deletion .pre-commit-config.yaml
Original file line number Diff line number Diff line change
@@ -1,4 +1,3 @@
exclude: '^patches/'
repos:
- repo: https://github.com/pre-commit/pre-commit-hooks
rev: v4.6.0
Expand Down
7 changes: 0 additions & 7 deletions scripts/west-commands.yml

This file was deleted.

67 changes: 0 additions & 67 deletions scripts/west_commands/patch.py

This file was deleted.

3 changes: 0 additions & 3 deletions west-ncs.yml
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,3 @@ manifest:
self:
path: modules/lib/golioth-firmware-sdk
west-commands: scripts/west-commands.yml
userdata:
patches_dirs:
- patches/west-ncs
3 changes: 0 additions & 3 deletions west-zephyr.yml
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,3 @@ manifest:
self:
path: modules/lib/golioth-firmware-sdk
west-commands: scripts/west-commands.yml
userdata:
patches_dirs:
- patches/west-zephyr

0 comments on commit 76f4104

Please sign in to comment.