From 561721bbb875d2944290113d723007616c8de1a3 Mon Sep 17 00:00:00 2001 From: Jiajie Chen Date: Fri, 8 Jul 2022 11:13:12 +0800 Subject: [PATCH 1/4] hdfview: copy missing libapplauncher.so This fixes issue #179793. --- pkgs/tools/misc/hdfview/default.nix | 1 + 1 file changed, 1 insertion(+) diff --git a/pkgs/tools/misc/hdfview/default.nix b/pkgs/tools/misc/hdfview/default.nix index 94acd6dcd947..682b4a7b356b 100644 --- a/pkgs/tools/misc/hdfview/default.nix +++ b/pkgs/tools/misc/hdfview/default.nix @@ -41,6 +41,7 @@ stdenv.mkDerivation rec { mkdir -p $out/bin $out/lib cp -a build/dist/HDFView/bin/HDFView $out/bin/ cp -a build/dist/HDFView/lib/app $out/lib/ + cp -a build/dist/HDFView/lib/libapplauncher.so $out/lib/ ln -s ${jdk}/lib/openjdk $out/lib/runtime mkdir -p $out/share/applications $out/share/icons/hicolor/32x32/apps From f4e3995bf67191548bd0763c89a1ad8fa5c937af Mon Sep 17 00:00:00 2001 From: Jiajie Chen Date: Mon, 11 Jul 2022 08:36:40 +0800 Subject: [PATCH 2/4] hdfview: 3.1.3 -> 3.1.4 Set machine.arch for aarch64 machines. --- pkgs/tools/misc/hdfview/default.nix | 23 ++++++++++++++--------- 1 file changed, 14 insertions(+), 9 deletions(-) diff --git a/pkgs/tools/misc/hdfview/default.nix b/pkgs/tools/misc/hdfview/default.nix index 682b4a7b356b..9a2ee1f0b66d 100644 --- a/pkgs/tools/misc/hdfview/default.nix +++ b/pkgs/tools/misc/hdfview/default.nix @@ -2,29 +2,34 @@ stdenv.mkDerivation rec { pname = "hdfview"; - version = "3.1.3"; + version = "3.1.4"; src = fetchurl { url = "https://support.hdfgroup.org/ftp/HDF5/releases/HDF-JAVA/${pname}-${version}/src/${pname}-${version}.tar.gz"; - sha256 = "sha256-VmgHSVMFoy09plU5pSnyaPz8N15toy7QfCtXI7mqDGY="; + sha256 = "sha256-iY/NUifU57aX795eWpaUBflrclF/nfvb3OKZEpD9VqA="; }; nativeBuildInputs = [ - ant jdk - nettools # "hostname" required + ant + jdk + nettools # "hostname" required copyDesktopItems ]; HDFLIBS = (hdf4.override { javaSupport = true; }).out; HDF5LIBS = (hdf5.override { javaSupport = true; }).out; - buildPhase = '' - runHook preBuild + buildPhase = + let + arch = if stdenv.isx86_64 then "x86_64" else "aarch64"; + in + '' + runHook preBuild - ant createJPackage + ant createJPackage -Dmachine.arch=${arch} - runHook postBuild - ''; + runHook postBuild + ''; desktopItem = makeDesktopItem rec { name = "HDFView"; From 433b6710665e99befaa61ce3ab089cd6e99101df Mon Sep 17 00:00:00 2001 From: Jiajie Chen Date: Tue, 12 Jul 2022 10:13:32 +0800 Subject: [PATCH 3/4] hdfview: add jiegec to maintainers Add jiegec to maintainer list of hdfview. --- pkgs/tools/misc/hdfview/default.nix | 1 + 1 file changed, 1 insertion(+) diff --git a/pkgs/tools/misc/hdfview/default.nix b/pkgs/tools/misc/hdfview/default.nix index 9a2ee1f0b66d..23e2a72557f6 100644 --- a/pkgs/tools/misc/hdfview/default.nix +++ b/pkgs/tools/misc/hdfview/default.nix @@ -63,5 +63,6 @@ stdenv.mkDerivation rec { license = lib.licenses.free; # BSD-like homepage = "https://portal.hdfgroup.org/display/HDFVIEW/HDFView"; platforms = lib.platforms.linux ++ lib.platforms.darwin; + maintainers = with lib.maintainers; [ jiegec ]; }; } From 73ba71eae6d90006fc90d17406fcec6cd52a8def Mon Sep 17 00:00:00 2001 From: Jiajie Chen Date: Fri, 15 Jul 2022 10:22:02 +0800 Subject: [PATCH 4/4] hdfview: avoid hostname dependency Signed-off-by: Jiajie Chen --- ...u-false-to-avoid-hostname-dependency.patch | 38 +++++++++++++++++++ pkgs/tools/misc/hdfview/default.nix | 6 ++- 2 files changed, 43 insertions(+), 1 deletion(-) create mode 100644 pkgs/tools/misc/hdfview/0001-Hardcode-isUbuntu-false-to-avoid-hostname-dependency.patch diff --git a/pkgs/tools/misc/hdfview/0001-Hardcode-isUbuntu-false-to-avoid-hostname-dependency.patch b/pkgs/tools/misc/hdfview/0001-Hardcode-isUbuntu-false-to-avoid-hostname-dependency.patch new file mode 100644 index 000000000000..fde26e0dca83 --- /dev/null +++ b/pkgs/tools/misc/hdfview/0001-Hardcode-isUbuntu-false-to-avoid-hostname-dependency.patch @@ -0,0 +1,38 @@ +From e5eb394458e19ce8f8a231e8b2005c80c64fd426 Mon Sep 17 00:00:00 2001 +From: Jiajie Chen +Date: Fri, 15 Jul 2022 10:13:23 +0800 +Subject: [PATCH] Hardcode isUbuntu=false to avoid hostname dependency. + +The original build.xml detects whether the system is ubuntu based on its +hostname, which is useless in nixpkgs and brings additional dependency. + +As suggested by @risicle in #180613, we can simply hardcode isUbuntu to +false. + +Signed-off-by: Jiajie Chen +--- + build.xml | 8 +++----- + 1 file changed, 3 insertions(+), 5 deletions(-) + +diff --git a/build.xml b/build.xml +index c4f0974..15a6078 100644 +--- a/build.xml ++++ b/build.xml +@@ -143,11 +143,9 @@ + + + +- +- +- +- +- ++ ++ + +