Skip to content

fix: support Linux real-time signals - #15

Merged
cataggar merged 4 commits into
kfrom
fix/issue-8-realtime-signals
Aug 20, 2026
Merged

cataggar merged 4 commits into
kfrom
fix/issue-8-realtime-signals

Conversation

@cataggar

Copy link
Copy Markdown
Owner

Summary

  • model Linux real-time signals using runtime libc bounds
  • support canonical SIGRTMIN/SIGRTMAX names, offsets, numeric lookup, listing, and delivery
  • keep non-Linux signal behavior unchanged and add focused unit/compatibility coverage

Validation

  • cargo fmt --check --all
  • cargo test --package brush-core signal --lib
  • cargo test --package brush-builtins
  • cargo test --test brush-compat-tests -- 'Builtins: kill'
  • cargo test --test brush-compat-tests -- 'Builtins: trap'
  • cargo clippy --workspace --all-features --all-targets

Closes #8

cataggar and others added 4 commits August 20, 2026 02:49
Assisted-by: GitHub Copilot CLI:gpt-5.6-sol

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Preserve the nix Signal payload while representing real-time signals separately, and decode signal-derived statuses in kill -l.

Assisted-by: GitHub Copilot:gpt-5.4 [CLI]

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Compile the stub modules during host unit tests so misplaced platform-only APIs are caught locally.

Assisted-by: GitHub Copilot:gpt-5.4

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Make the non-Linux realtime signal lookup and analogous stub helpers const so cross-platform Clippy accepts them.

Assisted-by: GitHub Copilot:gpt-5.4 [CLI]

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@cataggar
cataggar merged commit c9c41d6 into k Aug 20, 2026
38 checks passed
@cataggar
cataggar deleted the fix/issue-8-realtime-signals branch August 20, 2026 03:29
cataggar added a commit that referenced this pull request Aug 22, 2026
* fix: support Linux real-time signals

Assisted-by: GitHub Copilot CLI:gpt-5.6-sol

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* fix: address real-time signal review blockers

Preserve the nix Signal payload while representing real-time signals separately, and decode signal-derived statuses in kill -l.

Assisted-by: GitHub Copilot:gpt-5.4 [CLI]

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* fix: compile realtime signal stubs

Compile the stub modules during host unit tests so misplaced platform-only APIs are caught locally.

Assisted-by: GitHub Copilot:gpt-5.4

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* fix: mark signal stubs const

Make the non-Linux realtime signal lookup and analogous stub helpers const so cross-platform Clippy accepts them.

Assisted-by: GitHub Copilot:gpt-5.4 [CLI]

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

---------

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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.

Real-time signals (SIGRTMIN/SIGRTMAX and offsets) are not recognized by trap or kill -l

1 participant