Skip to content

Stop re-registering the WSL interop binfmt handler - #24

Merged
user3301 merged 1 commit into
masterfrom
fix-wsl-binfmt
Oct 5, 2026
Merged

user3301 merged 1 commit into
masterfrom
fix-wsl-binfmt

Conversation

@user3301

@user3301 user3301 commented Oct 5, 2026

Copy link
Copy Markdown
Owner

WSL 3 registers the WSLInterop handler itself and mounts a read-only file over /proc/sys/fs/binfmt_misc/status. systemd-binfmt can no longer flush rules, exits 1, and switch-to-configuration fails with status 4.

Drop wsl.interop.register so NixOS-WSL uses its default (the existing registration). WSLInterop was the only binfmt registration, so systemd-binfmt.service is no longer installed.

WSL 3 registers the WSLInterop handler itself and mounts a read-only
file over /proc/sys/fs/binfmt_misc/status. systemd-binfmt can no longer
flush rules, exits 1, and switch-to-configuration fails with status 4.

Drop wsl.interop.register so NixOS-WSL uses its default (the existing
registration). WSLInterop was the only binfmt registration, so
systemd-binfmt.service is no longer installed.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot review overview

🟢 Approval recommended

The change matches the pinned NixOS-WSL default and resolves the documented activation failure.

Review effort: Balanced
Findings: None

What changed in this PR

Removes redundant WSL interop registration to prevent systemd-binfmt failures on WSL 3.

Changes:

  • Relies on NixOS-WSL’s default existing interop handler.
  • Documents why explicit registration must remain disabled.
File Description
systems/​wsl/​configuration.nix Removes explicit binfmt registration and explains the WSL 3 constraint.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@user3301
user3301 merged commit b6e73ae into master Oct 5, 2026
2 checks passed
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.

2 participants