-
Notifications
You must be signed in to change notification settings - Fork 353
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
Fix opam on OpenBSD #6217
base: master
Are you sure you want to change the base?
Fix opam on OpenBSD #6217
Conversation
Works great on my OpenBSD desktop:
One test fails:
but looks unrelated (that |
Thanks. This PR fixes #6215 for me. My opinion may not matter on the patch, but that looked good to me too. I like the switch to |
with_process_in was fairly brittle and would display stderr by default and ignore errors. The new version of that function (renamed process_in) now uses Unix.open_process_full and returns None if anything went wrong, fixing the issue on OpenBSD where `getconf LONG_BIT` would fail.
5645f51
to
e2f67c1
Compare
turns out we still need to call the We could rely on I've also removed the C stub for FreeBSD as |
Queued on top of #6216
Fixes #6215
For the backport to 2.3, do we want to backport the entire PR or just the OpenBSD fix?