-
Notifications
You must be signed in to change notification settings - Fork 147
support 'west config --list-paths' #870
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
support 'west config --list-paths' #870
Conversation
45112ce to
8847ef7
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #870 +/- ##
==========================================
+ Coverage 84.34% 84.41% +0.07%
==========================================
Files 11 11
Lines 3366 3382 +16
==========================================
+ Hits 2839 2855 +16
Misses 527 527
|
|
The command does not appear to work correctly in my Zephyr workspace (my local config file is not printed): $ cat .west/config
[manifest]
path = zephyr
file = west.yml
group-filter = +optional
[zephyr]
base = zephyr
$ west config --list-paths
/home/pdgendt/.westconfig
$ west config --list-paths --local
$ west config manifest.path
zephyr |
8847ef7 to
5463aeb
Compare
pdgendt
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, small nit
5463aeb to
5b768ca
Compare
5b768ca to
9526d2d
Compare
The command prints a list of all configuration files currently considered and existing. They are listed in the order as they are loaded.
9526d2d to
1682248
Compare
Extracted from #867
Why?
It is helpful for users to see which configuration files are currently being used by
west, and in which order they are applied.This makes it easier to debug configuration issues and understand where specific settings originate from.
Proposal
Introduce a new command-line option
--list-paths, which prints all configuration files that are currently considered and exist.The files are listed in the order they are loaded, as later files override values from earlier ones.
This option can be combined with
--local,--global, or--systemto list only the path for a specific configuration level.