Why does this simple email script sends two emails every time

Question

Why does this script send two emails instead of one?

#!/bin/sh
MONITORDIR="/path/to/directory"
inotifywait -m -r -e create --format '%w%f' "${MONITORDIR}" | while read 
NEWFILE
do
[  -z "${NEWFILE}" ] && : || MOVIE=$(find "$NEWFILE" -type f -not -name ".*" 
\( -iname "*.mkv" -o -iname "*.mp4" -o -iname "*.avi" \) -exec basename {} 
\; | sed 's/\.[^.]*$//')
[  -z "${MOVIE}" ] && : || echo "Please don't reply to this 
automatically generated email message." | mail -s "$MOVIE is now on Plex!" 
"email1@gmail.com,email2@gmail.com"
done

tried replacing the two email addresses in the second last line with one? — Syfer, Sep 14 '17 at 00:49
Just confirming, it sends the same email twice to both the emails address? — swapnilsm, Sep 22 '17 at 09:32

score 2 · Answer 1 · answered Sep 25 '17 at 02:29

2

[  -z "${MOVIE}" ] && : || echo "Please don't reply to this

Note that A && B || C is not if-then-else. C may run when A is true.

answered Sep 25 '17 at 02:29

Duc Filan

6,769
3
21
26

Why does this simple email script sends two emails every time

1 Answers1