Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adapt to coq/coq#19741 (STM worker spawning doesn't use Sys.argv directly) #428

Merged
merged 1 commit into from
Oct 24, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 2 additions & 1 deletion serapi/serapi_protocol.ml
Original file line number Diff line number Diff line change
Expand Up @@ -890,7 +890,8 @@ let exec_cmd (st : State.t) (cmd : cmd) : answer_kind list * State.t =
let doc = Stm.get_doc !doc_id in
coq_protect st @@ fun () -> match cmd with
| NewDoc opts ->
let stm_options = Stm.AsyncOpts.default_opts in
(* spawn_args probably wrong *)
let stm_options = Stm.AsyncOpts.default_opts ~spawn_args:[] in
let require_libs = Option.default [{Coqargs.lib="Stdlib.Init.Prelude"; prefix=None; export=Some Lib.Export; allow_failure=false}] opts.require_libs in
Stm.init_process stm_options;
let ndoc = { Stm.doc_type = Stm.(Interactive opts.top_name)
Expand Down
3 changes: 2 additions & 1 deletion sertop/sertop_init.ml
Original file line number Diff line number Diff line change
Expand Up @@ -130,7 +130,8 @@ let update_fb_handler ~pp_feed out_fmt =

(* Set async flags; IMPORTANT, this has to happen before STM.init () ! *)
let process_stm_flags opts =
let stm_opts = Stm.AsyncOpts.default_opts in
(* spawn_args probably wrong *)
let stm_opts = Stm.AsyncOpts.default_opts ~spawn_args:[] in
(* Process error resilience *)
let async_proofs_tac_error_resilience, async_proofs_cmd_error_resilience =
if opts.error_recovery
Expand Down
Loading