Error with pomodor.notify when using a script

Bug Report

Summary

Affected module: pomodoro

Description:

Error with pomodoro.notify when using a script as outline in issue #532

How to reproduce

Following the steps in issue #532 I created a pomodoro-notify script with the following code and placed it in a Scripts folder in my home directory:

#!/bin/bash

notify-send -u critical "Time Up!" & tput bel

I then tested the script in a terminal and everything worked as expected so I inserted into the pomodo module as such: pomodoro.notify=“/home/dalton/Scripts/pomodoro-notify”

However when I ran the timer like normal (via left-click on the module) at the end of the work period it appears to have ran the script twice and not ran the bell. The “Time Up!” text pops up with the correct urgency so I know that the script is running but the bell sound doesn’t play and the timer resets with 1440 minutes as the work interval.

[ghost]

I believe the correct syntax is && not &

You can test with

echo "foo" & echo "bar"

vs

echo "foo" && echo "bar"

Single & is supposed to send the command to background

[ghost]

I didn’t realize that on most systems that would try to ring a pc speaker not a sound device via alsa, pulse, etc.

That was just my assumption. I think at least it used to be like that long ago, maybe even in pre-alsa era.