NixOS · Apr 2, 2020 · Apr 2, 2020
Showing with 9 additions and 19 deletions.

+1 −13 src/libexpr/flake/lockfile.cc

+0 −6 src/libfetchers/fetchers.cc

+5 −0 src/libfetchers/tree-info.hh

+2 −0 src/nix-env/user-env.cc

+1 −0 src/nix/flake.cc
diff --git a/src/libexpr/flake/lockfile.cc b/src/libexpr/flake/lockfile.cc
@@ -76,18 +76,6 @@ LockedNode::LockedNode(const nlohmann::json & json)
         throw Error("lockfile contains mutable flakeref '%s'", lockedRef);
 }
 
-static nlohmann::json treeInfoToJson(const TreeInfo & info)
-{
-    nlohmann::json json;
-    assert(info.narHash);
-    json["narHash"] = info.narHash.to_string(SRI);
-    if (info.revCount)
-        json["revCount"] = *info.revCount;
-    if (info.lastModified)
-        json["lastModified"] = *info.lastModified;
-    return json;
-}
-
 StorePath LockedNode::computeStorePath(Store & store) const
 {
     return info.computeStorePath(store);
@@ -193,7 +181,7 @@ nlohmann::json LockFile::toJson() const
         if (auto lockedNode = std::dynamic_pointer_cast<const LockedNode>(node)) {
             n["original"] = fetchers::attrsToJson(lockedNode->originalRef.toAttrs());
             n["locked"] = fetchers::attrsToJson(lockedNode->lockedRef.toAttrs());
-            n["info"] = treeInfoToJson(lockedNode->info);
+            n["info"] = lockedNode->info.toJson();
             if (!lockedNode->isFlake) n["flake"] = false;
         }
 

diff --git a/src/libfetchers/fetchers.cc b/src/libfetchers/fetchers.cc
@@ -81,10 +81,4 @@ std::shared_ptr<const Input> Input::applyOverrides(
     return shared_from_this();
 }
 
-StorePath TreeInfo::computeStorePath(Store & store) const
-{
-    assert(narHash);
-    return store.makeFixedOutputPath(true, narHash, "source");
-}
-
 }
diff --git a/src/libfetchers/tree-info.hh b/src/libfetchers/tree-info.hh
@@ -1,6 +1,9 @@
 #pragma once
 
 #include "path.hh"
+#include "hash.hh"
+
+#include <nlohmann/json_fwd.hpp>
 
 namespace nix { class Store; }
 
@@ -21,6 +24,8 @@ struct TreeInfo
     }
 
     StorePath computeStorePath(Store & store) const;
+
+    nlohmann::json toJson() const;
 };
 
 }
diff --git a/src/nix-env/user-env.cc b/src/nix-env/user-env.cc
@@ -15,6 +15,8 @@ namespace nix {
 DrvInfos queryInstalled(EvalState & state, const Path & userEnv)
 {
     DrvInfos elems;
+    if (pathExists(userEnv + "/manifest.json"))
+        throw Error("profile '%s' is incompatible with 'nix-env'; please use 'nix profile' instead", userEnv);
     Path manifestFile = userEnv + "/manifest.nix";
     if (pathExists(manifestFile)) {
         Value v;

diff --git a/src/nix/flake.cc b/src/nix/flake.cc
@@ -103,6 +103,7 @@ static nlohmann::json flakeToJson(const Store & store, const Flake & flake)
     j["url"] = flake.lockedRef.to_string();
     j["original"] = attrsToJson(flake.originalRef.toAttrs());
     j["locked"] = attrsToJson(flake.lockedRef.toAttrs());
+    j["info"] = flake.sourceInfo->info.toJson();
     if (auto rev = flake.lockedRef.input->getRev())
         j["revision"] = rev->to_string(Base16, false);
     if (flake.sourceInfo->info.revCount)