From 5b9065da7e0fcb68806545cf1963a676323664a3 Mon Sep 17 00:00:00 2001 From: Nikolay Korotkiy Date: Sat, 22 Apr 2023 01:50:30 +0400 Subject: [PATCH 1/2] pulseview: refactor * Remove Qt's mkDerivation in favour of stdenv.mkDerivation to fix https://github.com/NixOS/nixpkgs/issues/180841 * Move qttools to nativeBuildInputs --- .../science/electronics/pulseview/default.nix | 22 +++++++++---------- 1 file changed, 11 insertions(+), 11 deletions(-) diff --git a/pkgs/applications/science/electronics/pulseview/default.nix b/pkgs/applications/science/electronics/pulseview/default.nix index 7f001e73cf7b..8906076ec103 100644 --- a/pkgs/applications/science/electronics/pulseview/default.nix +++ b/pkgs/applications/science/electronics/pulseview/default.nix @@ -1,23 +1,23 @@ -{ mkDerivation, lib, fetchurl, fetchpatch, pkg-config, cmake, glib, boost, libsigrok +{ lib, stdenv, fetchurl, fetchpatch, pkg-config, cmake, glib, boost, libsigrok , libsigrokdecode, libserialport, libzip, udev, libusb1, libftdi1, glibmm -, pcre, python3, qtbase, qtsvg, qttools +, pcre, python3, qtsvg, qttools, wrapQtAppsHook }: -mkDerivation rec { +stdenv.mkDerivation rec { pname = "pulseview"; version = "0.4.2"; src = fetchurl { - url = "https://sigrok.org/download/source/pulseview/${pname}-${version}.tar.gz"; - sha256 = "1jxbpz1h3m1mgrxw74rnihj8vawgqdpf6c33cqqbyd8v7rxgfhph"; + url = "https://sigrok.org/download/source/pulseview/pulseview-${version}.tar.gz"; + hash = "sha256-8EL3ej4bNb8wZmMw427Dj6uNJIw2k8N7fjXUAcO/q8s="; }; - nativeBuildInputs = [ cmake pkg-config ]; + nativeBuildInputs = [ cmake pkg-config qttools wrapQtAppsHook ]; buildInputs = [ glib boost libsigrok libsigrokdecode libserialport libzip udev libusb1 libftdi1 glibmm pcre python3 - qtbase qtsvg qttools + qtsvg ]; patches = [ @@ -25,13 +25,13 @@ mkDerivation rec { # PR at https://github.com/sigrokproject/pulseview/pull/39 (fetchpatch { url = "https://github.com/sigrokproject/pulseview/commit/fb89dd11f2a4a08b73c498869789e38677181a8d.patch"; - sha256 = "07ifsis9jlc0jjp2d11f7hvw9kaxcbk0a57h2m4xsv1d7vzl9yfh"; + hash = "sha256-0PlE/z4tbN1JFfAUBeZiXc3ENzwuhCaulIBRmXTULh4="; }) # Fixes replaced/obsolete Qt methods (fetchpatch { - url = "https://github.com/sigrokproject/pulseview/commit/ae726b70a7ada9a4be5808e00f0c951318479684.patch"; - sha256 = "sha256-6bFXFAnTO+MBUmslw55gWWSCCPwnejqKGpHeJOoH0e8="; - }) + url = "https://github.com/sigrokproject/pulseview/commit/ae726b70a7ada9a4be5808e00f0c951318479684.patch"; + hash = "sha256-6bFXFAnTO+MBUmslw55gWWSCCPwnejqKGpHeJOoH0e8="; + }) ]; meta = with lib; { From 11b65fea772deecb4e6288e8a0aad922aa3a2b67 Mon Sep 17 00:00:00 2001 From: Nikolay Korotkiy Date: Sun, 30 Apr 2023 14:18:12 +0400 Subject: [PATCH 2/2] pulseview: enable on darwin --- .../science/electronics/pulseview/default.nix | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/pkgs/applications/science/electronics/pulseview/default.nix b/pkgs/applications/science/electronics/pulseview/default.nix index 8906076ec103..7472fa30e7ff 100644 --- a/pkgs/applications/science/electronics/pulseview/default.nix +++ b/pkgs/applications/science/electronics/pulseview/default.nix @@ -1,6 +1,6 @@ { lib, stdenv, fetchurl, fetchpatch, pkg-config, cmake, glib, boost, libsigrok , libsigrokdecode, libserialport, libzip, udev, libusb1, libftdi1, glibmm -, pcre, python3, qtsvg, qttools, wrapQtAppsHook +, pcre, python3, qtsvg, qttools, wrapQtAppsHook, desktopToDarwinBundle }: stdenv.mkDerivation rec { @@ -12,13 +12,14 @@ stdenv.mkDerivation rec { hash = "sha256-8EL3ej4bNb8wZmMw427Dj6uNJIw2k8N7fjXUAcO/q8s="; }; - nativeBuildInputs = [ cmake pkg-config qttools wrapQtAppsHook ]; + nativeBuildInputs = [ cmake pkg-config qttools wrapQtAppsHook ] + ++ lib.optional stdenv.isDarwin desktopToDarwinBundle; buildInputs = [ - glib boost libsigrok libsigrokdecode libserialport libzip udev libusb1 libftdi1 glibmm + glib boost libsigrok libsigrokdecode libserialport libzip libusb1 libftdi1 glibmm pcre python3 qtsvg - ]; + ] ++ lib.optional stdenv.isLinux udev; patches = [ # Allow building with glib 2.68 @@ -39,6 +40,6 @@ stdenv.mkDerivation rec { homepage = "https://sigrok.org/"; license = licenses.gpl3Plus; maintainers = with maintainers; [ bjornfor ]; - platforms = platforms.linux; + platforms = platforms.unix; }; }