Merge pull request #257687 from trofi/fluidsynth-update

{ stdenv, lib, fetchFromGitHub, fetchpatch, buildPackages, pkg-config, cmake

{ stdenv, lib, fetchFromGitHub, buildPackages, pkg-config, cmake

, alsa-lib, glib, libjack2, libsndfile, libpulseaudio

, AppKit, AudioUnit, CoreAudio, CoreMIDI, CoreServices

}:

stdenv.mkDerivation rec {

  pname = "fluidsynth";

  version = "2.3.3";

  version = "2.3.4";

  src = fetchFromGitHub {

    owner = "FluidSynth";

    repo = "fluidsynth";

    rev = "v${version}";

    sha256 = "sha256-RqhlpvMbRSwdcY2uuFAdJnihN3aObcLVMuvCZ294dgo=";

    hash = "sha256-3qLmo9Ibl44v6Jj5Ix17ixwqfPt3ITTXUqBETF5pzE4=";

  };

  patches = [

    # Fixes bad CMAKE_INSTALL_PREFIX + CMAKE_INSTALL_LIBDIR concatenation for Darwin install name dir

    # Remove when PR merged & in release

    (fetchpatch {

      name = "0001-Fix-incorrect-way-of-turning-CMAKE_INSTALL_LIBDIR-absolute.patch";

      url = "https://github.com/FluidSynth/fluidsynth/pull/1261/commits/03cd38dd909fc24aa39553d869afbb4024416de8.patch";

      hash = "sha256-nV+MbFttnbNBO4zWnPLpnnEuoiESkV9BGFlUS9tQQfk=";

    })

  ];

  outputs = [ "out" "dev" "man" ];

  nativeBuildInputs = [ buildPackages.stdenv.cc pkg-config cmake ];