Skip to content

depexts/brew: Add support for casks and full-names#4801

Merged
rjbou merged 3 commits intoocaml:masterfrom
kit-ty-kate:fix-3
Aug 23, 2021
Merged

depexts/brew: Add support for casks and full-names#4801
rjbou merged 3 commits intoocaml:masterfrom
kit-ty-kate:fix-3

Conversation

@kit-ty-kate
Copy link
Copy Markdown
Member

Fixes #4800

@fdagnat
Copy link
Copy Markdown

fdagnat commented Aug 13, 2021

I would gladly test your patch but I do not how to do it. From documentation, I would infer that I need a magic invocation close to opam pin add opam-devel --dev but I suspect this one only upgrade to the trunk version.

Could you give me the spell please?

Thank you

@kit-ty-kate
Copy link
Copy Markdown
Member Author

kit-ty-kate commented Aug 13, 2021

cd /tmp && git clone -b fix-3 git://github.com/kit-ty-kate/opam.git && cd opam && make cold

The binary is then available as ./opam (which is a symlink to _build/default/src/client/opamMain.exe)
This won't require you to install any dependencies or do anything with the opam already installed.

@fdagnat
Copy link
Copy Markdown

fdagnat commented Aug 13, 2021

Ouch, after a long compilation time I get an error:

File "_none_", line 1:   
Error: The external function `unix_spawn' is not available
make[1]: *** [build-opam] Error 1
make: *** [cold] Error 2

@kit-ty-kate
Copy link
Copy Markdown
Member Author

(oops i forgot to get my branch in the command above, I've updated it)

uh, that a weird one :/ I'm not able to produce this on an macOS/arm64. Could you give a bit more of log? (the full one would be good, but at least the last command to see where we are)

@fdagnat
Copy link
Copy Markdown

fdagnat commented Aug 13, 2021

I had already changed the git repo...

log.txt

@kit-ty-kate
Copy link
Copy Markdown
Member Author

Ah I see the issue. I was able to reproduce locally, thanks for your help. It's an issue with the cold target when you also have bits of older compiler (in this case your 4.10.1 switch).

You can take it back from where you left it with:

env PATH="$(pwd)/bootstrap/ocaml/bin:$PATH" CAML_LD_LIBRARY_PATH= make

@fdagnat
Copy link
Copy Markdown

fdagnat commented Aug 16, 2021

Hello,

Sorry for the week end break. The compilation finished gracefully using your command. The compiled opam does not ask to install xquartz. So it seems to be solved.

Thank you

@rjbou
Copy link
Copy Markdown
Collaborator

rjbou commented Aug 23, 2021

Thanks!

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.

Opam keep reinstalling conf-libX11

3 participants