Skip to content

Commit

Permalink
"Warnings" and "Debug" options are synterp not interp
Browse files Browse the repository at this point in the history
  • Loading branch information
SkySkimmer committed Jan 9, 2025
1 parent 7c3baa6 commit 84f409c
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 2 deletions.
1 change: 1 addition & 0 deletions dev/ci/user-overlays/19981-SkySkimmer-classify-warn.sh
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
overlay metacoq https://github.com/SkySkimmer/metacoq classify-warn 19981
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
- **Fixed:**
:opt:`Debug` and :opt:`Warnings` are classified as Synterp.
This changes the scheduling during :cmd:`Import` such that putting `#[export] Set Warnings` around a specific command may change behaviour.
(`#19981 <https://github.com/coq/coq/pull/19981>`_,
by Gaëtan Gilbert).
4 changes: 2 additions & 2 deletions vernac/vernacentries.ml
Original file line number Diff line number Diff line change
Expand Up @@ -1967,15 +1967,15 @@ let () =
CWarnings.normalize_flags_string flags
in
declare_string_option ~preprocess
{ optstage = Summary.Stage.Interp;
{ optstage = Summary.Stage.Synterp;
optdepr = None;
optkey = ["Warnings"];
optread = CWarnings.get_flags;
optwrite = CWarnings.set_flags }

let () =
declare_string_option
{ optstage = Summary.Stage.Interp;
{ optstage = Summary.Stage.Synterp;
optdepr = None;
optkey = ["Debug"];
optread = CDebug.get_flags;
Expand Down

0 comments on commit 84f409c

Please sign in to comment.