From aaf353e7ad5f1519a884a8066f60bd49c66cb2f7 Mon Sep 17 00:00:00 2001 From: talyz Date: Wed, 19 Dec 2018 14:14:09 +0100 Subject: [PATCH] nomachine-client: Fix audio support libnxcau.so needs libpulse.so.0 for audio to work, but doesn't have a DT_NEEDED entry for it, so it's not automatically detected. This adds the missing entry and libpulseaudio to buildInputs, enabling autoPatchelfHook to fix the RPATH. --- pkgs/tools/admin/nomachine-client/default.nix | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/pkgs/tools/admin/nomachine-client/default.nix b/pkgs/tools/admin/nomachine-client/default.nix index bc718aeb48c..32d0e584f31 100644 --- a/pkgs/tools/admin/nomachine-client/default.nix +++ b/pkgs/tools/admin/nomachine-client/default.nix @@ -1,4 +1,5 @@ -{ stdenv, lib, file, fetchurl, makeWrapper, autoPatchelfHook, jsoncpp }: +{ stdenv, lib, file, fetchurl, makeWrapper, + autoPatchelfHook, jsoncpp, libpulseaudio }: let versionMajor = "6.4"; versionMinor = "6_1"; @@ -31,7 +32,7 @@ in ''; nativeBuildInputs = [ file makeWrapper autoPatchelfHook ]; - buildInputs = [ jsoncpp ]; + buildInputs = [ jsoncpp libpulseaudio ]; installPhase = '' rm bin/nxplayer bin/nxclient @@ -63,6 +64,10 @@ in postFixup = '' makeWrapper $out/bin/nxplayer.bin $out/bin/nxplayer --set NX_SYSTEM $out/NX makeWrapper $out/bin/nxclient.bin $out/bin/nxclient --set NX_SYSTEM $out/NX + + # libnxcau.so needs libpulse.so.0 for audio to work, but doesn't + # have a DT_NEEDED entry for it. + patchelf --add-needed libpulse.so.0 $out/NX/lib/libnxcau.so ''; dontBuild = true;