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

Unable to set Coq properties #339

Open
arnoudvanderleer opened this issue Aug 15, 2023 · 3 comments
Open

Unable to set Coq properties #339

arnoudvanderleer opened this issue Aug 15, 2023 · 3 comments

Comments

@arnoudvanderleer
Copy link

When I try to configure coq properties, following the last two lines of the documentation, I get an exception:

Anomaly ""Assert_failure _vendor+v8.16+32bit/coq/library/lib.ml:169:57"."
Please report at http://coq.inria.fr/bugs/.
Anomaly "Uncaught exception Option.IsNone."
Please report at http://coq.inria.fr/bugs/.

With this in the js console:
image
(This is the default examples/scatchpad.html, to which I added the option coq: {'Implicit Arguments': true, 'Default Timeout': 10})

@corwin-of-amber
Copy link
Member

@ejgallego I think you mentioned this at some point and had an idea of why it might pop up?

@arnoudvanderleer
Copy link
Author

@ejgallego
Copy link
Member

Yes, this was broken in some migration, Coq init routine is different so we need to tweak the code that fixes this.

It should not be hard, basically the options need now to be set at document creation time using the Stm.new_doc API. Before Coq 8.16 you were expected to set up the options before the document creation time. This is what now triggers the assertion.

I won't have time to look at this until next week due to holidays, feel free to write the patch yourselves.

In particular, you should put the options in the injections options here: https://github.com/coq/coq/blob/v8.16/stm/stm.mli#L60

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants