Skip to content

Conversation

This started to spit a warning from 8.17 onwards, it is probably time to
get rid of this dubious behaviour.

Fixes rocq-prover#4949: [intuition] should not use [auto with *] by default.
@ppedrot ppedrot added this to the 9.2+rc1 milestone Sep 27, 2025
@ppedrot ppedrot added the kind: cleanup Code removal, deprecation, refactorings, etc. label Sep 27, 2025
@ppedrot ppedrot requested a review from a team as a code owner September 27, 2025 22:04
@ppedrot ppedrot added the request: full CI Use this label when you want your next push to trigger a full CI. label Sep 27, 2025
@coqbot-app coqbot-app bot removed the request: full CI Use this label when you want your next push to trigger a full CI. label Sep 27, 2025
Copy link
Contributor

coqbot-app bot commented Sep 27, 2025

🔴 CI failures at commit c6cd7d6 without any failure in the test-suite

✔️ Corresponding jobs for the base commit 17eccdd succeeded

❔ Ask me to try to extract minimal test cases that can be added to the test-suite

🏃 @coqbot ci minimize will minimize the following targets: ci-bbv, ci-category_theory, ci-cross_crypto, ci-ext_lib, ci-fiat_crypto_legacy, ci-fiat_parsers, ci-math_classes, ci-metarocq, ci-smtcoq
  • You can also pass me a specific list of targets to minimize as arguments.

@ppedrot
Copy link
Member Author

ppedrot commented Sep 28, 2025

Non trivial amount of broken code, which was not totally unexpected. I guess someone will have to write overlays...

ppedrot added a commit to ppedrot/bbv that referenced this pull request Sep 28, 2025
@ppedrot ppedrot added the needs: overlay This is breaking external developments we track in CI. label Sep 28, 2025
ppedrot added a commit to ppedrot/coq-ext-lib that referenced this pull request Sep 28, 2025
ppedrot added a commit to ppedrot/category-theory that referenced this pull request Sep 28, 2025
Copy link
Contributor

coqbot-app bot commented Sep 28, 2025

🔴 CI failures at commit c6cd7d6 without any failure in the test-suite

✔️ Corresponding jobs for the base commit 17eccdd succeeded

❔ Ask me to try to extract minimal test cases that can be added to the test-suite

🏃 @coqbot ci minimize will minimize the following targets: ci-category_theory, ci-cross_crypto, ci-ext_lib, ci-fiat_crypto_legacy, ci-fiat_parsers, ci-math_classes, ci-metarocq, ci-smtcoq
  • You can also pass me a specific list of targets to minimize as arguments.

ppedrot added a commit to ppedrot/smtcoq that referenced this pull request Sep 28, 2025
ppedrot added a commit to ppedrot/math-classes that referenced this pull request Sep 28, 2025
ppedrot added a commit to ppedrot/metarocq that referenced this pull request Sep 28, 2025
ppedrot added a commit to ppedrot/fiat that referenced this pull request Sep 28, 2025
ppedrot added a commit to MetaRocq/metarocq that referenced this pull request Sep 28, 2025
Copy link
Contributor

coqbot-app bot commented Sep 28, 2025

🔴 CI failures at commit c6cd7d6 without any failure in the test-suite

✔️ Corresponding jobs for the base commit 17eccdd succeeded

❔ Ask me to try to extract minimal test cases that can be added to the test-suite

🏃 @coqbot ci minimize will minimize the following targets: ci-category_theory, ci-cross_crypto, ci-ext_lib, ci-fiat_crypto_legacy, ci-fiat_parsers, ci-math_classes, ci-smtcoq
  • You can also pass me a specific list of targets to minimize as arguments.

ppedrot added a commit to rocq-community/math-classes that referenced this pull request Sep 28, 2025
Copy link
Contributor

coqbot-app bot commented Sep 28, 2025

🔴 CI failures at commit c6cd7d6 without any failure in the test-suite

✔️ Corresponding jobs for the base commit 17eccdd succeeded

❔ Ask me to try to extract minimal test cases that can be added to the test-suite

🏃 @coqbot ci minimize will minimize the following targets: ci-category_theory, ci-corn, ci-cross_crypto, ci-ext_lib, ci-fiat_crypto_legacy, ci-fiat_parsers, ci-smtcoq
  • You can also pass me a specific list of targets to minimize as arguments.

ckeller pushed a commit to smtcoq/smtcoq that referenced this pull request Sep 28, 2025
Copy link
Contributor

coqbot-app bot commented Sep 28, 2025

🔴 CI failures at commit c6cd7d6 without any failure in the test-suite

