Skip to content

Commit 006e8e7

Browse files
authored
Excluded the notify daemon from the jq version check. (#128)
This effectively bandages the issue of not being able to use the notification daemons on Broad filesystems because IT won't upgrade our version of JQ.
1 parent 4f53689 commit 006e8e7

File tree

1 file changed

+9
-4
lines changed

1 file changed

+9
-4
lines changed

cromshell

Lines changed: 9 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1873,10 +1873,6 @@ if ${ISINTERACTIVESHELL} ; then
18731873
r=$?
18741874
[[ ${r} -ne 0 ]] && exit 6
18751875

1876-
checkJQVersion
1877-
r=$?
1878-
[[ ${r} -ne 0 ]] && exit 6
1879-
18801876
# Sanitize the displayed sub-command:
18811877
SUB_COMMAND_FOR_DISPLAY=${SUB_COMMAND}
18821878
case ${SUB_COMMAND} in
@@ -1888,6 +1884,15 @@ if ${ISINTERACTIVESHELL} ; then
18881884
;;
18891885
esac
18901886

1887+
# Don't check JQ version if we're notifying.
1888+
# Why do this, you might ask?
1889+
# Because it's easier to hack our code than to get Broad IT to update software on our servers.
1890+
if [[ "${SUB_COMMAND_FOR_DISPLAY}" != "notify" ]] ; then
1891+
checkJQVersion
1892+
r=$?
1893+
[[ ${r} -ne 0 ]] && exit 6
1894+
fi
1895+
18911896
# Handle specific sub-command args and and call our sub-command:
18921897
error "Sub-Command: ${SUB_COMMAND_FOR_DISPLAY}"
18931898
case ${SUB_COMMAND} in

0 commit comments

Comments
 (0)