-
-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
Make "pihole -d -a" automatically upload the debug log #1268
Conversation
… token + some cosmetics for displaying on the web interface
You mean |
Whoops, Yes. Not |
Changed title to "Make Dan finish the code already..." |
@@ -336,7 +342,11 @@ countdown() { | |||
printf "::: Logging will automatically teminate in ${TIMEOUT} seconds\n" | |||
while [ $tuvix -ge 1 ] | |||
do | |||
printf ":::\t${tuvix} seconds left. \r" | |||
if ! ${automated} ; then | |||
printf ":::\t${tuvix} seconds left. \r" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Has no newline, causing output like this (captured via web debug):
::: 60 seconds left. ::: 55 seconds left. ::: 50 seconds left.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There shouldn't be a new line, it has a carriage return to overwrite the counter on a terminal display.
Do not merge until the back end code is there, or we're going to have a bad time. |
See #1281 for back and updates. |
Does this conflict with #1281 ? |
Implemented by #1281 |
By submitting this pull request, I confirm the following (please check boxes, eg [X]) Failure to fill the template will close your PR:
Please submit all pull requests against the
development
branch. Failure to do so will delay or deny your requestHow familiar are you with the codebase?: 10
Needed for pi-hole/web#399
Can be merged w/o merging pi-hole/web#399
This template was created based on the work of
udemy-dl
.