diff options
author | Tom Ryder <tom@sanctum.geek.nz> | 2017-10-31 22:32:49 +1300 |
---|---|---|
committer | Tom Ryder <tom@sanctum.geek.nz> | 2017-10-31 22:32:49 +1300 |
commit | e7213cb7cbd3149377cf6c08dc9c3813c99743f8 (patch) | |
tree | d754980f8c170cd260ea0984cadd9e295851a0b3 /check/sh.sh | |
parent | Use shell name not command in check/lint messages (diff) | |
download | dotfiles-e7213cb7cbd3149377cf6c08dc9c3813c99743f8.tar.gz dotfiles-e7213cb7cbd3149377cf6c08dc9c3813c99743f8.zip |
Tidy and correct linting for all three shells
Reduce each one to target only the dotfiles specifically for that shell,
as opposed to previously where for example the `check-sh` target was
checking shell shims in for `mpd` and `plenv`.
I'm still not completely sure that's the right approach, but it's at
least less conceptually muddy than what we had before.
Notably, the check and lint for Korn shell includes a single POSIX shell
script file in its `shrc.d` subdirectory, so that check is executed
separately.
Diffstat (limited to 'check/sh.sh')
-rw-r--r-- | check/sh.sh | 19 |
1 files changed, 9 insertions, 10 deletions
diff --git a/check/sh.sh b/check/sh.sh index c5a86955..92910c11 100644 --- a/check/sh.sh +++ b/check/sh.sh @@ -1,11 +1,10 @@ -for sh in \ - sh/* sh/profile.d/* sh/shrc.d/* \ - keychain/profile.d/* keychain/shrc.d/* \ - ksh/shrc.d/* \ - mpd/profile.d/* \ - plenv/profile.d/* plenv/shrc.d/* \ -; do - [ -f "$sh" ] || continue - sh -n "$sh" || exit +set \ + sh/profile \ + sh/profile.d/*.sh \ + sh/shinit \ + sh/shrc \ + sh/shrc.d/*.sh +for sh ; do + sh -n -- "$sh" || exit done -printf 'All sh(1) scripts parsed successfully.\n' +printf 'POSIX shell dotfiles parsed successfully.\n' |