From 640a1d523e6d0db3797e76e72ba5a3e8dd784f9f Mon Sep 17 00:00:00 2001
From: Yiyu Zhou <yiyu@yiyuzhou.io>
Date: Sun, 25 Jan 2026 13:36:53 -0800
Subject: [PATCH] ly: cleanup

---
 pkgs/by-name/ly/ly/package.nix | 38 +++++++++++++++++++++++-----------
 1 file changed, 26 insertions(+), 12 deletions(-)

diff --git a/pkgs/by-name/ly/ly/package.nix b/pkgs/by-name/ly/ly/package.nix
index ba447952895b..76f57db43fc7 100644
--- a/pkgs/by-name/ly/ly/package.nix
+++ b/pkgs/by-name/ly/ly/package.nix
@@ -1,16 +1,20 @@
 {
-  stdenv,
-  lib,
-  fetchFromGitea,
-  linux-pam,
-  libxcb,
-  makeBinaryWrapper,
-  zig_0_15,
   callPackage,
+  fetchFromGitea,
+  lib,
+  libxcb,
+  linux-pam,
+  makeBinaryWrapper,
   nixosTests,
+  stdenv,
+  versionCheckHook,
   x11Support ? true,
+  zig_0_15,
 }:
 
+let
+  zig = zig_0_15;
+in
 stdenv.mkDerivation (finalAttrs: {
   pname = "ly";
   version = "1.3.1";
@@ -25,8 +29,9 @@ stdenv.mkDerivation (finalAttrs: {
 
   nativeBuildInputs = [
     makeBinaryWrapper
-    zig_0_15
+    zig
   ];
+
   buildInputs = [
     linux-pam
   ]
@@ -35,22 +40,31 @@ stdenv.mkDerivation (finalAttrs: {
   postPatch = ''
     ln -s ${
       callPackage ./deps.nix {
-        zig = zig_0_15;
+        inherit zig;
       }
     } $ZIG_GLOBAL_CACHE_DIR/p
   '';
+
   zigBuildFlags = [
     "-Denable_x11_support=${lib.boolToString x11Support}"
   ];
 
+  doInstallCheck = true;
+  nativeInstallCheckInputs = [ versionCheckHook ];
+
   passthru.tests = { inherit (nixosTests) ly; };
 
   meta = {
     description = "TUI display manager";
-    license = lib.licenses.wtfpl;
+    longDescription = ''
+      Ly is a lightweight TUI (ncurses-like) display manager for Linux
+      and BSD, designed with portability in mind (e.g. it does not
+      require systemd to run).
+    '';
     homepage = "https://codeberg.org/fairyglade/ly";
-    maintainers = [ ];
-    platforms = lib.platforms.linux;
+    license = lib.licenses.wtfpl;
     mainProgram = "ly";
+    maintainers = [ ];
+    platforms = lib.platforms.unix;
   };
 })