diff options
| author | Craig Jennings <c@cjennings.net> | 2026-01-23 18:02:52 -0600 |
|---|---|---|
| committer | Craig Jennings <c@cjennings.net> | 2026-01-23 18:02:52 -0600 |
| commit | c74f656694e084fdf748f7ac86aa4b2361cbe1ed (patch) | |
| tree | c015ff6752ee648925a690f04782eff3e8be6b2a /custom/lib/config.sh | |
| parent | 8e4b16e481db5236d67e7a10e206cccd0e3285cd (diff) | |
| download | archangel-c74f656694e084fdf748f7ac86aa4b2361cbe1ed.tar.gz archangel-c74f656694e084fdf748f7ac86aa4b2361cbe1ed.zip | |
Fix set -e compatibility in lib functions
- Replace [[ ]] && error pattern with if/then/fi
- Pattern causes exit when condition is false with set -e
- Fixed in: common.sh, config.sh, disk.sh, install-archzfs
Discovered during VM testing: the && short-circuit returns exit
code 1 when condition is false, triggering set -e to abort.
Diffstat (limited to 'custom/lib/config.sh')
| -rw-r--r-- | custom/lib/config.sh | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/custom/lib/config.sh b/custom/lib/config.sh index 82f5d77..cec3d8c 100644 --- a/custom/lib/config.sh +++ b/custom/lib/config.sh @@ -122,6 +122,8 @@ validate_config() { ((errors++)) fi - [[ $errors -gt 0 ]] && error "Config validation failed with $errors error(s)" + if [[ $errors -gt 0 ]]; then + error "Config validation failed with $errors error(s)" + fi info "Config validation passed" } |