support for run in alpine linux#50
Open
niko-lay wants to merge 1 commit into
Open
Conversation
|
This won't work on non-Linux systems... |
Closed
edmundlod
added a commit
to edmundlod/postallow
that referenced
this pull request
Apr 3, 2026
ps(1) with a PID argument is not supported by busybox ps (used on Alpine Linux and other minimal environments), causing show_dots to error or behave incorrectly. Replace with `kill -0 $PID` which is POSIX-compliant and works universally across busybox, GNU coreutils, macOS, and BSD. kill -0 sends no signal; it merely checks whether the process exists and is signalable by the caller — safe to use since we own the background process we spawned. Spotted via stevejenkins#50 which worked around the same issue using /proc/$PID, a Linux-only approach. kill -0 is strictly more portable.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
As alpine linux using
psfrombusyboxit not support to check separate PID