Conversation
|
Why is the CI failing due to lack of source directories if the packages all have the correct |
|
It is because of a bug in opam-repo-ci. See ocurrent/opam-repo-ci#420. The PR was merged after the CI ran. |
|
ocurrent/opam-repo-ci#420 has also not been deployed yet. I'll fix this! |
|
thanks for the explanation! |
|
Thanks! |
| ["pkg-config" "--exists" "libevent"] | ||
| "pkgconf" {os = "win32" & os-distribution != "cygwinports"} | ||
| "--personality=i686-w64-mingw32" | ||
| {os = "win32" & os-distribution != "cygwinports" & | ||
| host-arch-x86_32:installed} | ||
| "--personality=x86_64-w64-mingw32" | ||
| {os = "win32" & os-distribution != "cygwinports" & | ||
| host-arch-x86_64:installed} | ||
| "pkg-config" {os != "Win32" | os-distribution != "cygwin"} | ||
| "libevent" |
There was a problem hiding this comment.
this dropped (on the non-win32 or cygwin) the --exists flag between pkg-config and libevent -- was this intentional?
There was a problem hiding this comment.
From the man page, --exists is implied when no other option is specified.
There was a problem hiding this comment.
@mtelvers Thanks for your comment. Since there are at least three pkg-config implementations, I'm not sure which one you're talking about. I find it very brittle to rely on the documented default of one of the pkg-config implementations.
There was a problem hiding this comment.
There was a problem hiding this comment.
I've had a quick check, and it seems to be the default in all of these implementations. There are multiple occurrences in opam-repository both with and without --exists and a variety of other parameters. :-)
5 participants
No description provided.