-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
conf-postgresql - update depexts package name for macos + homebrew (#23912) #24105
conf-postgresql - update depexts package name for macos + homebrew (#23912) #24105
Conversation
haochenx
commented
Jul 14, 2023
•
edited
Loading
edited
- this should fix conf-postgres depext needs to be updated for homebrew #23912.
It seems that postgres installation via brew itself was successful, but since the installed packages (namely, I'll investigate into options to work around this. Log:
|
To "link" keg-only formulas, one can run But I have no idea how to achieve that in the opam file. |
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.
Sorry for the delay.
The behaviour by homebrew is a bit annoying to map in opam but we could do a couple of things:
- Mirror what we did for
conf-libssl
(https://github.com/ocaml/opam-repository/blob/9faf3dbf816f376733b73f3ed8832c4213db4a02/packages/conf-libssl/conf-libssl.4/files/homebrew.sh) and copy the binaries and libraries in the opam switch. However i feel a bit icky doing this for this package (it's fine for libssl as it is a central package and not much needs to be done anyway) because it's fairly brittle and not the best to maintain. - Ignore the failures in CI as there isn't that many packages that rely on postgresql in opam and display a
post-message
inviting the user to callbrew link
manually
We could also try to bake this behaviour in opam as a more long term solution. Could you open a ticket in the opam bugtracker to at least get the conversation started? https://github.com/ocaml/opam/issues
For now i think we could go with the second solution with a message looking like that? What do you think?
Co-authored-by: Kate <kit-ty-kate@outlook.com>
Thanks! |
Many thanks @kit-ty-kate! I have created the issue in ocaml/opam to kick off the discussion as advised. |