[LEDE-DEV] [PATCH v3 00/13] shell code improvement for /scripts

John Crispin john at phrozen.org
Thu Oct 27 03:32:44 PDT 2016


Hi,

right now i see the chance of this series being merged as very slim.
alot of it is very useful. however every patch i have a detailed look at
has some minor issues, whether it be the actual formatting, description
text not correlating, unrelated changes the purpose of which is not
obvious, .... additionally i find it hard to believe that alot his stuff
has been properly tested. unless someone steps up and starts reviewing
these patches on the next 1-2 days i will NAK them and ask you to start
resending it in smaller chunks.

	John

On 06/10/2016 10:50, Jan-Tarek Butt wrote:
> Hi,
> 
> hopefully now all is clear. SoB should now set by all patches.
> Also I change the commit messages a bit.
> 
> If something unclear please tell me. For me that is a good
> praxis to learn it properly :)
> 
> cheers
> Tarek
> 
> Jan-Tarek Butt (13):
>   scripts: Double quote to prevent globbing and word splitting
>   scripts: fix formatting
>   scripts: add and fix shebang
>   scripts: $/${} is unnecessary on arithmetic variables
>   scripts: Use cd ... || exit in case cd fails
>   scripts: redirect matters, the 2>&1 has to be last
>   scripts: Quote to prevent word splitting
>   scripts: Use $(..) instead of legacy `..`
>   scripts: Argument mixes string and array. Use $*
>   scripts: quote literal {
>   scripts: using { cmd1; cmd2; } >> file instead of individual redirects
>   scripts: use * instead of @ to concatenate
>   scripts: read without -r will mangle backslashes
> 
>  scripts/arm-magic.sh                      |  11 ++-
>  scripts/clean-package.sh                  |   7 +-
>  scripts/combined-ext-image.sh             |  14 +--
>  scripts/combined-image.sh                 |   4 +-
>  scripts/config/lxdialog/check-lxdialog.sh |  20 ++--
>  scripts/deptest.sh                        |  20 ++--
>  scripts/diffconfig.sh                     |  18 ++--
>  scripts/ext-toolchain.sh                  | 153 +++++++++++++++++-------------
>  scripts/flashing/flash.sh                 |  16 ++--
>  scripts/freebsd.sh                        |   1 +
>  scripts/gen-dependencies.sh               |  14 +--
>  scripts/get_source_date_epoch.sh          |   4 +-
>  scripts/getver.sh                         |  14 +--
>  scripts/ipkg-make-index.sh                |  14 +--
>  scripts/make-ipkg-dir.sh                  |   6 +-
>  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, 229 insertions(+), 199 deletions(-)
> 



More information about the Lede-dev mailing list