Skip to content

Commit

Permalink
remove [--debug-store-digest-preimage] (ocaml#10329)
Browse files Browse the repository at this point in the history
this "feature" has been removed upstream

Signed-off-by: Rudi Grinberg <me@rgrinberg.com>
  • Loading branch information
rgrinberg authored and Philip White committed Apr 2, 2024
1 parent 09bd230 commit ed42fce
Show file tree
Hide file tree
Showing 4 changed files with 0 additions and 52 deletions.
16 changes: 0 additions & 16 deletions bin/common.ml
Original file line number Diff line number Diff line change
Expand Up @@ -595,7 +595,6 @@ module Builder = struct
; require_dune_project_file : bool
; watch_exclusions : string list
; build_dir : string
; store_digest_preimage : bool
; root : string option
; stats_trace_file : string option
; stats_trace_extended : bool
Expand Down Expand Up @@ -667,19 +666,6 @@ module Builder = struct
[ "debug-digests" ]
~docs
~doc:"Explain why Dune decides to re-digest some files")
and+ store_digest_preimage =
Arg.(
value
& flag
& info
[ "debug-store-digest-preimage" ]
~docs
~doc:
"Store digest preimage for all computed digests, so that it's possible to \
reverse them later, for debugging. The digests are stored in the shared \
cache (see --cache flag) as values, even if cache is otherwise disabled. \
This should be used only for debugging, since it's slow and it litters \
the shared cache.")
and+ no_buffer =
let doc =
"Do not buffer the output of commands executed by dune. By default dune buffers \
Expand Down Expand Up @@ -1025,7 +1011,6 @@ module Builder = struct
; require_dune_project_file
; watch_exclusions
; build_dir = Option.value ~default:default_build_dir build_dir
; store_digest_preimage
; root
; stats_trace_file
; stats_trace_extended
Expand Down Expand Up @@ -1182,7 +1167,6 @@ let build (builder : Builder.t) =
action_runner))
else `Forbid_builds
in
if builder.store_digest_preimage then Dune_engine.Reversible_digest.enable ();
if builder.print_metrics then Dune_metrics.enable ();
{ builder; root; rpc; stats }
;;
Expand Down
1 change: 0 additions & 1 deletion src/dune_engine/dune_engine.ml
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,6 @@ module Fs_cache = Fs_cache
module Fs_memo = Fs_memo
module Execution_parameters = Execution_parameters
module Cache_debug_flags = Cache_debug_flags
module Reversible_digest = Reversible_digest
module Report_errors_config = Report_errors_config
module Compound_user_error = Compound_user_error
module Reflection = Reflection
Expand Down
30 changes: 0 additions & 30 deletions src/dune_engine/reversible_digest.ml

This file was deleted.

5 changes: 0 additions & 5 deletions src/dune_engine/reversible_digest.mli

This file was deleted.

0 comments on commit ed42fce

Please sign in to comment.