Skip to content

Commit

Permalink
Merge pull request #649 from gares/quickfix-deprecated
Browse files Browse the repository at this point in the history
adapt to coq/coq#19300
  • Loading branch information
gares authored Jul 8, 2024
2 parents 50b305a + 1286184 commit f421547
Show file tree
Hide file tree
Showing 3 changed files with 19 additions and 3 deletions.
12 changes: 11 additions & 1 deletion src/coq_elpi_arg_syntax.mlg
Original file line number Diff line number Diff line change
Expand Up @@ -220,6 +220,16 @@ GRAMMAR EXTEND Gram
[ [ any_kwd -> { !the_kwd }] ];
END

{

[%%if coq = "8.19" || coq = "8.20"]
let mkAttributesFlagQualid q = Attributes.FlagIdent (Names.Id.to_string (Libnames.qualid_basename q))
[%%else]
let mkAttributesFlagQualid q = Attributes.FlagQualid q

Check failure on line 228 in src/coq_elpi_arg_syntax.mlg

View workflow job for this annotation

GitHub Actions / build (dev, 4.14-flambda)

Unbound constructor Attributes.FlagQualid
[%%endif]

}

ARGUMENT EXTEND attributes
PRINTED BY { pr_attributes }
END
Expand All @@ -236,7 +246,7 @@ GRAMMAR EXTEND Gram GLOBAL: attributes;
;
my_attr_value:
[ [ "=" ; v = string -> { Attributes.VernacFlagLeaf (Attributes.FlagString v) }
| "=" ; v = IDENT -> { Attributes.VernacFlagLeaf (Attributes.FlagIdent v) }
| "=" ; v = qualid -> { Attributes.VernacFlagLeaf (mkAttributesFlagQualid v) }
| "(" ; a = my_attribute_list ; ")" -> { Attributes.VernacFlagList a }
| -> { Attributes.VernacFlagEmpty } ]
]
Expand Down
6 changes: 5 additions & 1 deletion src/coq_elpi_builtins.ml
Original file line number Diff line number Diff line change
Expand Up @@ -939,9 +939,13 @@ let eval_of_constant c =
let eval_to_oeval = Evaluable.to_kevaluable
let mkCLocalAssum x y z = Constrexpr.CLocalAssum(x,None,y,z)
let pattern_of_glob_constr env g = Patternops.pattern_of_glob_constr env g
let warns_of_options options = options.user_warns
[%%endif]

[%%if coq = "8.20"]
let warns_of_options options = options.user_warns
[%%elif coq <> "8.19"]
let warns_of_options options = options.user_warns |> Option.map UserWarn.with_empty_qf
[%%endif]
let add_axiom_or_variable api id ty local options state =
let state, poly, cumul, udecl, _ = poly_cumul_udecl_variance_of_options state options in
let used = universes_of_term state ty in
Expand Down
4 changes: 3 additions & 1 deletion src/coq_elpi_vernacular.ml
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,9 @@ let atts2impl loc phase ~depth state atts q =
and convert_att att = convert_att_r att.CAst.v
and convert_atts l = List.map convert_att l
and convert_att_value = function
Attributes.FlagIdent s | Attributes.FlagString s -> AttributeString s
| Attributes.FlagIdent s [@if coq = "8.19" || coq = "8.20"] -> AttributeString s
| Attributes.FlagQualid q [@if coq <> "8.19" && coq <> "8.20"] -> AttributeString (Libnames.string_of_qualid q)
| Attributes.FlagString s -> AttributeString s
in
let phase = match phase with Summary.Stage.Interp -> "interp" | Summary.Stage.Synterp -> "synterp" in
let atts =
Expand Down

0 comments on commit f421547

Please sign in to comment.