From 0f7e9d051340531332b7c6854675006cc8a16f50 Mon Sep 17 00:00:00 2001
From: Eelco Dolstra <edolstra@gmail.com>
Date: Fri, 20 Oct 2023 19:13:35 +0200
Subject: [PATCH] Input: Remove 'direct' field

---
 src/libfetchers/fetchers.cc | 5 +++++
 src/libfetchers/fetchers.hh | 6 ++++--
 src/libfetchers/indirect.cc | 5 +++--
 3 files changed, 12 insertions(+), 4 deletions(-)

diff --git a/src/libfetchers/fetchers.cc b/src/libfetchers/fetchers.cc
index 000609f09a8..5f616786800 100644
--- a/src/libfetchers/fetchers.cc
+++ b/src/libfetchers/fetchers.cc
@@ -84,6 +84,11 @@ std::string Input::to_string() const
     return toURL().to_string();
 }
 
+bool Input::isDirect() const
+{
+    return !scheme || scheme->isDirect(*this);
+}
+
 Attrs Input::toAttrs() const
 {
     return attrs;
diff --git a/src/libfetchers/fetchers.hh b/src/libfetchers/fetchers.hh
index f52175e4134..4119dd17be0 100644
--- a/src/libfetchers/fetchers.hh
+++ b/src/libfetchers/fetchers.hh
@@ -35,7 +35,6 @@ struct Input
     std::shared_ptr<InputScheme> scheme; // note: can be null
     Attrs attrs;
     bool locked = false;
-    bool direct = true;
 
     /**
      * path of the parent of this input, used for relative path resolution
@@ -71,7 +70,7 @@ public:
      * Check whether this is a "direct" input, that is, not
      * one that goes through a registry.
      */
-    bool isDirect() const { return direct; }
+    bool isDirect() const;
 
     /**
      * Check whether this is a "locked" input, that is,
@@ -152,6 +151,9 @@ struct InputScheme
      * Is this `InputScheme` part of an experimental feature?
      */
     virtual std::optional<ExperimentalFeature> experimentalFeature();
+
+    virtual bool isDirect(const Input & input) const
+    { return true; }
 };
 
 void registerInputScheme(std::shared_ptr<InputScheme> && fetcher);
diff --git a/src/libfetchers/indirect.cc b/src/libfetchers/indirect.cc
index 947849802ad..9a71df3d4a5 100644
--- a/src/libfetchers/indirect.cc
+++ b/src/libfetchers/indirect.cc
@@ -41,7 +41,6 @@ struct IndirectInputScheme : InputScheme
         // FIXME: forbid query params?
 
         Input input;
-        input.direct = false;
         input.attrs.insert_or_assign("type", "indirect");
         input.attrs.insert_or_assign("id", id);
         if (rev) input.attrs.insert_or_assign("rev", rev->gitRev());
@@ -63,7 +62,6 @@ struct IndirectInputScheme : InputScheme
             throw BadURL("'%s' is not a valid flake ID", id);
 
         Input input;
-        input.direct = false;
         input.attrs = attrs;
         return input;
     }
@@ -98,6 +96,9 @@ struct IndirectInputScheme : InputScheme
     {
         return Xp::Flakes;
     }
+
+    bool isDirect(const Input & input) const override
+    { return false; }
 };
 
 static auto rIndirectInputScheme = OnStartup([] { registerInputScheme(std::make_unique<IndirectInputScheme>()); });