Adapt to https://github.com/rocq-community/micromega-plugin/pull/1 (tify)#251
Adapt to https://github.com/rocq-community/micromega-plugin/pull/1 (tify)#251fajb wants to merge 12 commits intorocq-prover:masterfrom
Conversation
|
This makes stdlib incompatible with all released versions of rocq |
|
Yes. What is the workaround?
What I see possible is
This is slow. |
Really? Deprecation warnings should not be fatal though. |
proux01
left a comment
There was a problem hiding this comment.
Yes, we won't be able to merge this until long. I'm offering a solution.
|
From what I recall, at least the test-suite checks for the exact results. Warnings makes this fail. |
|
Locally disable the warning then |
|
Ok, I see. I can try to do that. |
|
My opinion is that we should merge this instead of accruing technical debt to support unexpected combinations of rocq and stdlib. I raised this concern here when the stdlib split was proposed. |
|
Is this failure also due to compat somehow? |
|
I silenced globally the zify warnings. When this is deemed the right time, the stdlib can enable them. |
|
Global seems far too aggressive, please keep warning manipulation local. |
|
I don´t thing you can because otherwise each call to e.g. PS: Another option, no deprecation warnings in the tify PR (for now). When things stabilise, enable them. |
|
That sounds like the warning is badly implemented. |
|
I guess it's a similar problem as rocq-prover/rocq#21877 where the warning should be at intern time not interp time. |
In preparation of move to Corelib.
In preparation of move to Corelib.
In preparation of move to Corelib.
Was purely redundant with eKind
For instance conflicts between two libraries, one that would still be using the plugin shipped with Rocq and another that uses the new plugin.
1349c26 to
965932c
Compare
restructure components (isolate tify,zify)
PR rocq-community/micromega-plugin#1 generalises
zifybytify TwhereTis a target type.zifyis therefore almosttify Z.The PR removes introduced warnings and introduces a tify subcomponents.
It contains the existing Zify* files but could be complemented by
Rify.v,Qify.vetcOverlays (to be merged before the current PR)
Overlays (to be merged in sync with the current PR)