[LEDE-DEV] [patch master 00/15] shell code improvement for /scripts

Jan-Tarek Butt tarek at ring0.de
Fri Sep 30 13:02:14 PDT 2016


Hi,

I did some bugfixing and code quality improvement for
all shell scrips inside the dir ./scripts.

cheers
Tarek

Jan-Tarek Butt (15):
  Double quote to prevent globbing and word splitting
  code style: remove whitespaces on end of lines
  Add and fix a shebang
  $/${} is unnecessary on arithmetic variables
  Use cd ... || exit in case cd fails
  redirect matters, the 2>&1 has to be last.
  Quote to prevent word splitting
  Use $(..) instead of legacy `..`
  Argument mixes string and array. Use $*
  quote literal {
  Consider using { cmd1; cmd2; } >> file instead of individual redirects
  use * instead of @ to concatenate
  To read lines rather than words, pipe/redirect to a 'while read' loop
  Expressions don't expand in single quotes, use double quotes for that
  read without -r will mangle backslashes

 scripts/arm-magic.sh                      | 12 ++---
 scripts/clean-package.sh                  | 10 ++--
 scripts/combined-ext-image.sh             | 14 +++---
 scripts/combined-image.sh                 |  4 +-
 scripts/config/lxdialog/check-lxdialog.sh | 20 ++++----
 scripts/deptest.sh                        | 24 ++++-----
 scripts/diffconfig.sh                     | 16 +++---
 scripts/ext-toolchain.sh                  | 81 ++++++++++++++++---------------
 scripts/flashing/flash.sh                 | 16 +++---
 scripts/freebsd.sh                        |  1 +
 scripts/gen-dependencies.sh               | 14 +++---
 scripts/get_source_date_epoch.sh          |  2 +-
 scripts/getver.sh                         | 12 ++---
 scripts/ipkg-make-index.sh                | 14 +++---
 scripts/make-ipkg-dir.sh                  |  4 +-
 scripts/mkits.sh                          |  8 +--
 scripts/om-fwupgradecfg-gen.sh            | 26 +++++-----
 scripts/patch-kernel.sh                   | 26 +++++-----
 scripts/patch-specs.sh                    |  2 +-
 scripts/relink-lib.sh                     |  4 +-
 scripts/rstrip.sh                         | 16 +++---
 scripts/strip-kmod.sh                     |  8 +--
 scripts/sysupgrade-tar.sh                 |  2 +-
 scripts/ubinize-image.sh                  | 20 ++++----
 24 files changed, 183 insertions(+), 173 deletions(-)

--
2.10.0




More information about the Lede-dev mailing list