From 8ffeb1c4e58e8ed0032b097d3d1d9625726f5fec Mon Sep 17 00:00:00 2001
From: Ben Radford <benradf@users.noreply.github.com>
Date: Thu, 18 May 2023 13:47:31 +0100
Subject: [PATCH] Throw error instead of silently skipping CA migration.

---
 src/libstore/local-store.cc | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/src/libstore/local-store.cc b/src/libstore/local-store.cc
index 99fb9f434..c25aa0e72 100644
--- a/src/libstore/local-store.cc
+++ b/src/libstore/local-store.cc
@@ -355,8 +355,12 @@ LocalStore::LocalStore(const Params & params)
 
     else openDB(*state, false);
 
-    if (!readOnly && experimentalFeatureSettings.isEnabled(Xp::CaDerivations)) {
-        migrateCASchema(state->db, dbDir + "/ca-schema", globalLock);
+    if (experimentalFeatureSettings.isEnabled(Xp::CaDerivations)) {
+        if (!readOnly) {
+            migrateCASchema(state->db, dbDir + "/ca-schema", globalLock);
+        } else {
+            throw Error("need to migrate to CA schema, but this cannot be done in read-only mode");
+        }
     }
 
     /* Prepare SQL statements. */