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

Fix bug 3 #4

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

Fix bug 3 #4

wants to merge 2 commits into from

Conversation

fluks
Copy link

@fluks fluks commented Oct 12, 2019

This commit is built on top of the other PR, but I thought having a different PR for this commit is reasonable, because this deals with a different issue.

fluks added 2 commits October 8, 2019 22:09
Find sudo wherever in PATH it may be.
…-zero

If user gave correct password but the command returned non-zero, the
program kept asking for a password. This commit doesn't keep asking for
the password and it shows what the command outputs and returns a its
exit value.

Fixes dubs3c#3.
@dubs3c
Copy link
Owner

dubs3c commented Oct 12, 2019

This is super cool, I'll check the code soon!

@Kontrakills3
Copy link

What is password?

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