diff --git a/lib/core/wrappers.sh b/lib/core/wrappers.sh index 1fd8696..45206bb 100644 --- a/lib/core/wrappers.sh +++ b/lib/core/wrappers.sh @@ -1,9 +1,9 @@ function create_wrappers() { - mkdir -p ${JUNEST_HOME}/usr/bin_wrappers + mkdir -p "${JUNEST_HOME}/usr/bin_wrappers" - cd ${JUNEST_HOME}/usr/bin + cd "${JUNEST_HOME}/usr/bin" || return 1 for file in * do [[ -x $file ]] || continue @@ -14,21 +14,21 @@ function create_wrappers() { # Arguments inside a variable (i.e. `JUNEST_ARGS`) separated by quotes # are not recognized normally unless using `eval`. More info here: # https://github.com/fsquillace/junest/issues/262 - cat < ${JUNEST_HOME}/usr/bin_wrappers/${file} + cat < "${JUNEST_HOME}/usr/bin_wrappers/${file}" #!/usr/bin/env bash JUNEST_ARGS=\${JUNEST_ARGS:-ns --fakeroot} -eval junest \${JUNEST_ARGS} -- ${file} "\$@" +eval junest "\${JUNEST_ARGS}" -- ${file} "\$(printf "%q" "\$@")" EOF - chmod +x ${JUNEST_HOME}/usr/bin_wrappers/${file} + chmod +x "${JUNEST_HOME}/usr/bin_wrappers/${file}" done # Remove wrappers no longer needed - cd ${JUNEST_HOME}/usr/bin_wrappers + cd "${JUNEST_HOME}/usr/bin_wrappers" || return 1 for file in * do - [[ -e ${JUNEST_HOME}/usr/bin/$file ]] || rm -f $file + [[ -e ${JUNEST_HOME}/usr/bin/$file ]] || rm -f "$file" done }