Darko Poljak
596f3cf227
Disable SC2119,SC2120 where appropriate.
2018-10-05 12:31:05 +02:00
Darko Poljak
9b567ed8f7
Fix SC2230.
2018-10-05 12:27:43 +02:00
Darko Poljak
8cea8e67aa
Disable SC2031 where appropriate.
2018-10-05 12:25:40 +02:00
Darko Poljak
b120a813a2
Disable SC2030 where appropriate.
2018-10-05 12:24:03 +02:00
Darko Poljak
7b17ad4733
Fix SC2143.
2018-10-05 12:21:59 +02:00
Darko Poljak
e46420e7f5
Fix SC1090.
2018-10-05 12:19:16 +02:00
Darko Poljak
2f7b8e6db9
Fix SC2153.
2018-10-05 12:16:27 +02:00
Darko Poljak
73d0a5bbbe
Fix SC2235.
2018-10-05 12:14:51 +02:00
Darko Poljak
73acb35a8a
Fix SC2196.
2018-10-05 12:04:45 +02:00
Darko Poljak
28889ef38a
Disable SC2028 where intended.
2018-10-05 12:02:10 +02:00
Darko Poljak
ed542cc7cb
Fix SC2046.
2018-10-05 11:57:31 +02:00
Darko Poljak
b25aff17c6
Disable SC2039 where appropriate.
2018-10-05 11:49:28 +02:00
Darko Poljak
a601fa91d9
Fix SC2116.
2018-10-05 11:34:10 +02:00
Darko Poljak
3b6d759660
Disable SC2016 check.
2018-10-05 10:32:38 +02:00
Darko Poljak
0cb127822f
Merge pull request #706 from darko-poljak/SC2164
...
Fix SC2164
2018-10-05 10:28:04 +02:00
Darko Poljak
49bb041330
Merge pull request #696 from thriqon/shellcheck-party-SC2086
...
Fix SC2086 on all scripts
2018-10-05 10:26:53 +02:00
Darko Poljak
c01cf38660
Fix SC2164
2018-10-04 23:10:42 +02:00
Darko Poljak
68b6477fe3
Merge pull request #705 from thriqon/shellcheck-party-various
...
Fix various shellcheck issues (SC2215, SC2059, SC2155, SC2091)
2018-10-04 22:39:05 +02:00
Darko Poljak
f0fb5215e8
Merge pull request #700 from thriqon/shellcheck-party-SC2034
...
Remove unused variables (SC2034)
2018-10-04 22:35:08 +02:00
Jonas Weber
34b571b7d9
Fix SC2091 (execute result of condition)
...
For some reason, the echo|grep pattern was enclosed four times in a $()
operation. Nothing happened, since grep is always invoked with -q
(quiet, nothing printed to stdout).
2018-10-04 22:23:48 +02:00
Jonas Weber
bcd7fbbcc4
Fix SC2155 (directly export computed variables)
2018-10-04 22:23:48 +02:00
Jonas Weber
a0db375f91
Disable SC2059 (printf injection) for staged_file
...
We're explicitly allowing for 'printf injection' here, so it's ok to
just disable the corresponding shellcheck check.
2018-10-04 22:23:38 +02:00
Jonas Weber
a6fdf4255a
Fix SC2069 (wrong stdout/stderr redirect order)
...
In the original order, stderr was connected to the old stdout
(terminal). This was _probably_ not intended. The new order fixes this
by first connecting stdout to /dev/null and then attaching stderr to
that as well.
2018-10-04 22:04:24 +02:00
Jonas Weber
e2593c0dcb
Fix SC2215 (missing brackets on if)
...
This might have never worked as intended, but how would you know?
2018-10-04 22:03:52 +02:00
Jonas Weber
fff656d1ac
Unquote $recursive to support empty-case
...
This is safe to do, as the value of $recursive is not dependent on
user-supplied code.
2018-10-04 19:45:26 +02:00
Jonas Weber
53728a7ff0
Change backslash-dquote to single quote
...
Slightly breaking change, but improves security and readability.
2018-10-04 19:45:26 +02:00
Jonas Weber
ae7ccc5995
Fix SC2086 on all scripts
...
Relevant documentation: https://github.com/koalaman/shellcheck/wiki/SC2086
relates to: #540
This commit addresses 241 separate cases of missing quotes around
variables.
2018-10-04 19:45:22 +02:00
Jonas Weber
23debd5b6f
Use cur_ip in jail for freebsd
2018-10-04 19:22:28 +02:00
Jonas Weber
352679386d
Drop unused options parameter in __install_reboot
2018-10-04 19:22:28 +02:00
Jonas Weber
b3f4d3849a
Fix __install_umount target parameter
2018-10-04 19:22:28 +02:00
Jonas Weber
6b0f8fba22
Remove unused variables (almost, found bugs?)
...
Solves SC2034
references #540 .
2018-10-04 19:22:28 +02:00
085b2a2b10
remove set -e on explorers (they should be able to fail gracefully)
2018-10-04 17:38:39 +02:00
a46da35bbc
fix SC2148 (missing shebang) and make executable
2018-10-04 17:38:38 +02:00
d950ddada3
fix SC2045 (use globs instead of ls)
2018-10-04 16:01:45 +02:00
134c84607f
fix another SC2166 (test [ .. -a.. ] replaced by [ .. ] && ])
2018-10-04 12:28:59 +02:00
Darko Poljak
e1b62ec560
Merge pull request #699 from darko-poljak/SC2039
...
Fix SC2039
2018-10-04 09:08:23 +02:00
Darko Poljak
8ef2773c0d
Fix SC2039
2018-10-03 14:32:39 +02:00
Darko Poljak
44acfcdd12
Fix SC2002
2018-10-03 14:12:07 +02:00
Darko Poljak
e69be05f70
Merge pull request #694 from darko-poljak/SC2005
...
Fix SC2005
2018-10-02 21:49:45 +02:00
Darko Poljak
d29534861b
Merge pull request #695 from darko-poljak/SC2004
...
Fix SC2004
2018-10-02 21:49:01 +02:00
Darko Poljak
0928708d19
Fix SC2004
2018-10-02 20:34:54 +02:00
Darko Poljak
5ed95ce93b
Fix SC2005
2018-10-02 20:22:44 +02:00
Jonas Weber
6326eae1bb
Fix SC2166 on all manfest/gencode/explorers
2018-10-01 22:13:24 +02:00
Darko Poljak
82077aff25
Add timestamping log option to config file.
2018-09-27 18:55:33 +02:00
Darko Poljak
1e34fc20be
Merge pull request #691 from darko-poljak/log-timestamping
...
Add timestamp to log messages.
2018-09-26 20:35:14 +02:00
Darko Poljak
4b9994f134
Merge pull request #690 from myeisha/batch-scp
...
transfer all files of a directory at once
2018-09-23 22:22:57 +02:00
Darko Poljak
0c57b3083e
Fix getting loggers.
2018-09-23 21:38:03 +02:00
Darko Poljak
6acf6f64dc
Add timestamp to log messages.
2018-09-23 20:36:57 +02:00
dhivael
c021cd15cd
transfer all files of a directory at once
...
instead of calling the copy command once per file in a directory (eg a
type explorer dir), call the copy command only once with all files of
the directory.
batch copy saves 20% of dry-run time on my test manifest in sequential
mode, and 15% in parallel mode.
2018-09-22 17:35:32 +02:00
dhivael
f918acd725
don't duplicate Remote::mkdir in explorer transfer
...
Remote::transfer contains a call to mkdir(destination) if the source is
a directory. since the destination must also be a directory we can omit
extra mkdir calls in Explorer.
this saves about 10% on my manifests in sequential mode, and about 6% in
parallel mode.
2018-09-22 13:16:22 +02:00
