Skip to content

Adapt to https://github.com/math-comp/math-comp/pull/1456#214

Merged
t6s merged 1 commit intoaffeldt-aist:masterfrom
proux01:mc1456
Apr 5, 2026
Merged

Adapt to https://github.com/math-comp/math-comp/pull/1456#214
t6s merged 1 commit intoaffeldt-aist:masterfrom
proux01:mc1456

Conversation

@proux01
Copy link
Copy Markdown
Contributor

@proux01 proux01 commented Apr 4, 2026

Adapt to math-comp/math-comp#1456 and cleanup of previous overlay.

And cleanup of previous overlay.
@t6s
Copy link
Copy Markdown
Collaborator

t6s commented Apr 4, 2026

It's interesting that most of the required changes are just the loading order of realType_ext. Were those early Requires introduced for the correct behavior of lra?

@proux01
Copy link
Copy Markdown
Contributor Author

proux01 commented Apr 4, 2026

The previous Require were for a previous version of math-comp/math-comp#1456 whose compatibility with already released versions of Stdlib was worse than the current version (the order of importing some files with some Register commands mattered a lot (at least for some transitory time)).

@proux01
Copy link
Copy Markdown
Contributor Author

proux01 commented Apr 4, 2026

@t6s CI green, this can be merged (the CI failure is unrelated and will hopefully be fixed by #215 )

@t6s
Copy link
Copy Markdown
Collaborator

t6s commented Apr 5, 2026

Looks good, and the CI is indeed green as much as external dependencies permit (i.e., only 9.1 & interval is failing).

@t6s t6s merged commit f57e313 into affeldt-aist:master Apr 5, 2026
12 of 13 checks passed
@proux01 proux01 deleted the mc1456 branch April 5, 2026 08:57
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

Successfully merging this pull request may close these issues.

2 participants