[PATCH] tools: mark start-vmm test as requiring KVM
This will tell Nix (on a correctly configured system) that if it doesn't have KVM, it can't build the package, allowing it to fall back to outsourcing it to a remote builder in that case. Ideally, we don't want to have any part of the Spectrum build requiring KVM, but until that's fixed, occurences should at least be correctly marked. Fixes: a4e3a40 ("host/start-vmm: actually run "run" test") Signed-off-by: Alyssa Ross <hi@alyssa.is> --- Demi, if you find other occurences of derivations requiring KVM to build, please send a patch like this. :) tools/default.nix | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/tools/default.nix b/tools/default.nix index 7a2926d..908ec0d 100644 --- a/tools/default.nix +++ b/tools/default.nix @@ -142,7 +142,9 @@ stdenv.mkDerivation (finalAttrs: { } ); - run = runCommand "start-vmm-test" {} '' + run = runCommand "start-vmm-test" { + requiredSystemFeatures = [ "kvm" ]; + } '' ${run-spectrum-vm.override { spectrum-host-tools = finalAttrs.finalPackage; }} > $out base-commit: 9090caebe25310caa80a13787ef58b1f81658a78 -- 2.49.0
participants (1)
-
Alyssa Ross