Skip to content

Fix: Use correct PID for signals to processes#1581

Merged
deckstose merged 1 commit intoaristocratos:mainfrom
yinghao-w:fix-signal-send-process
Mar 23, 2026
Merged

Fix: Use correct PID for signals to processes#1581
deckstose merged 1 commit intoaristocratos:mainfrom
yinghao-w:fix-signal-send-process

Conversation

@yinghao-w
Copy link
Copy Markdown
Contributor

Fixes #846. Process signal popup menus use the "current" selected process, which can drift around when not following processes. Fix by retaining and using the initial PID selected at popup creation. Applies to terminate, Kill, signals and Nice.

Comment thread src/btop_menu.cpp Outdated
@yinghao-w yinghao-w force-pushed the fix-signal-send-process branch from 49d7a77 to e62dad0 Compare March 23, 2026 14:33
Process signal popup menus use the "current" selected process, which can drift
around. Fix by retaining the initial PID at popup creation.
@yinghao-w yinghao-w force-pushed the fix-signal-send-process branch from e62dad0 to 08704a5 Compare March 23, 2026 14:33
@deckstose deckstose merged commit 6164470 into aristocratos:main Mar 23, 2026
43 checks passed
@yinghao-w yinghao-w deleted the fix-signal-send-process branch March 23, 2026 16:52
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] signal gets send to wrong process

2 participants