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

not-ocamlfind 0.14: windows bugfixes #27658

Merged
merged 6 commits into from
Mar 27, 2025

Conversation

chetmurthy
Copy link
Contributor

executable name needs .EXE suffix. Also updated cached copy of ocamlfind to latest (1.9.8)

executable name needs .EXE suffix.  Also updated cached copy of
ocamlfind to latest (1.9.8)
@chetmurthy
Copy link
Contributor Author

The only error is on "lint": this package depends on "xdot" (for displaying dependency-graph of packages). I can make "xdot" a separate conf-package if that would be preferable, but it's been this way for a while. Can you let me know if I should change it?

Comment on lines 33 to 35
conflicts: [
"frama-clang" { = "0.0.14" }
]
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Your commit message on that part seems to imply that frama-clang.0.0.14 is broken.

Was there a CI error?
Is it broken just in relation with not-ocamlfind? Or more generally?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This particular release seems busted when it shows up in revdeps across a number of packages I've released. And the build failure is down at the C level, so I suspect it's related to the package, not anything it depends upon.

Comment on lines +23 to +27
depexts: [
[
"xdot"
] {os-family = "debian"}
]
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We can probably leave as is for now. We can split out a conf-xdot package later.

Co-authored-by: Raphaël Proust <raphael-proust@users.noreply.github.com>
chetmurthy and others added 2 commits March 26, 2025 08:08
The C code has failed to build a number of times
when built as part of revdeps, and there are a
number of newer releases
Co-authored-by: Marcello Seri <mseri@users.noreply.github.com>
@chetmurthy
Copy link
Contributor Author

everything passed except freebsd, where the CI jobs died early:

Uncaught exception: Failure("\"zfs\" \"inherit\" \"mountpoint\" \"obuilder/result/8e3bb20e72d2e21c2187f05182d367f92781319767e9cbe8ce44a3088de266d0/rootfs\" failed with exit status 1")
2025-03-26 20:19.39: Job failed: Failed: Internal error

@mtelvers
Copy link
Contributor

Yea, this is because the FreeBSD workers have been updated to 14.2. I am about to push a PR to opam-repo-ci to update the test matrix.

@mseri
Copy link
Member

mseri commented Mar 27, 2025

Thanks

@mseri mseri merged commit 353a241 into ocaml:master Mar 27, 2025
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants