diff options
| author | Yeliz Taneroğlu <yeliztaneroglu@gmail.com> | 2017-05-19 18:03:04 +0300 |
|---|---|---|
| committer | Chris Sherlock <chris.sherlock79@gmail.com> | 2017-05-20 11:31:11 +0200 |
| commit | 027d661ef7e2490f1182d3e5003ae4da4135e0ca (patch) | |
| tree | f4201c4e05ab0f50117445a3acef073cae65397e /solenv/bin | |
| parent | 020e1bff16a7bb5f401a2547d211807acd7db0fb (diff) | |
tdf#105204 fix shellcheck warnings in solenv/bin/create-ids
Change-Id: Iab3855cdd8fb1ae3719dd7267b1592eea0ef1e22
SC2006: Use $(..) instead of legacy `..`.
SC2035: Use ./*/* so names with dashes won't become options.
SC2086: Double quote to prevent globbing and word splitting.
SC2046: Quote this to prevent word splitting.
Reviewed-on: https://gerrit.libreoffice.org/37832
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Chris Sherlock <chris.sherlock79@gmail.com>
Diffstat (limited to 'solenv/bin')
| -rwxr-xr-x | solenv/bin/create-ids | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/solenv/bin/create-ids b/solenv/bin/create-ids index 0722091b2386..59c532719e38 100755 --- a/solenv/bin/create-ids +++ b/solenv/bin/create-ids @@ -11,8 +11,8 @@ # TODO: should this include workdir headers? if [ "$1" = "--with-outpath" ]; then - dirs="`ls -d */* | sed -e '/\(\/\(cscope\.\|tags\|ID\)\)\|^\(clone\|workdir\|instdir\)\//d'`" + dirs="$(ls -d ./*/* | sed -e '/\(\/\(cscope\.\|tags\|ID\)\)\|^\(clone\|workdir\|instdir\)\//d')" else - dirs="`ls -d */* | sed -e '/\(\/\(cscope\.\|tags\|ID\)\)\|^\(clone\|workdir\|instdir\)\//d'`" + dirs="$(ls -d ./*/* | sed -e '/\(\/\(cscope\.\|tags\|ID\)\)\|^\(clone\|workdir\|instdir\)\//d')" fi -mkid --lang-map=`dirname $0`/id-lang.map --include='C C++ asm perl java make' --statistics $dirs +mkid --lang-map="$(dirname "$0")"/id-lang.map --include='C C++ asm perl java make' --statistics "$dirs" |