Make it obvious when an error is thrown from the daemon rather than the client #379

New issue

Open

opened 2024-06-07 07:45:44 +00:00 by jade · 0 comments

jade commented

2024-06-07 07:45:44 +00:00

Owner

This is just useful user information. I am not exactly sure how to achieve it, because our exceptions all have what() that says nothing about what they are (very silly), but it can definitely be done with more thinking capacity than I have at hand rn.

The code to look at is src/libutil/serialise.cc in readError and RemoteStore::Connection::processStderr.

Also here on the daemon side: b8cb7abcf0/src/libstore/daemon.cc (L112-L127)

This is just useful user information. I am not exactly sure how to achieve it, because our exceptions all have `what()` that says nothing about what they are (very silly), but it can definitely be done with more thinking capacity than I have at hand rn. The code to look at is [`src/libutil/serialise.cc` in `readError`](https://git.lix.systems/lix-project/lix/src/a39ba22ff7112cd3984bbf28d8610d84dd525a0f/src/libutil/serialise.cc#L438-L460) and [`RemoteStore::Connection::processStderr`](https://git.lix.systems/lix-project/lix/src/a39ba22ff7112cd3984bbf28d8610d84dd525a0f/src/libstore/remote-store.cc#L969-L1035). Also here on the daemon side: https://git.lix.systems/lix-project/lix/src/b8cb7abcf08be01a291c970164a3f44dfca0ddbf/src/libstore/daemon.cc#L112-L127

jade added the

label 2024-06-07 07:45:44 +00:00

qyriad added the

Area/store

label 2024-06-07 18:13:17 +00:00

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: lix-project/lix#379

No description provided.