Skip to content

Commit

Permalink
manifest: move optional modules to a submanifest and make them optional
Browse files Browse the repository at this point in the history
Move optional modules to a submanifest and make them optional by
default.
This is just a POC, might require some more thought. The idea is to look
at the optional manifest as an area for modules that work with Zephyr,
but not needed directly by zephyr. This could be documented somewhere
for discovery purposes allowing users to enable such modules in their
downstream if desired.

See #54276

Signed-off-by: Anas Nashif <anas.nashif@intel.com>
  • Loading branch information
nashif committed Aug 16, 2023
1 parent 038351a commit 9886a58
Show file tree
Hide file tree
Showing 2 changed files with 62 additions and 33 deletions.
61 changes: 61 additions & 0 deletions submanifests/optional.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,61 @@

Check warning on line 1 in submanifests/optional.yaml

View workflow job for this annotation

GitHub Actions / Run compliance checks on patch series (PR)

YAMLLint (empty-lines)

submanifests/optional.yaml:1 too many blank lines (1 > 0)
manifest:
remotes:
- name: upstream
url-base: https://github.com/zephyrproject-rtos
projects:
- name: thrift
path: optional/modules/lib/thrift
revision: 10023645a0e6cb7ce23fcd7fd3dbac9f18df6234
remote: upstream
groups:
- optional
- name: tflite-micro
revision: 9156d050927012da87079064db59d07f03b8baf6
path: optional/modules/lib/tflite-micro
repo-path: tflite-micro
remote: upstream
groups:
- optional
- name: zscilib
path: modules/lib/zscilib
revision: 34c3432e81085bb717e4871d21ca419ae0058ec5
remote: upstream
groups:
- optional
- name: nanopb
revision: 42fa8b211e946b90b9d968523fce7b1cfe27617e
path: modules/lib/nanopb
remote: upstream
groups:
- optional
- name: chre
revision: b7955c27e50485b7dafdc3888d7d6afdc2ac6d96
path: modules/lib/chre
remote: upstream
groups:
- optional
- name: lz4
revision: 8e303c264fc21c2116dc612658003a22e933124d
path: modules/lib/lz4
remote: upstream
groups:
- optional
- name: sof
revision: ee40f61b5725b6615f1abea4a78cc4d90ce144b8
path: modules/audio/sof
remote: upstream
groups:
- optional
- name: tf-m-tests
revision: a878426da78fbd1486dfc29d6c6b82be4ee79e72
path: modules/tee/tf-m/tf-m-tests
remote: upstream
groups:
- optional
- name: psa-arch-tests
revision: 6a17330e0dfb5f319730f974d5b05f7b7f04757b
path: modules/tee/tf-m/psa-arch-tests
remote: upstream
groups:
- optional
34 changes: 1 addition & 33 deletions west.yml
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ manifest:
- name: babblesim
url-base: https://github.com/BabbleSim

group-filter: [-babblesim]
group-filter: [-babblesim,-optional]

Check warning on line 27 in west.yml

View workflow job for this annotation

GitHub Actions / Run compliance checks on patch series (PR)

YAMLLint (commas)

west.yml:27 too few spaces after comma

#
# Please add items below based on alphabetical order
Expand Down Expand Up @@ -118,9 +118,6 @@ manifest:
- name: canopennode
revision: dec12fa3f0d790cafa8414a4c2930ea71ab72ffd
path: modules/lib/canopennode
- name: chre
revision: b7955c27e50485b7dafdc3888d7d6afdc2ac6d96
path: modules/lib/chre
- name: cmsis
revision: 74981bf893e8b10931464b9945e2143d99a3f0a3
path: modules/hal/cmsis
Expand Down Expand Up @@ -272,9 +269,6 @@ manifest:
- name: lvgl
revision: 8a6a2d1d29d17d1e4bdc94c243c146a39d635fdd
path: modules/lib/gui/lvgl
- name: lz4
revision: 8e303c264fc21c2116dc612658003a22e933124d
path: modules/lib/lz4
- name: mbedtls
revision: c38dc78d9a8dcbe43b898cc1171ab33ba3e6fc26
path: modules/crypto/mbedtls
Expand All @@ -288,9 +282,6 @@ manifest:
groups:
- debug
revision: a819419603a2dfcb47f7f39092e1bc112e45d1ef
- name: nanopb
revision: 42fa8b211e946b90b9d968523fce7b1cfe27617e
path: modules/lib/nanopb
- name: net-tools
revision: e0828aa9629b533644dc96ff6d1295c939bd713c
path: tools/net-tools
Expand All @@ -313,13 +304,6 @@ manifest:
path: modules/debug/segger
groups:
- debug
- name: sof
revision: ee40f61b5725b6615f1abea4a78cc4d90ce144b8
path: modules/audio/sof
- name: tflite-micro
revision: 9156d050927012da87079064db59d07f03b8baf6
path: modules/lib/tflite-micro
repo-path: tflite-micro
- name: tinycrypt
revision: 3e9a49d2672ec01435ffbf0d788db6d95ef28de0
path: modules/crypto/tinycrypt
Expand All @@ -340,28 +324,12 @@ manifest:
path: modules/tee/tf-a/trusted-firmware-a
groups:
- tee
- name: tf-m-tests
revision: a878426da78fbd1486dfc29d6c6b82be4ee79e72
path: modules/tee/tf-m/tf-m-tests
groups:
- tee
- name: psa-arch-tests
revision: 6a17330e0dfb5f319730f974d5b05f7b7f04757b
path: modules/tee/tf-m/psa-arch-tests
groups:
- tee
- name: uoscore-uedhoc
revision: 5fe2cb613bd7e4590bd1b00c2adf181ac0229379
path: modules/lib/uoscore-uedhoc
- name: zcbor
revision: 67fd8bb88d3136738661fa8bb5f9989103f4599e
path: modules/lib/zcbor
- name: zscilib
path: modules/lib/zscilib
revision: 34c3432e81085bb717e4871d21ca419ae0058ec5
- name: thrift
path: modules/lib/thrift
revision: 10023645a0e6cb7ce23fcd7fd3dbac9f18df6234

self:
path: zephyr
Expand Down

0 comments on commit 9886a58

Please sign in to comment.