From 32740242f3375ca62af92aa6598868f756cfcbea Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Sandro=20J=C3=A4ckel?= <sandro.jaeckel@gmail.com>
Date: Thu, 29 Feb 2024 19:33:23 +0100
Subject: [PATCH] plasma6: fix meta.position

Fixes nix edit kdePackages.plasma-browser-integration
---
 pkgs/kde/lib/mk-kde-derivation.nix | 27 +++++++++++++--------------
 1 file changed, 13 insertions(+), 14 deletions(-)

diff --git a/pkgs/kde/lib/mk-kde-derivation.nix b/pkgs/kde/lib/mk-kde-derivation.nix
index ddada8fb77f0..80508eebb7d4 100644
--- a/pkgs/kde/lib/mk-kde-derivation.nix
+++ b/pkgs/kde/lib/mk-kde-derivation.nix
@@ -78,7 +78,6 @@ in
     extraNativeBuildInputs ? [],
     extraPropagatedBuildInputs ? [],
     extraCmakeFlags ? [],
-    meta ? {},
     ...
   } @ args: let
     # FIXME(later): this is wrong for cross, some of these things really need to go into nativeBuildInputs,
@@ -101,18 +100,6 @@ in
       cmakeFlags = ["-DQT_MAJOR_VERSION=6"] ++ extraCmakeFlags;
 
       separateDebugInfo = true;
-
-      meta =
-        {
-          description = projectInfo.${pname}.description;
-          homepage = "https://invent.kde.org/${projectInfo.${pname}.repo_path}";
-          license = lib.filter (l: l != null) (map (l: licensesBySpdxId.${l}) licenseInfo.${pname});
-          maintainers = lib.teams.qt-kde.members;
-          # Platforms are currently limited to what upstream tests in CI, but can be extended if
-          # there's interest.
-          platforms = lib.platforms.linux ++ lib.platforms.freebsd;
-        }
-        // meta;
     };
 
     cleanArgs = builtins.removeAttrs args [
@@ -122,5 +109,17 @@ in
       "extraCmakeFlags"
       "meta"
     ];
+
+    meta = let
+      pos = builtins.unsafeGetAttrPos "pname" args;
+    in {
+      description = projectInfo.${pname}.description;
+      homepage = "https://invent.kde.org/${projectInfo.${pname}.repo_path}";
+      license = lib.filter (l: l != null) (map (l: licensesBySpdxId.${l}) licenseInfo.${pname});
+      maintainers = lib.teams.qt-kde.members;
+      # Platforms are currently limited to what upstream tests in CI, but can be extended if there's interest.
+      platforms = lib.platforms.linux ++ lib.platforms.freebsd;
+      position = "${pos.file}:${toString pos.line}";
+    } // (args.meta or { });
   in
-    stdenv.mkDerivation (defaultArgs // cleanArgs)
+    stdenv.mkDerivation (defaultArgs // cleanArgs) // { inherit meta; }