From 44f97b56d9cbf99fd22c751834cab731f34bec41 Mon Sep 17 00:00:00 2001 From: Profpatsch Date: Tue, 11 Jun 2019 20:10:03 +0200 Subject: [PATCH] bazel: move the python test to `py_binary` `py_test` tries to download unnecessary dependencies at runtime. We can just as well run it to check the assertion. Upstream issue: https://github.com/bazelbuild/bazel/issues/8575 --- .../tools/build-managers/bazel/python-bin-path-test.nix | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/pkgs/development/tools/build-managers/bazel/python-bin-path-test.nix b/pkgs/development/tools/build-managers/bazel/python-bin-path-test.nix index 54ae154a620..b78b736e47f 100644 --- a/pkgs/development/tools/build-managers/bazel/python-bin-path-test.nix +++ b/pkgs/development/tools/build-managers/bazel/python-bin-path-test.nix @@ -22,7 +22,7 @@ let srcs = [ "lib.py" ], ) - py_test( + py_binary( name = "bin", srcs = [ "bin.py" ], deps = [ ":lib" ], @@ -46,10 +46,9 @@ let # about why to create a subdir for the workspace. cp -r ${workspaceDir} wd && chmod u+w wd && cd wd ${bazel}/bin/bazel \ - test \ - --test_output=errors \ + run \ --host_javabase='@local_jdk//:jdk' \ - //... + //python:bin touch $out '';