4 files changed, 6 insertions, 4 deletions
diff --git a/src/libstore/crypto.cc b/src/libstore/crypto.cc
index 53e94e1f5..caba22c1e 100644
--- a/src/libstore/crypto.cc
+++ b/src/libstore/crypto.cc
@@ -37,10 +37,12 @@ SecretKey::SecretKey(const string & s)
 #endif
 }
 
+#if !HAVE_SODIUM
 [[noreturn]] static void noSodium()
 {
     throw Error("Nix was not compiled with libsodium, required for signed binary cache support");
 }
+#endif
 
 std::string SecretKey::signDetached(const std::string & data) const
 {
diff --git a/src/libstore/crypto.hh b/src/libstore/crypto.hh
index 33b79cb2e..38d5fe2a8 100644
--- a/src/libstore/crypto.hh
+++ b/src/libstore/crypto.hh
@@ -39,7 +39,7 @@ struct PublicKey : Key
 private:
     PublicKey(const std::string & name, const std::string & key)
         : Key(name, key) { }
-    friend class SecretKey;
+    friend struct SecretKey;
 };
 
 typedef std::map<std::string, PublicKey> PublicKeys;
diff --git a/src/libstore/http-binary-cache-store.cc b/src/libstore/http-binary-cache-store.cc
index 9614d0b4c..1d707c043 100644
--- a/src/libstore/http-binary-cache-store.cc
+++ b/src/libstore/http-binary-cache-store.cc
@@ -48,7 +48,7 @@ protected:
         }
     }
 
-    void upsertFile(const std::string & path, const std::string & data)
+    void upsertFile(const std::string & path, const std::string & data) override
     {
         throw Error("uploading to an HTTP binary cache is not supported");
     }
diff --git a/src/libstore/nar-accessor.cc b/src/libstore/nar-accessor.cc
index ff7890af8..8896862be 100644
--- a/src/libstore/nar-accessor.cc
+++ b/src/libstore/nar-accessor.cc
@@ -33,7 +33,7 @@ struct NarIndexer : ParseSink, StringSource
     {
     }
 
-    void createDirectory(const Path & path)
+    void createDirectory(const Path & path) override
     {
         members.emplace(path,
             NarMember{FSAccessor::Type::tDirectory, false, 0, 0});
@@ -44,7 +44,7 @@ struct NarIndexer : ParseSink, StringSource
         currentPath = path;
     }
 
-    void isExecutable()
+    void isExecutable() override
     {
         isExec = true;
     }