From 2ba87b6e79793e04da36c4e00768edf90b937f71 Mon Sep 17 00:00:00 2001
From: Profpatsch <mail@profpatsch.de>
Date: Mon, 13 Jul 2020 14:16:17 +0200
Subject: [PATCH] bazel: use install for completions

Preserving existing behavior: the bash completion was not executable,
the zsh completion was; according to lukegb the fish completion does
not have to be executable.
---
 .../tools/build-managers/bazel/bazel_3/default.nix       | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

diff --git a/pkgs/development/tools/build-managers/bazel/bazel_3/default.nix b/pkgs/development/tools/build-managers/bazel/bazel_3/default.nix
index c867ed7a85d..a9701b605d5 100644
--- a/pkgs/development/tools/build-managers/bazel/bazel_3/default.nix
+++ b/pkgs/development/tools/build-managers/bazel/bazel_3/default.nix
@@ -507,9 +507,12 @@ stdenv.mkDerivation rec {
     mv ./bazel_src/output/bazel $out/bin/bazel-${version}-${system}-${arch}
 
     # shell completion files
-    mkdir -p $out/share/bash-completion/completions $out/share/zsh/site-functions
-    mv ./bazel_src/output/bazel-complete.bash $out/share/bash-completion/completions/bazel
-    cp ./bazel_src/scripts/zsh_completion/_bazel $out/share/zsh/site-functions/
+    install -Dm644 \
+      ./bazel_src/output/bazel-complete.bash \
+      $out/share/bash-completion/completions/bazel/bazel-complete.bash
+    install -Dm644 \
+      ./bazel_src/scripts/zsh_completion/_bazel \
+      $out/share/zsh/site-functions/_bazel
   '';
 
   doInstallCheck = true;