Separate buildInputs and nativeBuildInputs

lowRISC · Jan 3, 2024 · 1127ca6 · 1127ca6
1 parent 9eebc9b
commit 1127ca6
Showing 1 changed file with 13 additions and 13 deletions.
diff --git a/flake.nix b/flake.nix
@@ -45,13 +45,16 @@
               poetry2nix.defaultPoetryOverrides
             ];
           };
-
-        # This is the final list of dependencies we need to build the project.
-        project_deps =
-          [
+      in {
+        devShells.default = pkgs.mkShell {
+          name = "labenv";
+          buildInputs = with pkgs; [
+            libelf
+            zlib
+          ];
+          nativeBuildInputs = with pkgs; [
             pythonEnv
-          ]
-          ++ (with pkgs; [
+
             cmake
             openocd
             screen
@@ -70,17 +73,14 @@
             gtkwave
             srecord
             openfpgaloader
-            libelf
-            zlib
             # vivado
 
             # Poetry tool not required, add for convience in case update is needed
             poetry
-          ]);
-      in {
-        devShells.default = pkgs.mkShell {
-          name = "labenv";
-          buildInputs = project_deps;
+
+            # By default mkShell adds non-interactive bash to PATH
+            bashInteractive
+          ];
           shellHook = ''
             # FIXME This works on Ubuntu, may not on other distros. FIXME
             export LOCALE_ARCHIVE=/usr/lib/locale/locale-archive