From 4dbbd721eb9db75d4968a624b8cb9e75e979a144 Mon Sep 17 00:00:00 2001
From: Alois Wohlschlager <alois1@gmx-topmail.de>
Date: Mon, 2 Sep 2024 20:09:35 +0200
Subject: treewide: consistently mark overridden settings as such

Only overridden settings are sent to the daemon, and we're going to do the same
for the build hook to. It needs to be ensured that overridden settings are in
fact consistently marked as such, so that they actually get sent.

Change-Id: I7cd58d925702f86cf2c35ad121eb191ceb62a355
---
 src/nix/main.cc | 34 +++++++++++++++-------------------
 1 file changed, 15 insertions(+), 19 deletions(-)

(limited to 'src/nix/main.cc')

diff --git a/src/nix/main.cc b/src/nix/main.cc
index fdd3ac2ae..cf1f876dd 100644
--- a/src/nix/main.cc
+++ b/src/nix/main.cc
@@ -247,8 +247,8 @@ static void showHelp(std::vector<std::string> subcommand, NixArgs & toplevel)
 {
     auto mdName = subcommand.empty() ? "nix" : fmt("nix3-%s", concatStringsSep("-", subcommand));
 
-    evalSettings.restrictEval = false;
-    evalSettings.pureEval = false;
+    evalSettings.restrictEval.override(false);
+    evalSettings.pureEval.override(false);
     EvalState state({}, openStore("dummy://"));
 
     auto vGenerateManpage = state.allocValue();
@@ -389,7 +389,7 @@ void mainWrapped(int argc, char * * argv)
         if (legacy) return legacy(argc, argv);
     }
 
-    evalSettings.pureEval = true;
+    evalSettings.pureEval.setDefault(true);
 
     setLogFormat(LogFormat::bar);
     settings.verboseBuild = false;
@@ -408,11 +408,11 @@ void mainWrapped(int argc, char * * argv)
     }
 
     if (argc == 2 && std::string(argv[1]) == "__dump-language") {
-        experimentalFeatureSettings.experimentalFeatures = ExperimentalFeatures{}
+        experimentalFeatureSettings.experimentalFeatures.override(ExperimentalFeatures{}
             | Xp::Flakes
             | Xp::FetchClosure
-            | Xp::DynamicDerivations;
-        evalSettings.pureEval = false;
+            | Xp::DynamicDerivations);
+        evalSettings.pureEval.override(false);
         EvalState state({}, openStore("dummy://"));
         auto res = nlohmann::json::object();
         res["builtins"] = ({
@@ -513,24 +513,20 @@ void mainWrapped(int argc, char * * argv)
 
     if (!args.useNet) {
         // FIXME: should check for command line overrides only.
-        if (!settings.useSubstitutes.overridden)
-            settings.useSubstitutes = false;
-        if (!settings.tarballTtl.overridden)
-            settings.tarballTtl = std::numeric_limits<unsigned int>::max();
-        if (!fileTransferSettings.tries.overridden)
-            fileTransferSettings.tries = 0;
-        if (!fileTransferSettings.connectTimeout.overridden)
-            fileTransferSettings.connectTimeout = 1;
+        settings.useSubstitutes.setDefault(false);
+        settings.tarballTtl.setDefault(std::numeric_limits<unsigned int>::max());
+        fileTransferSettings.tries.setDefault(0);
+        fileTransferSettings.connectTimeout.setDefault(1);
     }
 
     if (args.refresh) {
-        settings.tarballTtl = 0;
-        settings.ttlNegativeNarInfoCache = 0;
-        settings.ttlPositiveNarInfoCache = 0;
+        settings.tarballTtl.override(0);
+        settings.ttlNegativeNarInfoCache.override(0);
+        settings.ttlPositiveNarInfoCache.override(0);
     }
 
-    if (args.command->second->forceImpureByDefault() && !evalSettings.pureEval.overridden) {
-        evalSettings.pureEval = false;
+    if (args.command->second->forceImpureByDefault()) {
+        evalSettings.pureEval.setDefault(false);
     }
     args.command->second->run();
 }
-- 
cgit v1.2.3