diff options
| author | Ed Santiago <santiago@redhat.com> | 2022-09-15 16:18:46 -0600 |
|---|---|---|
| committer | Ed Santiago <santiago@redhat.com> | 2022-09-15 20:10:34 -0600 |
| commit | cfbc4aaeb50b5fb12ec9363328bb08fc459a1067 (patch) | |
| tree | e396257e102a885561638fe6702d6c0a6e917743 /hack/man-page-checker | |
| parent | 750726e62c13941a739ef70040a1ec0d745cdb43 (diff) | |
| download | podman-cfbc4aaeb50b5fb12ec9363328bb08fc459a1067.tar.gz podman-cfbc4aaeb50b5fb12ec9363328bb08fc459a1067.tar.bz2 podman-cfbc4aaeb50b5fb12ec9363328bb08fc459a1067.zip | |
Cleanup: fix problems reported by shell lint
Followup to #15616, which is not usable as it is (way, way, way
too much noise) but actually found a few real nits that should
be fixed.
Signed-off-by: Ed Santiago <santiago@redhat.com>
Diffstat (limited to 'hack/man-page-checker')
| -rwxr-xr-x | hack/man-page-checker | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/hack/man-page-checker b/hack/man-page-checker index 8ee0aaf6d..83e0b8b1d 100755 --- a/hack/man-page-checker +++ b/hack/man-page-checker @@ -87,7 +87,7 @@ function compare_usage() { # strip off command name from both from_man=$(sed -e "s/\*\*$cmd\*\*[[:space:]]*//" <<<"$from_man") - from_help=$(sed -e "s/^[[:space:]]*$cmd[[:space:]]*//" <<<"$from_help") + from_help=$(sed -e "s/^[[:space:]]*${cmd}[[:space:]]*//" <<<"$from_help") # man page lists 'foo [*options*]', help msg shows 'foo [flags]'. # Make sure if one has it, the other does too. @@ -153,7 +153,7 @@ for md in *.1.md;do # special case: the command is "auto-update", with a hyphen md_nodash='podman auto-update' fi - if [ "$cmd" != "$md_nodash" -a "$cmd" != "podman-remote" ]; then + if [[ "$cmd" != "$md_nodash" ]] && [[ "$cmd" != "podman-remote" ]]; then echo printf "Inconsistent program name in SYNOPSIS in %s:\n" $md printf " SYNOPSIS = %s (expected: '%s')\n" "$cmd" "$md_nodash" |