-
-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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: add psmisc as dependency #2133
Conversation
automated install/basic-install.sh
Outdated
@@ -163,7 +163,13 @@ if command -v apt-get &> /dev/null; then | |||
# These programs are stored in an array so they can be looped through later | |||
INSTALLER_DEPS=(apt-utils dialog debconf dhcpcd5 git ${iproute_pkg} whiptail) | |||
# Pi-hole itself has several dependencies that also need to be installed | |||
<<<<<<< HEAD |
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.
Couldn't parse this here string.
The mentioned parser error was in this simple command.
Fix any mentioned problems and try again.
Debian 9.4 does not install `psmisc` by default and the following error will happen during installation: ``` [✗] /usr/local/bin/pihole: line 353: killall: command not found /usr/local/bin/pihole: line 364: killall: command not found ``` This patch adds `psmisc` (that contains `killall`) as dependency Signed-off-by: Michele Bologna <michele.bologna@gmail.com>
Thanks for the contribution. |
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.
Already included in Fedora deps. LGTM
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.
LGTM
@mbologna please contact me directly for a special thank you for contributing to our project. |
By submitting this pull request, I confirm the following:
please fill any appropriate checkboxes, e.g: [X]
git rebase
)Please make sure you Sign Off all commits. Pi-hole enforces the DCO.
What does this PR aim to accomplish?:
Debian 9.4 does not install
psmisc
by default and the following error will happen during installation:How does this PR accomplish the above?:
This patch adds
psmisc
(that containskillall
) as dependency