Skip to content

Commit

Permalink
Move all controller deps to buildInputs, remove utop, nodejs
Browse files Browse the repository at this point in the history
Having them in propagatedBuildInputs resulted in ocaml being included
in the system image and bloated it by an extra 500 MB.
  • Loading branch information
yfyf committed Sep 24, 2024
1 parent a9e2269 commit 41bca5f
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 6 deletions.
5 changes: 0 additions & 5 deletions controller/default.nix
Original file line number Diff line number Diff line change
Expand Up @@ -32,11 +32,6 @@ ocamlPackages.buildDunePackage rec {
];

buildInputs = with ocamlPackages; [
nodejs
utop
];

propagatedBuildInputs = with ocamlPackages; [
opium
ocaml_lwt
logs
Expand Down
1 change: 0 additions & 1 deletion controller/shell.nix
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,5 @@ pkgs.mkShell {
packages =
playos-controller.buildInputs
++ playos-controller.nativeBuildInputs
++ playos-controller.propagatedBuildInputs
++ [ pkgs.watchexec ];
}

0 comments on commit 41bca5f

Please sign in to comment.