Commit Graph
2 Commits
Author SHA1 Message Date
Krish De Souza 416bea727e Shellcheck fixes for configure.sh and steamrt-bootstrap.sh
In steamrt-bootstrap,sh
  + Used quotes to prevent word splitting SC2046
  + Used $() notation rather than legacy backtick SC2006
  + which is non-standard. 'command -v' is builtin SC2230

In configure.sh
  + Assigned to local variable separately to avoid masking return values SC2155
  + Used to quote to prevent glob matching SC2053
  + Used -z command rather than ! -n SC2236
  + Fixed SC2129 which would have a minor performance gain of avoiding constantly opening and closing the makefile.
2018-11-16 12:54:56 -06:00
Krish De Souza b0e4ddf8e3 Minor linting fixes to build_proton.sh
--Added double quotes around various variables to avoid globbing and word splitting.
  --Removed use of legacy basetick and replaced with $()
  --Avoid use of -a and -o and replaced with || and && which are more preferred.

Suggestions and fixes were recommended by shellcheck.
2018-08-27 08:49:57 -05:00