diff options
author | Paweł Krupa <pawel@krupa.net.pl> | 2018-09-29 01:09:34 +0200 |
---|---|---|
committer | Costa Tsaousis <costa@tsaousis.gr> | 2018-09-29 02:09:34 +0300 |
commit | a1b6a7f5328974aaddd7f631135c58e1d620c820 (patch) | |
tree | f834274490755daeddc0b445e9c75d18b71b88e1 /charts.d/apache.chart.sh | |
parent | 986e32a03b2f01ac09b23d7f1be28a2799d7fd3b (diff) |
[cleanup crusade] Shellcheck (#4261)
* lint libreswan module
* disable linting obsolete shell modules
* shellcheck more shell modules
* fix review; lint helper scripts
* more shellcheck
* fix SC2235 in apache.chart.sh
* shellcheck last portion of charts.d scripts
* final shellcheck; make travis blocking
* fix cppcheck
* fix indents; fix expansion
* fix local var assignments
Diffstat (limited to 'charts.d/apache.chart.sh')
-rw-r--r-- | charts.d/apache.chart.sh | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/charts.d/apache.chart.sh b/charts.d/apache.chart.sh index 779a4026a0..84d4e1bf07 100644 --- a/charts.d/apache.chart.sh +++ b/charts.d/apache.chart.sh @@ -103,7 +103,10 @@ apache_get() { ret=$? IFS="${oIFS}" - ([ $ret -ne 0 ] || [ "${#apache_response[@]}" -eq 0 ]) && return 1 + if [ $ret -ne 0 ] || [ "${#apache_response[@]}" -eq 0 ] + then + return 1 + fi # the last line on the apache output is "Scoreboard" # we use this label to detect that the output has a new word count |