about summary refs log tree commit diff
path: root/pkgs/development/tools/sauce-connect
diff options
context:
space:
mode:
authorTobias Geerinckx-Rice <me@tobias.gr>2016-06-23 12:10:44 +0200
committerTobias Geerinckx-Rice <me@tobias.gr>2016-06-23 12:13:24 +0200
commita0551529633fc2228484c4890cd96247f30f7dfa (patch)
tree6ac1d396f1e98bd3efd2919b765b2e1dd8f8aa3e /pkgs/development/tools/sauce-connect
parent6669fac77e79d4af0f956eec5317d2ae1f220503 (diff)
sauce-connect: listify phases and run in correct order
patchPhase was being run after installPhase, causing patchelf to be
run on the left-behind binary instead of the copy in $out. Fixes:

  ```
  $ .../bin/sc
  bash: .../bin/sc: No such file or directory
  $ patchelf --print-interpreter .../bin/sc
  /lib64/ld-linux-x86-64.so.2
  ```
Diffstat (limited to 'pkgs/development/tools/sauce-connect')
-rw-r--r--pkgs/development/tools/sauce-connect/default.nix4
1 files changed, 3 insertions, 1 deletions
diff --git a/pkgs/development/tools/sauce-connect/default.nix b/pkgs/development/tools/sauce-connect/default.nix
index 6b6d5b1439700..e93006b89ce3a 100644
--- a/pkgs/development/tools/sauce-connect/default.nix
+++ b/pkgs/development/tools/sauce-connect/default.nix
@@ -20,7 +20,9 @@ stdenv.mkDerivation rec {
   );
 
   buildInputs = [ unzip ];
-  phases = "unpackPhase installPhase " + (if stdenv.system == "x86_64-darwin" then "" else "patchPhase");
+  phases = [ "unpackPhase" ]
+   ++ (lib.optionals (stdenv.system != "x86_64-darwin") [ "patchPhase" ])
+   ++ [ "installPhase " ];
 
   patchPhase = ''
     patchelf \