Skip to content

Kill: Display error on invalid UTF-8 rather than list of signals#12878

Open
iburaky2 wants to merge 2 commits into
uutils:mainfrom
iburaky2:fix/kill-invalid-utf8-error-message
Open

Kill: Display error on invalid UTF-8 rather than list of signals#12878
iburaky2 wants to merge 2 commits into
uutils:mainfrom
iburaky2:fix/kill-invalid-utf8-error-message

Conversation

@iburaky2

Copy link
Copy Markdown
Contributor

Fixes #12816

@iburaky2 iburaky2 marked this pull request as draft June 14, 2026 18:30
@iburaky2 iburaky2 force-pushed the fix/kill-invalid-utf8-error-message branch from dd6998c to 1e33f1a Compare June 14, 2026 18:32
@github-actions

Copy link
Copy Markdown

GNU testsuite comparison:

Skip an intermittent issue tests/cut/bounded-memory (fails in this run but passes in the 'main' branch)
Skip an intermittent issue tests/tail/pid-pipe (fails in this run but passes in the 'main' branch)
Note: The gnu test tests/csplit/csplit-heap is now being skipped but was previously passing.
Note: The gnu test tests/rm/many-dir-entries-vs-OOM is now being skipped but was previously passing.
Congrats! The gnu test tests/cp/link-heap is now passing!
Congrats! The gnu test tests/seq/seq-epipe is now passing!
Note: The gnu test tests/env/env-signal-handler was skipped on 'main' but is now failing.

@iburaky2 iburaky2 marked this pull request as ready for review June 14, 2026 19:25
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.

bug(kill): when using -l with $'\xFF' no errors reported

1 participant