Skip to content

fix(nix): pylsp 1.14.0 の checkPhase を overlay でスキップ - #67

Merged
Kurichi merged 1 commit into
mainfrom
fix/nix-pylsp-overlay
May 3, 2026
Merged

Kurichi merged 1 commit into
mainfrom
fix/nix-pylsp-overlay

Conversation

@Kurichi

@Kurichi Kurichi commented May 3, 2026

Copy link
Copy Markdown
Owner

Summary

  • nix run .#switch が python-lsp-server-1.14.0 のテスト失敗(upstream python-lsp/python-lsp-server#605: pytest teardown で logging on closed stream)で止まる問題の workaround
  • nix/overlays/default.nix に pythonPackagesExtensions 経由で python-lsp-server のみ doCheck = false を当てる
  • upstream / nixpkgs に修正が降りたら overlay 追加分を削除する

背景

  • programs.neovim.extraPackages (nix/modules/home/programs/neovim.nix:19) で python312Packages.python-lsp-server を引き込んでおり、neovim wrapper の build closure に乗るためエラーが neovim 連鎖失敗を起こしていた
  • nixpkgs master でも 1.14.0 + 既存 disabledTests の組み合わせで本テスト失敗は拾えていない
  • 局所スコープ(python312Packages のみ)ではなく追従性を優先して pythonPackagesExtensions 経由で当て、pyPrev ? python-lsp-server でガード

Test plan

  • nix eval --raw .#darwinConfigurations.personal.pkgs.python312Packages.python-lsp-server.drvPath → 新 drv 生成
  • nix derivation show <drv> | jq → doCheck: ""(false 反映)
  • nix run .#build 完走
  • nix log <drv> に pytest / test session starts 出現なし(pythonImportsCheckPhase のみ)
  • nix run .#switch 完走(activation hook 全通)
  • Neovim で Python ファイルを開き :LspInfo で pylsp attached を実機確認

🤖 Generated with Claude Code

nixpkgs-unstable の python-lsp-server 1.14.0 が Python 3.13 で
pytest teardown の logging-on-closed-stream 競合 (upstream
python-lsp/python-lsp-server#605) に当たり、`nix run .#switch` が
neovim 連鎖ビルド失敗で止まる。

programs.neovim.extraPackages 経由で pylsp が neovim wrapper の
build closure に入っており、nixpkgs 側に修正が降りていないため
overlay で doCheck = false する暫定 workaround を入れる。
upstream / nixpkgs 修正後に削除する。

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@Kurichi
Kurichi merged commit d97c768 into main May 3, 2026
2 checks passed
@Kurichi
Kurichi deleted the fix/nix-pylsp-overlay branch May 3, 2026 08:05
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.

1 participant