From f7341940275e24857369bf468923c60073defc99 Mon Sep 17 00:00:00 2001
From: AndersonTorres <torres.anderson.85@protonmail.com>
Date: Mon, 11 Apr 2022 23:18:29 -0300
Subject: [PATCH] pkgconf: refactor

---
 .../tools/misc/pkgconf/default.nix            | 45 ++++++++++++-------
 pkgs/top-level/all-packages.nix               |  2 +-
 2 files changed, 30 insertions(+), 17 deletions(-)

diff --git a/pkgs/development/tools/misc/pkgconf/default.nix b/pkgs/development/tools/misc/pkgconf/default.nix
index e3e0de4e7f16..8db8a709bb6e 100644
--- a/pkgs/development/tools/misc/pkgconf/default.nix
+++ b/pkgs/development/tools/misc/pkgconf/default.nix
@@ -1,31 +1,35 @@
-{ lib, stdenv, fetchurl, removeReferencesTo }:
+{ lib
+, stdenv
+, fetchurl
+, removeReferencesTo
+}:
 
 stdenv.mkDerivation rec {
   pname = "pkgconf";
   version = "1.8.0";
 
-  nativeBuildInputs = [ removeReferencesTo ];
+  src = fetchurl {
+    url = "https://distfiles.dereferenced.org/${pname}/${pname}-${version}.tar.xz";
+    hash = "sha256-75x+YYIrfLg1bm6eHcpY2VVvMgDXisqzXkNH6dTCu68=";
+  };
 
   outputs = [ "out" "lib" "dev" "man" "doc" ];
 
+  nativeBuildInputs = [ removeReferencesTo ];
+
   enableParallelBuilding = true;
 
-  src = fetchurl {
-    url = "https://distfiles.dereferenced.org/${pname}/${pname}-${version}.tar.xz";
-    sha256 = "sha256-75x+YYIrfLg1bm6eHcpY2VVvMgDXisqzXkNH6dTCu68=";
-  };
-
-  # Debian has outputs like these too:
-  # https://packages.debian.org/source/buster/pkgconf, so take it this
-  # reference removing is safe.
+  # Debian has outputs like these too
+  # (https://packages.debian.org/source/bullseye/pkgconf), so it is safe to
+  # remove those references
   postFixup = ''
+    remove-references-to \
+      -t "${placeholder "out"}" \
+      "${placeholder "lib"}"/lib/*
     remove-references-to \
       -t "${placeholder "dev"}" \
       "${placeholder "lib"}"/lib/* \
       "${placeholder "out"}"/bin/*
-    remove-references-to \
-      -t "${placeholder "out"}" \
-      "${placeholder "lib"}"/lib/*
   ''
   # Move back share/aclocal. Yes, this normally goes in the dev output for good
   # reason, but in this case the dev output is for the `libpkgconf` library,
@@ -37,10 +41,19 @@ stdenv.mkDerivation rec {
   '';
 
   meta = with lib; {
+    homepage = "https://github.com/pkgconf/pkgconf";
     description = "Package compiler and linker metadata toolkit";
-    homepage = "https://git.dereferenced.org/pkgconf/pkgconf";
-    platforms = platforms.all;
+    longDescription = ''
+      pkgconf is a program which helps to configure compiler and linker flags
+      for development libraries. It is similar to pkg-config from
+      freedesktop.org.
+
+      libpkgconf is a library which provides access to most of pkgconf's
+      functionality, to allow other tooling such as compilers and IDEs to
+      discover and use libraries configured by pkgconf.
+    '';
     license = licenses.isc;
-    maintainers = with maintainers; [ zaninime ];
+    maintainers = with maintainers; [ zaninime AndersonTorres ];
+    platforms = platforms.all;
   };
 }
diff --git a/pkgs/top-level/all-packages.nix b/pkgs/top-level/all-packages.nix
index a72406d6f1a1..da6285295488 100644
--- a/pkgs/top-level/all-packages.nix
+++ b/pkgs/top-level/all-packages.nix
@@ -15540,7 +15540,7 @@ with pkgs;
 
   pmccabe = callPackage ../development/tools/misc/pmccabe { };
 
-  pkgconf-unwrapped = callPackage ../development/tools/misc/pkgconf {};
+  pkgconf-unwrapped = callPackage ../development/tools/misc/pkgconf { };
   pkgconf = callPackage ../build-support/pkg-config-wrapper {
     pkg-config = pkgconf-unwrapped;
     baseBinName = "pkgconf";