BunsenLabs / bunsen-utilities

https://pkg.bunsenlabs.org/debian/pool/main/b/bunsen-utilities/
GNU General Public License v3.0
30 stars 21 forks source link

beepmein #67

Closed brontosaurusrex closed 4 years ago

brontosaurusrex commented 4 years ago

alarm clock.

johnraff commented 4 years ago

I've been getting emails to root@localhost from the at jobs, like:

Playing: /home/john/.cache/beepmein/remind
 (+) Audio --aid=1 (pcm_s16le)
AO: [pulse] 11025Hz mono 1ch s16
A: 00:00:00 / 00:00:01 (2%)
A: 00:00:00 / 00:00:01 (8%)
A: 00:00:00 / 00:00:01 (14%)
A: 00:00:00 / 00:00:01 (18%)
A: 00:00:00 / 00:00:01 (29%)
A: 00:00:00 / 00:00:01 (36%)
A: 00:00:00 / 00:00:01 (48%)
A: 00:00:01 / 00:00:01 (59%)
A: 00:00:01 / 00:00:01 (70%)
A: 00:00:01 / 00:00:01 (81%)

Exiting... (End of file)

Running mpv in a terminal, passing --no-terminal suppresses the output, as does adding &>/dev/null For cvlc, even the option -q still outputs a line and only &>/dev/null seems to work. So, possibly amend playa()? https://github.com/BunsenLabs/bunsen-utilities/blob/75be425991a4e25ccf4175d9b5f322eb06956fa1/bin/beepmein#L149-L152

mpv --no-resume-playback --no-video "$file" &>/dev/null
...
cvlc --no-video "$file" vlc://quit &>/dev/null
johnraff commented 4 years ago

This branch looks as if it was forked from helium, not lithium. That probably means complications if it is merged directly into the lithium branch.

@brontosaurusrex I wonder if you could make a new fork from the lithium branch, add bin/beepmein to that and raise a new Pull Request? Sorry for the extra work.

EDIT: alternatively I could checkout the single file bin/beepmein from brex into lithium. The new file would be added with a single commit message, though. All your previous commit messages would be lost.

johnraff commented 4 years ago

beepmein has been pulled into the lithium branch as a single file, so now closing this PR. Please post any further issues or PRs referring to bin/beepmein with respect to the version on the lithium branch.

johnraff commented 4 years ago

Please do not attempt to merge this branch into lithium - it was forked from helium.