refactor: xserverとディスプレイマネージャとIceWMの設定を分離#1310
Conversation
ncaq
changed the title
nvd diff: seminar (base: master)Version changesClosure size: 4965 -> 4965 (3 paths added, 3 paths removed, delta +0, disk usage +0B). |
![claude[bot]](https://avatars.githubusercontent.com/in/1236702?s=60&v=4){kind=small}
There was a problem hiding this comment.
xserverの基本設定・ディスプレイマネージャ・IceWM復旧環境を3ファイルに分離するリファクタリングです。nixos/native-linux/はdefault.nixのimportDirModules ./.で自動importされるため, 新規追加したdisplay-manager.nixとxserver-recovery.nixはimportsへの手動追記なしに読み込まれます。設定内容(displayManagerのsession/autoLogin, icewm, 復旧用パッケージ, xkb)は移動のみで欠落や重複はなく, 挙動は等価に保たれています。関心事ごとにファイルが分かれ, 新しいウィンドウマネージャを追加する際の見通しが良くなる妥当な分割です。
nvd diffでもclosure size据え置き(delta +0)で, システムへの影響がないことが確認できています。
指摘は設計意図コメントの喪失に関するNOTE 1件と, マジックストリング重複に関するTIP 1件のみで, いずれも任意対応の軽微なものです。
新しいウィンドウマネージャを追加したくなったときなどにスッキリとさせたいため。
There was a problem hiding this comment.
前回レビュー(commit 8a7f672)から, マジックストリングhm-xsessionのlet束縛化と, デフォルトセッションであることを示すdescriptionの追記が入り, 前回のTIP/NOTEはいずれも解消されています.
差分は依然として設定の移動が中心で, services.xserver.enable/xkb/libinputをxserver.nixに, ディスプレイマネージャ関連をdisplay-manager.nixに, IceWM復旧環境をxserver-recovery.nixに振り分ける関心分離として妥当です. nixos/native-linux/はimportDirModules ./.で自動importされるため, 新規2ファイルはimportsへの手動追記なしに読み込まれます.
残る指摘は, 分割で生じたファイル間の暗黙依存とdescriptionの状態依存記述に関するNOTEのみで, いずれも任意対応の軽微なものです. 挙動は等価に保たれており, リファクタリングの目的は達成されています.
1 participant
新しいウィンドウマネージャを追加したくなったときなどにスッキリとさせたいため。