Skip to content

Configure zarith and gmp with pkg-config on macOS with homebrew/macports - #22954

Closed
artempyanykh wants to merge 4 commits into
ocaml:masterfrom
artempyanykh:fix-zarith-gmp
Closed

Configure zarith and gmp with pkg-config on macOS with homebrew/macports#22954
artempyanykh wants to merge 4 commits into
ocaml:masterfrom
artempyanykh:fix-zarith-gmp

Conversation

@artempyanykh

Copy link
Copy Markdown

Benefits:

  1. Works fine when homebrew is installed in a custom location.
  2. Can use the same command-line for x64 and aarch64 macOS.

I did some local testing on

  • aarch64 mac with a homebrew in a custom location
  • x64 mac with a homebrew in the default location

Comment thread packages/conf-gmp/conf-gmp.4/opam
Comment thread packages/conf-gmp/conf-gmp.4/opam Outdated
Comment thread packages/zarith/zarith.1.12/opam Outdated
@artempyanykh

Copy link
Copy Markdown
Author

Thanks for the review @mseri! I extracted changes as naw package versions.

@artempyanykh artempyanykh changed the title Configure zarith and gmp with pkg-config on macOS/homebrew Configure zarith and gmp with pkg-config on macOS with homebrew/macports Jan 13, 2023
@artempyanykh

Copy link
Copy Markdown
Author

rebase

@artempyanykh

Copy link
Copy Markdown
Author

rebase

1. This makes conf-gmp behave similarly to other conf-* packages that use
pkg-config.
2. This works when homebrew is installed in a custom location.
Benefits:
1. Works fine when brew is installed in a custom location.
2. Can use the same command-line for x64 and aarch64 macOS.
@artempyanykh

Copy link
Copy Markdown
Author

rebase

@artempyanykh

Copy link
Copy Markdown
Author

@mseri what is the usual process here? There are failures on CI but they don't seem to be related to the change in this PR.

@mseri

mseri commented Feb 2, 2023

Copy link
Copy Markdown
Member

changes in gmp may have hidden repercussions on other packages, I need to check them, add some upper bounds and then make the new release. It is basically pending on actions from may part, sorry. I have been very busy in the past two weeks and, unfortunately, will be for one more week. After that this will be my priority

@artempyanykh

Copy link
Copy Markdown
Author

@mseri thank you! Didn't intend to push. I was mainly interested if there's something on my end that needs to be done.

mseri added a commit to mseri/opam-repository that referenced this pull request Feb 16, 2023
See ocaml#18129 and ocaml#22954

Signed-off-by: Marcello Seri <marcello.seri@gmail.com>
@mseri

mseri commented Feb 16, 2023

Copy link
Copy Markdown
Member

I have opened a draft PR with a simplified update on the same lines

@mseri mseri closed this Feb 16, 2023
@artempyanykh

artempyanykh commented Feb 16, 2023

Copy link
Copy Markdown
Author

@mseri in #23330 you left out a change to zarith. Was it intentional? Without this change we'll still have a hardcoded library path, which is a problem.

@mseri

mseri commented Feb 16, 2023

Copy link
Copy Markdown
Member

Hi @artempyanykh, thanks for the ping. I have done it on purpose, to check the breakage. I already have the change locally, will push it after having finished evaluating the CI failures

mseri added a commit to mseri/opam-repository that referenced this pull request Mar 20, 2025
See ocaml#18129 and ocaml#22954

Signed-off-by: Marcello Seri <marcello.seri@gmail.com>

Signed-off-by: Marcello Seri <marcello.seri@gmail.com>
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.

2 participants