zephyrproject-rtos / west

West, Zephyr's meta-tool
https://docs.zephyrproject.org/latest/guides/west/index.html
Apache License 2.0
215 stars 117 forks source link

[RFC] New `inactive-by-default:` project key #666

Open mbolivar-nordic opened 1 year ago

mbolivar-nordic commented 1 year ago

This issue tracks a potential extension to the manifest schema. It is not a proposal at this point, just a placeholder for discussion.

Proposed syntax:

manifest:
  projects:
    - name: foo
      inactive-by-default: <true|false>

Proposed semantics:

The new behavior that applies if and only if the key is true is:

This applies regardless of where the project is defined in the import hierarchy.

Concerns:

mbolivar-nordic commented 1 year ago

Discussion among @carlescufi @tejlmand @aescolar @mbolivar-nordic :

Fact that this cannot be overridden in an importing manifest is a serious problem. E.g. in NCS we could not write a manifest that imports zephyr and makes bsim active by default.

mbolivar-nordic commented 1 year ago

Potential way forward, not vetted, just an RFC

manifest:
  projects:
    - name: zephyr
      import:
        name-allowlist:
          - bsim
          - ...
        activate:
          - bsim