✔️ Corresponding jobs for the base commit 17eccdd succeeded

❔ Ask me to try to extract minimal test cases that can be added to the test-suite

🏃 @coqbot ci minimize will minimize the following targets: ci-category_theory, ci-corn, ci-cross_crypto, ci-ext_lib, ci-fiat_crypto_legacy, ci-fiat_parsers
  • You can also pass me a specific list of targets to minimize as arguments.

JasonGross pushed a commit to mit-plv/fiat that referenced this pull request Sep 28, 2025
Copy link
Contributor

coqbot-app bot commented Sep 28, 2025

🔴 CI failures at commit c6cd7d6 without any failure in the test-suite

✔️ Corresponding jobs for the base commit 17eccdd succeeded

❔ Ask me to try to extract minimal test cases that can be added to the test-suite

🏃 @coqbot ci minimize will minimize the following targets: ci-category_theory, ci-corn, ci-cross_crypto, ci-ext_lib, ci-fiat_crypto_legacy
  • You can also pass me a specific list of targets to minimize as arguments.

ppedrot added a commit to ppedrot/corn that referenced this pull request Sep 28, 2025
ppedrot added a commit to rocq-community/corn that referenced this pull request Sep 28, 2025
ppedrot added a commit to jwiegley/category-theory that referenced this pull request Sep 28, 2025
Copy link
Contributor

coqbot-app bot commented Sep 28, 2025

🔴 CI failures at commit c6cd7d6 without any failure in the test-suite

✔️ Corresponding jobs for the base commit 17eccdd succeeded

❔ Ask me to try to extract minimal test cases that can be added to the test-suite

🏃 @coqbot ci minimize will minimize the following targets: ci-cross_crypto, ci-ext_lib, ci-fiat_crypto_legacy
  • You can also pass me a specific list of targets to minimize as arguments.

@ppedrot
Copy link
Member Author

ppedrot commented Sep 29, 2025

@JasonGross fcl will need a bbv bump it seems.

ppedrot added a commit to rocq-community/coq-ext-lib that referenced this pull request Sep 29, 2025
Copy link
Contributor

coqbot-app bot commented Sep 29, 2025

🔴 CI failures at commit c6cd7d6 without any failure in the test-suite

✔️ Corresponding jobs for the base commit 17eccdd succeeded

❔ Ask me to try to extract minimal test cases that can be added to the test-suite

🏃 @coqbot ci minimize will minimize the following targets: ci-cross_crypto, ci-fiat_crypto_legacy, ci-http
  • You can also pass me a specific list of targets to minimize as arguments.

@JasonGross
Copy link
Member

@JasonGross fcl will need a bbv bump it seems.

dependabot + CI takes care of this automatically, cf mit-plv/fiat-crypto#2168. Or is there a bbv bump beyond this?

@ppedrot
Copy link
Member Author

ppedrot commented Sep 30, 2025

OK, then it should be fine.

ppedrot added a commit to ppedrot/coq-http that referenced this pull request Sep 30, 2025
@ppedrot
Copy link
Member Author

ppedrot commented Sep 30, 2025

Fixing cross-crypto is a bit annoying. It depends on FCF which calls intuition auto with * a lot, and fixing the code takes some time. I've done a little bit already but if anybody wants to take the baton I wouldn't complain.

@SkySkimmer
Copy link
Contributor

How annoying the fix is to make really depends on how precise you want it to be. In principle you could add at the beginning of every file

(* TODO stop using "with *" *)
Local Ltac intuition_solver ::= solve [auto with *].

and it should work.

@ppedrot
Copy link
Member Author

ppedrot commented Sep 30, 2025

Meh. Out of procrastination I started writing a script to parse the warning messages and autofix the issues.

ppedrot added a commit to ppedrot/fcf that referenced this pull request Sep 30, 2025
@ppedrot
Copy link
Member Author

ppedrot commented Sep 30, 2025

Here is the fcf PR: adampetcher/fcf#50. It needs to be merged and then the corresponding submodule must be bumped in cross-crypto.

ppedrot added a commit to ppedrot/cross-crypto that referenced this pull request Sep 30, 2025
liyishuai pushed a commit to liyishuai/coq-http that referenced this pull request Oct 1, 2025
ppedrot added a commit to ppedrot/fcf that referenced this pull request Oct 2, 2025
adampetcher pushed a commit to adampetcher/fcf that referenced this pull request Oct 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind: cleanup Code removal, deprecation, refactorings, etc. needs: overlay This is breaking external developments we track in CI.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[intuition] should not use [auto with *] by default
3 participants