-
Notifications
You must be signed in to change notification settings - Fork 1.2k
frama-clang Version 0.0.19~beta #28843
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
base: master
Are you sure you want to change the base?
frama-clang Version 0.0.19~beta #28843
Conversation
|
Depend on Frama-C 32.0~beta: #28842 |
|
OK, thanks. Like for the other one, we should wait until #28842 is merged and then rerun CI for this PR. |
|
Retriggered a CI run here too, now that #28842 has been merged. |
|
Hm. CI logs is full of
There are several things at play here
|
|
This is indeed
This is indeed a recurring problem, and I think the root cause lies in the fact that conf-llvm and conf-libclang have differing views on how to install their external libraries if they are found to be missing. Now, looking at the synopsis of |
|
OK, 54f70db should ensure that |
|
I've now opened #28950 which hopefully should address the |
|
I've repeated the close-open dance and pressed 'rebuild all' in opam-ci now that #28950 has been merged. |
| "zarith" {>= "1.5"} | ||
| "camlp5" | ||
| "camlp-streams" | ||
| "conf-libclang" {>= "11.0.0" & < "20"} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| "conf-libclang" {>= "11.0.0" & < "20"} | |
| "conf-libclang" {>= "11.0.0"} |
I now realize a new all-green conf-libclang.21 is not going to help here! 😅
Could you explain why there is such a conservative upper bound here?
We generally try to avoid them unless there is a good reason for it https://github.com/ocaml/opam-repository/tree/master/governance/policies#5-strict-dependency-constraints-should-be-avoided---or-preventive-upper-bounds
Up until #28950 conf-libclang.15 was the repo's max version... 🤔
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There are known incompatibilities with clang 21. I have a branch to work on that, but it is not planned to be merged for this release of frama-clang, especially as it involves some changes to the architecture of the plug-in to better abide with clang's guidelines. Generally speaking, there are very often a few changes in libclang's API that break frama-clang between two major versions, hence the upper bound.
By the way, thanks for your efforts on this topic!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah, thanks for clarrifying! Sounds like we should roll a conf-libclang.19 as we also discussed...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added in #28968
No description provided.