Skip to content

Don't output \r on SIGINT#13

Open
ghost wants to merge 1 commit intomasterfrom
unknown repository
Open

Don't output \r on SIGINT#13
ghost wants to merge 1 commit intomasterfrom
unknown repository

Conversation

@ghost
Copy link
Copy Markdown

@ghost ghost commented Aug 26, 2012

The \r character causes the 'Exiting' message to overwrite other output. For example, when used with Foreman, exiting Foreman causes this output:

Exiting1 worker.1      | 

instead of

17:00:01 worker.1      | Exiting

It's not the responsibility of stalker to clean up weird terminal states. Besides, stdout might not even be connected to a terminal.

This patch removes the \r character.

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.

0 participants