From f3f68fcfac50ff6ee73152e32848a4a6f2acd021 Mon Sep 17 00:00:00 2001 From: Qyriad Date: Thu, 23 May 2024 20:22:15 -0600 Subject: nix3-upgrade-nix: fix when there are differing pnames Change-Id: I19c7e24a4d46137127e76b7bb133e0184d73d1b6 --- src/nix/upgrade-nix.cc | 18 +++++++++++++++++- 1 file changed, 17 insertions(+), 1 deletion(-) diff --git a/src/nix/upgrade-nix.cc b/src/nix/upgrade-nix.cc index 55f833833..3e47fc8f0 100644 --- a/src/nix/upgrade-nix.cc +++ b/src/nix/upgrade-nix.cc @@ -111,7 +111,23 @@ struct CmdUpgradeNix : MixDryRun, EvalCommand if (pathExists(canonProfileDir + "/manifest.nix")) { - std::string nixEnvCmd = settings.nixBinDir + "/nix-env"; + // {settings.nixBinDir}/nix-env is a symlink to a {settings.nixBinDir}/nix, which *then* + // is a symlink to /nix/store/meow-nix/bin/nix. We want /nix/store/meow-nix/bin/nix-env. + Path const nixInStore = canonPath(settings.nixBinDir + "/nix-env", true); + Path const nixEnvCmd = dirOf(nixInStore) + "/nix-env"; + + // First remove the existing Nix, then use that Nix by absolute path to + // install the new one, in case the new and old versions aren't considered + // to be "the same package" by nix-env's logic (e.g., if their pnames differ). + Strings removeArgs = { + "--uninstall", + nixEnvCmd, + "--profile", + this->profileDir, + }; + printTalkative("running %s %s", nixEnvCmd, concatStringsSep(" ", removeArgs)); + runProgram(nixEnvCmd, false, removeArgs); + Strings upgradeArgs = { "--profile", this->profileDir, -- cgit v1.2.3