Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding an infinite loop option. #4

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

Nicals
Copy link

@Nicals Nicals commented Oct 13, 2012

Hi there,
I use to use my computer as one of many alarm clock. And I use beep for that.
I feel that this program lacks of a loop option that loops the whole beep sequence.
Well, not anymore.

For exemple, this will launch the old Appolo spacecraft alarm system until beep recieves a SIGINT:
$ beep -f750 -l400 -n -f2000 -l400 -i

@federico-razzoli
Copy link

Great idea! I hope this will be accepted

@Nicals
Copy link
Author

Nicals commented Feb 7, 2013

Thanks. The patch contains a bug, the beep doesn't stop when SIGTERM is sent. You've just remind me to work on this ! I will correct this in a few days.

@rhonda
Copy link
Collaborator

rhonda commented Feb 7, 2013

Niclas, that part is a different issue, and I patched it in Debian already with this patch:
http://patch-tracker.debian.org/patch/series/view/beep/1.3-3/catch-sig-term

I just haven't sent any pull request for that yet so that Jonath can include it. Sorry to have forgotten about it. :)

@Nicals
Copy link
Author

Nicals commented Feb 7, 2013

Got it ! I remember seeing your patch in the Debian sources, and didn't notive that this patch wasn't in this version so I thought I've messes up something.
It is now corrected. Thanks

@rhonda
Copy link
Collaborator

rhonda commented Feb 7, 2013

Doh, I didn't remember that I have push access myself, so I just did that right ahead now. Sorry for duplication of work. :)

ndim added a commit to ndim/beep that referenced this pull request Sep 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants