fix: disable threaded console when we use fork

This is similar to ocaml#8100 for the threaded console. The reason we're doing that is to avoid threads when fork is involved. Signed-off-by: Etienne Millon <me@emillon.org>
emillon · Jul 6, 2023 · a53cb5e · a53cb5e
1 parent bb06683
commit a53cb5e
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/CHANGES.md b/CHANGES.md
@@ -1,8 +1,8 @@
 Unreleased
 ----------
 
-- Disable background operations on MacOS and other Unixes where we rely on
-  fork. (#8100, fixes #8083, @rgrinberg)
+- Disable background operations and threaded console on MacOS and other Unixes
+  where we rely on fork. (#8100, #8121, fixes #8083, @rgrinberg, @emillon)
 
 - Add `dune build --dump-gc-stats FILE` argument to dump Garbage Collection
   stats to a named file. (#8072, @Alizter)

diff --git a/src/dune_config/config.ml b/src/dune_config/config.ml
@@ -146,7 +146,7 @@ let threaded_console =
   let t =
     { name = "threaded_console"
     ; of_string = Toggle.of_string
-    ; value = `Enabled
+    ; value = background_default
     }
   in
   register t;