Package updated freebsd, shellcheck SC2086 fixed #360

Open
michellest wants to merge 1 commits from CamilionEU/cdist:bugfix-openldap into master
1 changed files with 3 additions and 3 deletions

View File

@ -19,7 +19,7 @@ CONF_OWNER="root"
CONF_GROUP="root" CONF_GROUP="root"
case "${os}" in case "${os}" in
freebsd) freebsd)
PKGS="openldap-server" PKGS="openldap26-server"
Review

Question: does this not break older FreeBSD versions? And if yes, which ones?

Question: does this not break older FreeBSD versions? And if yes, which ones?
ETC="/usr/local/etc" ETC="/usr/local/etc"
SLAPD_DIR="/usr/local/etc/openldap" SLAPD_DIR="/usr/local/etc/openldap"
SLAPD_DATA_DIR="/var/db/openldap-data" SLAPD_DATA_DIR="/var/db/openldap-data"
@ -197,12 +197,12 @@ if [ -z "${_skip_letsencrypt_cert}" ]; then
staging="" staging=""
fi fi
__directory "${SLAPD_DIR}/sasl2"
# shellcheck disable=SC2086 # shellcheck disable=SC2086
__directory ${SLAPD_DIR}/sasl2
require="__directory/${SLAPD_DIR}/sasl2" __letsencrypt_cert "${name}" \ require="__directory/${SLAPD_DIR}/sasl2" __letsencrypt_cert "${name}" \
--admin-email "${admin_email}" \ --admin-email "${admin_email}" \
--renew-hook "cp ${ETC}/letsencrypt/live/${name}/*.pem ${SLAPD_DIR}/sasl2 && chown -R ${CONF_OWNER}:${CONF_GROUP} ${SLAPD_DIR}/sasl2 && service slapd restart" \ --renew-hook "cp ${ETC}/letsencrypt/live/${name}/*.pem ${SLAPD_DIR}/sasl2 && chown -R ${CONF_OWNER}:${CONF_GROUP} ${SLAPD_DIR}/sasl2 && service slapd restart" \
--automatic-renewal "${staging}" --automatic-renewal ${staging}
fi fi
require="__package/${PKG_MAIN}" __directory ${SLAPD_DIR}/slapd.d --state absent require="__package/${PKG_MAIN}" __directory ${SLAPD_DIR}/slapd.d --state absent