diff options
| author | Eelco Dolstra <eelco.dolstra@logicblox.com> | 2016-04-20 15:28:07 +0200 |
|---|---|---|
| committer | Eelco Dolstra <eelco.dolstra@logicblox.com> | 2016-04-20 15:28:07 +0200 |
| commit | ddea253ff8312bf1d2559999c4dd8d9cc3e5b240 (patch) | |
| tree | 5b2a0e8bdbaddcbab0b2f8124983d52577128d63 /src/libstore | |
| parent | c0c4ddcd9c19a29e3dfd26160929f3278150547f (diff) | |
RemoteStore: Propagate InvalidPath exceptions from the daemon
Diffstat (limited to 'src/libstore')
| -rw-r--r-- | src/libstore/remote-store.cc | 13 | ||||
| -rw-r--r-- | src/libstore/worker-protocol.hh | 2 |
2 files changed, 13 insertions, 2 deletions
diff --git a/src/libstore/remote-store.cc b/src/libstore/remote-store.cc index 430d0ecf1..1edf3662e 100644 --- a/src/libstore/remote-store.cc +++ b/src/libstore/remote-store.cc @@ -243,7 +243,18 @@ std::shared_ptr<ValidPathInfo> RemoteStore::queryPathInfoUncached(const Path & p { auto conn(connections->get()); conn->to << wopQueryPathInfo << path; - conn->processStderr(); + try { + conn->processStderr(); + } catch (Error & e) { + // Ugly backwards compatibility hack. + if (e.msg().find("is not valid") != std::string::npos) + throw InvalidPath(e.what()); + throw; + } + if (GET_PROTOCOL_MINOR(conn->daemonVersion) >= 17) { + bool valid = readInt(conn->from) != 0; + if (!valid) throw InvalidPath(format("path ‘%s’ is not valid") % path); + } auto info = std::make_shared<ValidPathInfo>(); info->path = path; info->deriver = readString(conn->from); diff --git a/src/libstore/worker-protocol.hh b/src/libstore/worker-protocol.hh index d62244d18..d133328d1 100644 --- a/src/libstore/worker-protocol.hh +++ b/src/libstore/worker-protocol.hh @@ -6,7 +6,7 @@ namespace nix { #define WORKER_MAGIC_1 0x6e697863 #define WORKER_MAGIC_2 0x6478696f -#define PROTOCOL_VERSION 0x110 +#define PROTOCOL_VERSION 0x111 #define GET_PROTOCOL_MAJOR(x) ((x) & 0xff00) #define GET_PROTOCOL_MINOR(x) ((x) & 0x00ff) |