Skip to content

Commit 71b0724

Browse files
committed
RExec: Use [Exit_code] instead of [Done]
Try to reflect that it's the remote service exit code
1 parent 8ba639c commit 71b0724

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

lib/rExec.ml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -145,7 +145,7 @@ module Client_flow = struct
145145
t.stderr_buf <- Cstruct.append t.stderr_buf data;
146146
`Ok t
147147
| `Ok (`Data_exit_code, data) ->
148-
`Done (Formats.Qrexec.get_exit_status_return_code data)
148+
`Exit_code (Formats.Qrexec.get_exit_status_return_code data)
149149
| `Ok (ty, _) ->
150150
Log.err Formats.Qrexec.(fun f -> f "unexpected message of type %ld (%s) received; \
151151
ignoring it" (int_of_type ty) (string_of_type ty));
@@ -154,7 +154,7 @@ module Client_flow = struct
154154

155155
let read t =
156156
let rec aux = function
157-
| `Eof | `Done _ as s -> Lwt.return s
157+
| `Eof | `Exit_code _ as s -> Lwt.return s
158158
| `Ok t ->
159159
let drain_stdout () =
160160
let output = t.stdout_buf in

lib/rExec.mli

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ module Client_flow : sig
1717
(* Write a formatted string to stdin *)
1818

1919
val read : t -> [`Stdout of Cstruct.t | `Stderr of Cstruct.t
20-
| `Eof | `Done of Cstruct.uint32] Lwt.t
20+
| `Eof | `Exit_code of Cstruct.uint32] Lwt.t
2121
(** Read from stdout and stderr *)
2222
end
2323

0 commit comments

Comments
 (0)