Skip to content

Commit c102c6f

Browse files
correct shellcheck errors (NOAA-EMC#2294)
1 parent 91b9308 commit c102c6f

File tree

5 files changed

+14
-14
lines changed

5 files changed

+14
-14
lines changed

parm/config/gfs/config.esfc

+1-1
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ if [ $DOIAU_ENKF = "YES" ]; then
1717
fi
1818

1919
# Turn off NST in JEDIATMENS
20-
if [ $DO_JEDIATMENS = "YES" ]; then
20+
if [[ "${DO_JEDIATMENS}" == "YES" ]]; then
2121
export DONST="NO"
2222
fi
2323

parm/config/gfs/config.nsst

+1-1
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ echo "BEGIN: config.nsst"
1111
export NST_MODEL=2
1212

1313
# Set NST_MODEL for JEDIATMVAR or JEDIATMENS
14-
if [[ $DO_JEDIATMVAR = "YES" || $DO_JEDIATMENS = "YES" ]]; then
14+
if [[ "${DO_JEDIATMVAR}" == "YES" || "${DO_JEDIATMENS}" == "YES" ]]; then
1515
export NST_MODEL=1
1616
fi
1717

parm/config/gfs/config.sfcanl

+1-1
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ echo "BEGIN: config.sfcanl"
99
. $EXPDIR/config.resources sfcanl
1010

1111
# Turn off NST in JEDIATMVAR
12-
if [ $DO_JEDIATMVAR = "YES" ]; then
12+
if [[ "${DO_JEDIATMVAR}" == "YES" ]]; then
1313
export DONST="NO"
1414
fi
1515

parm/config/gfs/config.stage_ic

+1-1
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,7 @@ esac
4242
fi # End block for coupled ICs
4343

4444
# Stage ICs for cycled runs
45-
if [[ "$MODE" = "cycled" ]]; then
45+
if [[ "${MODE}" == "cycled" ]]; then
4646
export CPL_ATMIC="C48C48"
4747
export CPL_ATMDAIC="C48C48"
4848
fi

scripts/exglobal_stage_ic.sh

+10-10
Original file line numberDiff line numberDiff line change
@@ -158,9 +158,9 @@ done # for MEMDIR in "${MEMDIR_ARRAY[@]}"; do
158158
else
159159
# Stage deterministic ICs from previous cycle
160160
RUN=${rCDUMP} YMD=${gPDY} HH=${gcyc} generate_com COM_TOP:COM_TOP_TMPL
161-
COM_TOP_BASE=$(dirname $COM_TOP)
161+
COM_TOP_BASE=$(dirname "${COM_TOP}")
162162
[[ ! -d "${COM_TOP_BASE}" ]] && mkdir -p "${COM_TOP_BASE}"
163-
[[ -d "${COM_TOP_BASE}/${gcyc}" ]] && rm -rf "${COM_TOP_BASE}/${gcyc}"
163+
[[ -d "${COM_TOP_BASE}/${gcyc}" ]] && rm -f "${COM_TOP_BASE:-}/${gcyc:-}"
164164
src="${BASE_CPLIC}/${CPL_ATMIC:-}/${RUN}.${gPDY}/${gcyc}"
165165
tgt="${COM_TOP_BASE}/"
166166
${NLN} "${src}" "${tgt}"
@@ -171,9 +171,9 @@ else
171171
# Stage ensemble ICs from previous cycle
172172
if [[ "${DOHYBVAR}" == "YES" ]]; then
173173
RUN=enkf${rCDUMP} YMD=${gPDY} HH=${gcyc} generate_com COM_TOP:COM_TOP_TMPL
174-
COM_TOP_BASE=$(dirname $COM_TOP)
174+
COM_TOP_BASE=$(dirname "${COM_TOP}")
175175
[[ ! -d "${COM_TOP_BASE}" ]] && mkdir -p "${COM_TOP_BASE}"
176-
[[ -d "${COM_TOP_BASE}/${gcyc}" ]] && rm -rf "${COM_TOP_BASE}/${gcyc}"
176+
[[ -d "${COM_TOP_BASE}/${gcyc}" ]] && rm -rf "${COM_TOP_BASE:-}/${gcyc:-}"
177177
src="${BASE_CPLIC}/${CPL_ATMIC:-}/enkf${RUN}.${gPDY}/${gcyc}"
178178
tgt="${COM_TOP_BASE}/"
179179
${NLN} "${src}" "${tgt}"
@@ -192,17 +192,17 @@ else
192192
else
193193
flist="abias abias_air abias_pc radstat"
194194
fi
195-
for ftype in $flist; do
196-
file=${rCDUMP}.t${cyc}z.$ftype
197-
${NCP} "${src}/$file" "${tgt}"
195+
for ftype in ${flist}; do
196+
file=${rCDUMP}.t${cyc}z.${ftype}
197+
${NCP} "${src}"/"${file}" "${tgt}"
198198
rc=$?
199199
((rc != 0)) && error_message "${src}" "${tgt}" "${rc}"
200200
err=$((err + rc))
201201
done
202202
if [[ "${DO_JEDIATMVAR}" == "YES" || "${DO_JEDIATMENS}" == "YES" ]]; then
203-
cd ${tgt}
204-
tar -xvf ${rCDUMP}.t${cyc}z.radbcor
205-
cd ${DATA}
203+
cd "${tgt}" || exit
204+
tar -xvf "${rCDUMP}".t"${cyc}"z.radbcor
205+
cd "${DATA}" || exit
206206
fi
207207

208208
fi

0 commit comments

Comments
 (0)