Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: NixOS/nixpkgs
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: fca0aad2586f
Choose a base ref
...
head repository: NixOS/nixpkgs
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: df89889a674d
Choose a head ref
  • 3 commits
  • 5 files changed
  • 3 contributors

Commits on Nov 12, 2020

  1. drumstick: fix plugin lookup and enable SonivoxEAS and FluidSynth plu…

    …gins
    
    SonivoxEAS is enabled by building with PulseAudio.
    orivej committed Nov 12, 2020

    Verified

    This commit was signed with the committer’s verified signature. The key has expired.
    NeQuissimus Tim Steinbach
    Copy the full SHA
    a1c9c43 View commit details
  2. vmpk: 0.5.1 -> 0.7.2

    Also migrate from qt4 to qt5 (#33248)
    freezeboy authored and orivej committed Nov 12, 2020

    Verified

    This commit was signed with the committer’s verified signature. The key has expired.
    NeQuissimus Tim Steinbach
    Copy the full SHA
    fd9f28c View commit details
  3. Merge pull request #102521 from freezeboy/update-vmpk

    vmpk: 0.5.1 -> 0.7.2
    orivej-nixos authored Nov 12, 2020
    Copy the full SHA
    df89889 View commit details
17 changes: 10 additions & 7 deletions pkgs/applications/audio/vmpk/default.nix
Original file line number Diff line number Diff line change
@@ -1,19 +1,22 @@
{ stdenv, fetchurl, cmake, pkgconfig, alsaLib, libjack2, qt4 }:
{ mkDerivation, lib, fetchurl, cmake, pkg-config
, qttools, qtx11extras, drumstick
, docbook-xsl-nons
}:

stdenv.mkDerivation rec {
mkDerivation rec {
pname = "vmpk";
version = "0.5.1";
version = "0.7.2";

src = fetchurl {
url = "mirror://sourceforge/${pname}/${version}/${pname}-${version}.tar.bz2";
sha256 = "11fqnxgs9hr9255d93n7lazxzjwn8jpmn23nywdksh0pb1ffvfrc";
sha256 = "5oLrjQADg59Mxpb0CNLQAE574IOSYLDLJNaQ/9q2cMQ=";
};

nativeBuildInputs = [ cmake pkgconfig ];
nativeBuildInputs = [ cmake pkg-config qttools docbook-xsl-nons ];

buildInputs = [ alsaLib libjack2 qt4 ];
buildInputs = [ qtx11extras drumstick ];

meta = with stdenv.lib; {
meta = with lib; {
description = "Virtual MIDI Piano Keyboard";
homepage = "http://vmpk.sourceforge.net/";
license = licenses.gpl3Plus;
24 changes: 16 additions & 8 deletions pkgs/development/libraries/drumstick/default.nix
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
{ stdenv, fetchurl, alsaLib, cmake, docbook_xsl, docbook_xml_dtd_45, doxygen
, fluidsynth, pkgconfig, qtbase, qtsvg
{ stdenv, fetchurl
, cmake, docbook_xml_dtd_45, docbook_xsl, doxygen, pkg-config, wrapQtAppsHook
, alsaLib, fluidsynth, qtbase, qtsvg, libpulseaudio
}:

stdenv.mkDerivation rec {
@@ -11,18 +12,25 @@ stdenv.mkDerivation rec {
sha256 = "1n9wvg79yvkygrkc8xd8pgrd3d7hqmr7gh24dccf0px23lla9b3m";
};

patches = [
./drumstick-fluidsynth.patch
./drumstick-plugins.patch
];

postPatch = ''
substituteInPlace library/rt/backendmanager.cpp --subst-var out
'';

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

enableParallelBuilding = true;

#Temporarily remove drumstick-piano; Gives segment fault. Submitted ticket
postInstall = ''
rm $out/bin/drumstick-vpiano
'';
nativeBuildInputs = [
cmake docbook_xml_dtd_45 docbook_xml_dtd_45 docbook_xsl doxygen pkg-config wrapQtAppsHook
];

nativeBuildInputs = [ cmake pkgconfig docbook_xsl docbook_xml_dtd_45 docbook_xml_dtd_45 ];
buildInputs = [
alsaLib doxygen fluidsynth qtbase qtsvg
alsaLib fluidsynth libpulseaudio qtbase qtsvg
];

meta = with stdenv.lib; {
Original file line number Diff line number Diff line change
@@ -0,0 +1,9 @@
It works with fluidsynth 2.

Backported from r400: https://sourceforge.net/p/drumstick/code/400/

--- a/library/rt-backends/CMakeLists.txt
+++ b/library/rt-backends/CMakeLists.txt
@@ -54,1 +54,1 @@ if (PKG_CONFIG_FOUND)
- pkg_check_modules(FLUIDSYNTH fluidsynth>=1.1.1 fluidsynth<=1.1.11)
+ pkg_check_modules(FLUIDSYNTH fluidsynth>=1.1.1)
12 changes: 12 additions & 0 deletions pkgs/development/libraries/drumstick/drumstick-plugins.patch
Original file line number Diff line number Diff line change
@@ -0,0 +1,12 @@
Make it look for its plugin in its own installation directory.

--- a/library/rt/backendmanager.cpp
+++ b/library/rt/backendmanager.cpp
@@ -159,6 +159,7 @@ namespace rt {
foreach(const QString& path, QCoreApplication::libraryPaths()) {
d->appendDir( path + QDir::separator() + QSTR_DRUMSTICK, result );
}
+ d->appendDir( "@out@/lib/drumstick", result );
return result;
}

2 changes: 1 addition & 1 deletion pkgs/top-level/all-packages.nix
Original file line number Diff line number Diff line change
@@ -24265,7 +24265,7 @@ in
onlyLibVLC = true;
};

vmpk = callPackage ../applications/audio/vmpk { };
vmpk = libsForQt5.callPackage ../applications/audio/vmpk { };

vmware-horizon-client = callPackage ../applications/networking/remote/vmware-horizon-client { };