0

I was working on a zsh script to stop me from commiting outside of working hours, to enforce a stronger work life balance. I have this script at the moment which will warn me if I am outside of working hours with a console message, however it doesn't stop the commit from going through, any ideas?

#!/bin/zsh

while :; do
  currenttime=$(date +%H:%M)
  if [[ "$currenttime" < "15:00" ]] || [[ "$currenttime" > "22:00" ]]; then
    echo "Not in working time"
    exit 1
  else
    echo "Within work time"
  fi
done &
exit 0
Holger Just
  • 52,918
  • 14
  • 115
  • 123
Frustrated
  • 111
  • 1
  • 6

1 Answers1

3

This construct:

while ... do ... done &

runs the entire loop as a background process spun off from the current shell and does not wait for it to complete and therefore ignores its exit status. That leaves your script running just one more line:

exit 0

which sets the status for the hook: "all is well, let the commit happen!"

You should not be using a loop in the first place: just use a simple if test. Do not background it.

(As a general rule for shell tests, beware of lexicographic comparisons. As LeGEC notes in a comment, it should work OK for your particular case. I don't use zsh so I'm not sure if it behaves the same as bash here, but see Why is "[[ 10 < 2 ]]" true when comparing numbers in bash? and How can I compare numbers in Bash? Note further that you only need the numeric hour for the test you're using here, which could simplify things, but if you do want to check both hours and minutes, it may help to split them into two numbers. A handy trick is to use set -- $(date +%H +%M) so that $1 holds the hour and $2 holds the minute.)

torek
  • 448,244
  • 59
  • 642
  • 775