Skip to content

Add openssl to the macOS build instructions - #5209

Open
karpovantonme wants to merge 1 commit into
Orange-OpenSource:masterfrom
karpovantonme:doc/macos-openssl
Open

Add openssl to the macOS build instructions#5209
karpovantonme wants to merge 1 commit into
Orange-OpenSource:masterfrom
karpovantonme:doc/macos-openssl

Conversation

@karpovantonme

Copy link
Copy Markdown

Closes #5096

The macOS section installs pkg-config but not OpenSSL, so on a clean machine cargo build stops in openssl-sys before anything is compiled. The Linux section right above already names the native libraries and lists them per distribution, so this says the same thing for macOS and adds openssl to the brew line

Wording of the first sentence is taken from the Linux section so the two read the same way

Three files, because README.md and packages/hurl/README.md are generated: I edited docs/installation.md and regenerated both with bin/docs/build_readme.py github and ... crates, the way bin/docs/update_all.sh calls them

@jcamiel

jcamiel commented Aug 13, 2026

Copy link
Copy Markdown
Collaborator

Thanks for the PR @karpovantonme

A fresh macOS has no OpenSSL where openssl-sys looks for it, so cargo build
stops before it starts. The Linux section already lists the native libraries
per distribution; this says the same for macOS.

Closes Orange-OpenSource#5096
@karpovantonme

Copy link
Copy Markdown
Author

rebased onto master. 2 red windows jobs were libxml v0.3.21 failing its own build script rather than anything here, and your crates bump has landed since, so a fresh run should come back clean

Checks are sitting on approval to start

@karpovantonme

Copy link
Copy Markdown
Author

spoke too soon, the rebased run fails the same way. the line above the crate error is the real one:

C:\vcpkg\scripts\vcpkg-tools.json: error: document schema version 1 is not supported by this version of vcpkg

so the windows runner image has moved past the vcpkg checkout pinned here, and libxml never gets its library. nothing a PR can do about that, and since windows only runs on pull requests, master will not show it either

fine by me if you would rather close this one

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update build documentation on macOS

2 participants