add a print statement for drv path -> builder name

Author: DieracDelta

src/libstore/build/derivation-building-goal.cc

@@ -263,23 +263,33 @@ Goal::Co DerivationBuildingGoal::tryToBuild()
                 : buildMode == bmCheck ? "checking outputs of '%s'"
                                        : "building '%s'",
                 worker.store.printStorePath(drvPath));
+
+        std::string builderDescription;
 #ifndef _WIN32 // TODO enable build hook on Windows
-        if (hook)
-            msg += fmt(" on '%s'", hook->machineName);
+        if (hook) {
+            builderDescription = fmt("remote builder: %s", hook->machineName);
+        } else if (builder) {
+            auto maybeUID = builder->getBuilderUID();
+            if (maybeUID)
+                builderDescription = fmt("localhost builder with uid: %s", std::to_string(*maybeUID));
+            else
+                builderDescription = "";
+        } else {
+            builderDescription = "";
+        }
+#else
+        builderDescription = "";
 #endif
+        if (builderDescription != "")
+            msg += fmt(" on '%s'", builderDescription);
+
         act = std::make_unique<Activity>(
             *logger,
             lvlInfo,
             actBuild,
             msg,
-            Logger::Fields{
-                worker.store.printStorePath(drvPath),
-#ifndef _WIN32 // TODO enable build hook on Windows
-                hook ? hook->machineName :
-#endif
-                     "",
-                1,
-                1});
+            Logger::Fields{worker.store.printStorePath(drvPath), builderDescription, 1, 1});
+
         mcRunningBuilds = std::make_unique<MaintainCount<uint64_t>>(worker.runningBuilds);
         worker.updateProgress();
     };

src/libstore/include/nix/store/build/derivation-builder.hh

@@ -180,6 +180,13 @@ struct DerivationBuilder : RestrictionContext
      * killed.
      */
     virtual bool killChild() = 0;
+
+    /**
+     * Get the builder UID.
+     *
+     * @returns the builder UID if run on localhost (e.g. if it exists).
+     */
+    virtual std::optional<uid_t> getBuilderUID() const = 0;
 };
 
 struct ExternalBuilder

src/libstore/unix/build/derivation-builder.cc

@@ -415,6 +415,8 @@ class DerivationBuilderImpl : public DerivationBuilder, public DerivationBuilder
 
     bool killChild() override;
 
+    std::optional<uid_t> getBuilderUID() const override;
+
 private:
 
     bool decideWhetherDiskFull();
@@ -494,6 +496,13 @@ bool DerivationBuilderImpl::killChild()
     return ret;
 }
 
+std::optional<uid_t> DerivationBuilderImpl::getBuilderUID() const
+{
+    if (buildUser)
+        return buildUser->getUID();
+    return std::nullopt;
+}
+
 SingleDrvOutputs DerivationBuilderImpl::unprepareBuild()
 {
     /* Since we got an EOF on the logger pipe, the builder is presumed