From b248f3592fb529f3076d7c3c61bbf5df917c154b Mon Sep 17 00:00:00 2001
From: Simon Gardling <titaniumtown@proton.me>
Date: Mon, 4 May 2026 20:36:24 -0400
Subject: [PATCH] pi: drop redundant pkgs.ghidra from home.packages

The pyghidra-mcp wrapper bakes in GHIDRA_INSTALL_DIR via makeWrapperArgs
referencing pkgs.ghidra, which makes ghidra a runtime closure dep.
Adding pkgs.ghidra explicitly to home.packages caused buildEnv to merge
*two* ghidra-12.0.4 store paths (one from pyghidra's propagatedBuildInputs,
one from the explicit list) and fail with a path collision on
GPL/DMG/LICENSE.txt.

Drop the explicit add. The agent-driven workflow doesn't need the GUI;
manual exploration via 'nix run nixpkgs#ghidra' is one command away if
ever wanted.
---
 home/progs/pi.nix | 1 -
 1 file changed, 1 deletion(-)

diff --git a/home/progs/pi.nix b/home/progs/pi.nix
index 2a8bc75..ffcd922 100644
--- a/home/progs/pi.nix
+++ b/home/progs/pi.nix
@@ -138,7 +138,6 @@ in
   home.packages = [
     inputs.llm-agents.packages.${pkgs.stdenv.hostPlatform.system}.omp
     pyghidra-mcp
-    pkgs.ghidra # GUI Ghidra for occasional manual exploration
   ];
 
   home.file = androidSkillFiles // {