drewrisinger/nix-scripts
1
0
Fork 0
forked from M-Labs/nix-scripts
Code Issues Pull Requests Projects Releases Wiki Activity

shells (artiq-fast): change artiqpkgs, vivado to arguments

Browse Source 
This allows overriding the artiqpkgs used downstream.
The issue is that previously artiqpkgs required <artiqSrc> to be defined,
and overriding that required setting paths, added complexity, and possible impurities.

This solution allows reuse of the shell[-dev], from a given version of
ARTIQ, without necessarily specifying source or a local source repo.
This commit is contained in:
Drew Risinger 2022-01-10 09:26:11 -05:00
parent 9a8a59c525
commit 47960f539b
2 changed files with 25 additions and 27 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
artiq-fast/shell-dev.nix
View File

@ -1,11 +1,10 @@
{ pkgs ? import <nixpkgs> {}}: { pkgs ? import <nixpkgs> {}
, artiqpkgs ? import ./default.nix { inherit pkgs; }
, vivado ? import ./vivado.nix { inherit pkgs; }
}:
let assert pkgs.lib.asserts.assertMsg (!pkgs.lib.strings.versionAtLeast artiqpkgs.artiq.version "7.0") "For ARTIQ 7+, use 'nix develop' on the flake instead.";
artiqpkgs = import ./default.nix { inherit pkgs; }; pkgs.mkShell {
vivado = import ./vivado.nix { inherit pkgs; };
in
assert pkgs.lib.asserts.assertMsg (!pkgs.lib.strings.versionAtLeast artiqpkgs.artiq.version "7.0") "For ARTIQ 7+, use 'nix develop' on the flake instead.";
pkgs.mkShell {
buildInputs = [ buildInputs = [
vivado vivado
pkgs.gnumake pkgs.gnumake
@ -18,4 +17,4 @@ in
artiqpkgs.openocd artiqpkgs.openocd
]; ];
TARGET_AR="or1k-linux-ar"; TARGET_AR="or1k-linux-ar";
} }

11
artiq-fast/shell.nix
View File

@ -1,8 +1,7 @@
{ pkgs ? import <nixpkgs> {}}: { pkgs ? import <nixpkgs> {}
, artiqpkgs ? import ./default.nix { inherit pkgs; }
}:
let pkgs.mkShell {
artiqpkgs = import ./default.nix { inherit pkgs; };
in
pkgs.mkShell {
buildInputs = [ (pkgs.python3.withPackages(ps: [artiqpkgs.artiq])) ]; buildInputs = [ (pkgs.python3.withPackages(ps: [artiqpkgs.artiq])) ];
} }