From 7f2052d6ff469d18ffcc46d087372a7e2d20975f Mon Sep 17 00:00:00 2001
From: Phil Dyer <phildyer@protonmail.com>
Date: Fri, 10 May 2024 17:38:15 +1000
Subject: [PATCH] ryzen-monitor-ng: Fix build

makeTargets is not used by the default builder, and was not
calling the necessary make targets.

Added `make clean` to `preBuild` to ensure old files are removed
before building.
---
 pkgs/by-name/ry/ryzen-monitor-ng/package.nix | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/pkgs/by-name/ry/ryzen-monitor-ng/package.nix b/pkgs/by-name/ry/ryzen-monitor-ng/package.nix
index 5bcb65be3622..570cc9ca3cf4 100644
--- a/pkgs/by-name/ry/ryzen-monitor-ng/package.nix
+++ b/pkgs/by-name/ry/ryzen-monitor-ng/package.nix
@@ -14,13 +14,14 @@ stdenv.mkDerivation {
     hash = "sha256-fcW2fEsCFliRnMFnboR0jchzVIlCYbr2AE6AS06cb6o=";
   };
 
-  ## Remove binaries committed into upstream repo
+  # Upstream repo contains pre-compiled binaries and object files
+  # that are out of date.
+  # These need to be removed before build stage.
   preBuild = ''
     rm src/ryzen_monitor
+    make clean
   '';
 
-  makeTargets = [ "clean" "install" ];
-
   installPhase = ''
     runHook preInstall