diff options
author | Vito Caputo <vcaputo@pengaru.com> | 2023-08-30 23:16:39 -0700 |
---|---|---|
committer | Vito Caputo <vcaputo@pengaru.com> | 2023-08-30 23:16:39 -0700 |
commit | 8353fc7c4d571af72e4a17a3b453c4f1dac652db (patch) | |
tree | 1deac4b50762bdd4fc6e82f21abab1b0bf7dac63 /src/modules/sparkler/simple.c | |
parent | f5e4e0c1816a83c06ff4d57ee1c5272b055a54f4 (diff) |
setup: return to single setup_func() call site
Now that all the module setup_funcs are returning a res_setting
w/-EINVAL in res_setup baking, it should be fine for
setup_interactively() to resume using the single setup_func()
loop passing res_setup, and always using res_setting on -EINVAL.
This is especially desirable now that :-prefix settings are
accepted as overrides. You can now get arbitrary values down to
the res_setup baking phase, previously when you got something
wrong there you'd get an ambiguous error without a setting path.
With this commit, you should get a much more useful error
including a setting path.
This partially undoes 5191d68, where res_setup-baking was split
off from the core loop, to occur after the res_failed_desc on
EINVAL storage.
Diffstat (limited to 'src/modules/sparkler/simple.c')
0 files changed, 0 insertions, 0 deletions