search

topisani / Discurses

A Python discord CLI focused on usability
MIT License
140 stars 14 forks source link

Notify-send spam #54

Closed LarBob closed 5 years ago

LarBob commented 6 years ago

On platforms without notify-send (e.g. macOS), "sh: notify-send: command not found" gets spammed within Discurses.

Ge0 commented 6 years ago

Nice catch,

I think it's even dangerous to use os.system this way (I mean, with $PATH tricking).

Moreover, I think this should not be used in a few cases (like when you don't have any X server running...).