manifest: add allowlist filtering#71
Conversation
trantanen
left a comment
trantanen
left a comment
There was a problem hiding this comment.
@hlord2000 What's your plan on this? I think we could take this into use. Needs a rebase though.
hlord2000
force-pushed
the
manifest/allowlist
branch
from
6e55a2b to
20675b1
Compare
Just rebased, ran tests locally. Working well now and much faster to clone. |
hlord2000
force-pushed
the
manifest/allowlist
branch
3 times, most recently
from
244613b to
bd449ab
Compare
|
Compliance seems to miss some missing Kconfig although compilation works fine. I didn't have time to check what to do with that. Whether we'd just need to ignore some Kconfig check in compliance workflow. |
|
The improvement I got for running This PR: And the folder sizes are: |
|
I verified this with our nightly CI tests and everything works fine. |
Exclude some Kconfig checks because they alert missing Kconfigs from modules that are not in the "allowlist" of the west.yml so these are false alerts. "allowlist" taken into use in PR nrfconnect#71. Signed-off-by: Tommi Rantanen <[email protected]>
Exclude some Kconfig checks because they alert missing Kconfigs from modules that are not in the "allowlist" of the west.yml so these are false alerts. "allowlist" taken into use in PR #71. Signed-off-by: Tommi Rantanen <[email protected]>
Add allowlist filtering for faster initialization. May be able to improve by excluding other irrelevant Zephyr projects like lvgl, zscilib, etc. Signed-off-by: Helmut Lord <[email protected]>
trantanen
force-pushed
the
manifest/allowlist
branch
from
bd449ab to
f2287d2
Compare
|
I tried to figure out how to get rid of the compliance issues with Kconfig references to Kconfigs that are in modules that are not in the I have rebased this PR now and compliance seems to pass. I will merge once the build is through too. |
Exclude some Kconfig checks because they alert missing Kconfigs from modules that are not in the "allowlist" of the west.yml so these are false alerts. "allowlist" taken into use in PR nrfconnect#71. Signed-off-by: Tommi Rantanen <[email protected]>
Exclude some Kconfig checks because they alert missing Kconfigs from modules that are not in the "allowlist" of the west.yml so these are false alerts. "allowlist" taken into use in PR #71. Signed-off-by: Tommi Rantanen <[email protected]>
2 participants
Add allowlist filtering for faster initialization. May be able to improve by excluding other irrelevant Zephyr projects like lvgl, zscilib, etc.